次のソースコード修正に関するヒントを教えてください。（大学の授業の課題）

Question

大学の授業の課題で次のようなものが出されました。「答え（ソースコード）」ではなく、「解く為のヒント」を教えてください。言語は”Java”です。

【問題】

以下は「商品の在庫管理に関するプログラム」である。
次のプログラムでは、コマンドライン引数として商品種類の数を指定できたが、
商品の初期在庫を指定できない。コマンドライン引数として、商品種類の数の代わりに、
商品の初期在庫を指定できるようにこのプログラムを修正し、プログラムを作成せよ。
コマンドライン引数がない場合は、デフォルトの要素数(5)で初期化する。
また、初期化直後の在庫も表示せよ。

該当のソースコード

import java.io.BufferedReader;
import java.io.InputStreamReader;
public class Kadai12_7 {
    public static void main(String[] args) throws Exception{
        int[] inventory = initialize(args);  //在庫データの初期化
        update(inventory);                   //在庫データの更新処理
        output(inventory);                   //現在の在庫データ表示
    }

    //在庫変更データの入力
    static int[] input() throws Exception{
        BufferedReader buf = new BufferedReader(new InputStreamReader(System.in));
        System.out.print("商品番号?");
        int itemnumber = Integer.parseInt(buf.readLine());

        System.out.print("個数増減?");
        int itemchange = Integer.parseInt(buf.readLine());

        //ここで要素数2の1次元整数配列を生成し
        //在庫変更データを設定した後
        //戻り値として配列を返したい
        
        int item[] = new int[2];
        item[0] = itemnumber;
        item[1] = itemchange;
        
        return item;
        
    }

    //在庫データの表示
    static void output(int[] inventory){
        System.out.println("商品在庫:");

        //ここで配列 inventory を forループで巡回し
        //在庫にある商品番号(要素番号)と個数を全て表示する
        
        for( int a=0; a<inventory.length; a++){
            System.out.println("商品 "+a+" 個数= "+inventory[a]);
        }
    }

    //在庫データの更新
    static void change(int[] inventory, int[] item){
        //ここで引数int[] itemの在庫変更データを参照し
        //在庫変更データに従い引数 int[] inventory の在庫データを更新
        inventory[(item[0])]=inventory[(item[0])]+item[1];
    }

    //在庫データの初期化
    static int[] initialize(String[] args){
        int size = 5;                         //デフォルトの要素数
        if(args.length > 0){                  //実行時引数がある場合
            size = Integer.parseInt(args[0]); //実行時引数で要素数を設定
        }

        //1次元整数配列 inventory を 要素数 size で生成
        //生成した配列を戻り値として返す
        
        int inventory[] = new int[size]; //配列生成
        
        return inventory; //戻り値設定
    }

    //在庫データの繰り返し更新処理
    static void update(int[] inventory) throws Exception{

        int[] item = input();        //商品増減入力 

        //while文を用いて商品番号(item[0])がマイナスの値で無い限り
        //下記更新処理を繰り返し行う
        while(item[0] > 0){
            change(inventory, item); //在庫データ更新
            output(inventory);       //現在の在庫データ表示
            item = input();          //商品増減入力
        }
        //繰り返しここまで
    }
}

###現時点で修正をした箇所
◎修正前

public static void main(String[] args) throws Exception{
        int[] inventory = initialize(args);  //在庫データの初期化
        update(inventory);                   //在庫データの更新処理
        output(inventory);                   //現在の在庫データ表示
    }

◎修正後

public static void main(String[] args) throws Exception{
        int[] inventory = initialize(args);  //在庫データの初期化
        output(inventory);                   //初期化直後の在庫データ表示
        update(inventory);                   //在庫データの更新処理
        output(inventory);                   //現在の在庫データ表示
    }

上記のソースコードを実行した時の出力結果

（”商品番号”と”個数増減”は標準入力で、商品番号がマイナスになった時点で処理は終了する。）

商品番号?1
個数増減?2
商品在庫:
商品 0 個数= 0
商品 1 個数= 2
商品 2 個数= 0
商品番号?2
個数増減?5
商品在庫:
商品 0 個数= 0
商品 1 個数= 2
商品 2 個数= 5
商品番号?2
個数増減?2
商品在庫:
商品 0 個数= 0
商品 1 個数= 2
商品 2 個数= 7
商品番号?-1
個数増減?0
商品在庫:
商品 0 個数= 0
商品 1 個数= 2
商品 2 個数= 7

修正後の出力結果（コマンドライン引数＝1 3 2　である時）

商品在庫:
商品 0 個数= 1
商品 1 個数= 3
商品 2 個数= 2
商品番号?1
個数増減?2
商品在庫:
商品 0 個数= 1
商品 1 個数= 5
商品 2 個数= 2
商品番号?2
個数増減?3
商品在庫:
商品 0 個数= 1
商品 1 個数= 5
商品 2 個数= 5
商品番号?2
個数増減?2
商品在庫:
商品 0 個数= 1
商品 1 個数= 5
商品 2 個数= 7
商品番号?-1
個数増減?0
商品在庫:
商品 0 個数= 1
商品 1 個数= 5
商品 2 個数= 7

補足情報（使用言語など）

使用しているプログラミング言語はJavaです。

またプログラム作成にはJ-doodleを使用しています。

現時点で自身で分かっている事と分かっていない事は
『初期化後に現在の在庫データを表示する事およびどこを修正するか』は分かっており、
『初期化に関わる部分(initializeメソッド)を修正する事で、目的の出力結果が出る。（要は、この部分を修正すれば、初期化直後の在庫データの表示、つまりは目的の出力結果の表示が可能と考えた。）』事も分かっていますが『その部分の中でも何処を、どのように修正したら良いか。』が分かりません。

※【追記１】現時点での分かっている事、分からない事を追記しました。情報に不足があり、ただ単なる答え丸投げのような書き方に見えてしまい、申し訳ございません。私自身には一切そのような気持ちはありません。授業内容や教科書の参照、先生からのアドバイスをもとに何回も修正・出力を繰り返しましたが、それでも分かりませんでしたので"ヒントをいただきたい"という名目で質問させていただきました。

※【追記２】現時点で、自身で修正した部分のソースコードと『初期化に関わる部分(initializeメソッド)を修正する事で、目的の出力結果が出る。』の理由を直後に（）書きで追記しました。

Comments

[hoshi-takanori]

「解く為のヒント」を教えてください。と言われても、問題文をそのままコピペしただけで、それに対してあなたが何をどう理解して、何が分からないのかということを書いてくれないとヒントの出しようがないし、単なる丸投げですね。

[Y.H.]

>『初期化に関わる部分(initializeメソッド)を修正する事で、目的の出力結果が出る。』事も分かっていますが
このように考えた根拠を詳細に記載されると良いと思います。

[hoshi-takanori]

> 『初期化に関わる部分(initializeメソッド)を修正する事で、目的の出力結果が出る。』
は明らかなので、むしろこのメソッドでは現在何が行われていて、問題文は何を要求しているか、そのために何を変更すべきかを考えるべきでしょうね。
また、実際に修正したコードがあるなら、それを貼ってくれたら有益なアドバイスができる可能性がありますよ。

[hoshi-takanori]

2021/01/19 00:30 の編集内容を確認しました。main メソッドで initialize を呼んだ直後に output を呼ぶことで、「また、初期化直後の在庫も表示せよ。」は実現できますね。(実際には、インデントが全角スペースのためにエラーになったので、これを半角スペースに修正する必要がありますが。)

次は、「コマンドライン引数として、商品種類の数の代わりに、商品の初期在庫を指定できるようにこのプログラムを修正し」の部分を頑張りましょう。

Answer 1

ソースコードの色々な部分をいじって、出力して・・・を繰り返していましたら、目的の結果となりましたので、修正後のソースコードを掲示します。

質問にご協力していただいた皆様、本当にありがとうございました。

import java.io.BufferedReader;
import java.io.InputStreamReader;
public class Kadai12_8{
    public static void main(String[] args) throws Exception{
        int[] inventory = initialize(args);  //在庫データの初期化
        output(inventory);                   //初期化直後の在庫データ表示
        update(inventory);                   //在庫データの更新処理
        output(inventory);                   //現在の在庫データ表示
    }

    //在庫変更データの入力
    static int[] input() throws Exception{
        BufferedReader buf = new BufferedReader(new InputStreamReader(System.in));
        System.out.print("商品番号?");
        int itemnumber = Integer.parseInt(buf.readLine());

        System.out.print("個数増減?");
        int itemchange = Integer.parseInt(buf.readLine());

        //ここで要素数2の1次元整数配列を生成し
        //在庫変更データを設定した後
        //戻り値として配列を返したい
        
        int item[] = new int[2];
        item[0] = itemnumber;
        item[1] = itemchange;
        
        return item;
        
    }

    //在庫データの表示
    static void output(int[] inventory){
        System.out.println("商品在庫:");
        //ここで配列 inventory を forループで巡回し
        //在庫にある商品番号(要素番号)と個数を全て表示する
        
        for( int a=0; a<inventory.length; a++){
            System.out.println("商品 "+a+" 個数= "+inventory[a]);
        }
    }

    //在庫データの更新
    static void change(int[] inventory, int[] item){
        //ここで引数int[] itemの在庫変更データを参照し
        //在庫変更データに従い引数 int[] inventory の在庫データを更新
        inventory[(item[0])]=inventory[(item[0])]+item[1];
    }

    //在庫データの初期化
    static int[] initialize(String[] args){
        int size = 5;                         //デフォルトの要素数
        if(args.length > 0){                  //実行時引数がある場合
            size = Integer.parseInt(args[(args.length-2)]); //実行時引数で要素数を設定
        }

        //1次元整数配列 inventory を 要素数 size で生成
        //生成した配列を戻り値として返す
        
        int inventory[] = new int[size]; //配列生成
        
        for(int b=0; b<args.length; b++){
            
            inventory[b] = Integer.parseInt(args[b]);
            
            
            
        }
        return inventory; //戻り値設定
    }

    //在庫データの繰り返し更新処理
    static void update(int[] inventory) throws Exception{

        int[] item = input();        //商品増減入力 

        //while文を用いて商品番号(item[0])がマイナスの値で無い限り
        //下記更新処理を繰り返し行う
        while(item[0] > 0){
            change(inventory, item); //在庫データ更新 
            output(inventory);       //現在の在庫データ表示
            item = input();          //商品増減入力 
        }
        //繰り返しここまで
    }
}

/*（コマンドライン引数＝1 3 2）
商品在庫:
商品 0 個数= 1
商品 1 個数= 3
商品 2 個数= 2
商品番号?1
個数増減?2
商品在庫:
商品 0 個数= 1
商品 1 個数= 5
商品 2 個数= 2
商品番号?2
個数増減?3
商品在庫:
商品 0 個数= 1
商品 1 個数= 5
商品 2 個数= 5
商品番号?2
個数増減?2
商品在庫:
商品 0 個数= 1
商品 1 個数= 5
商品 2 個数= 7
商品番号?-1
個数増減?0
商品在庫:
商品 0 個数= 1
商品 1 個数= 5
商品 2 個数= 7
*/

/*(コマンドライン引数無し)
商品在庫:
商品 0 個数= 0
商品 1 個数= 0
商品 2 個数= 0
商品 3 個数= 0
商品 4 個数= 0
商品番号?1
個数増減?2
商品在庫:
商品 0 個数= 0
商品 1 個数= 2
商品 2 個数= 0
商品 3 個数= 0
商品 4 個数= 0
商品番号?2
個数増減?3
商品在庫:
商品 0 個数= 0
商品 1 個数= 2
商品 2 個数= 3
商品 3 個数= 0
商品 4 個数= 0
商品番号?2
個数増減?2
商品在庫:
商品 0 個数= 0
商品 1 個数= 2
商品 2 個数= 5
商品 3 個数= 0
商品 4 個数= 0
商品番号?-1
個数増減?0
商品在庫:
商品 0 個数= 0
商品 1 個数= 2
商品 2 個数= 5
商品 3 個数= 0
商品 4 個数= 0
*/

Answer 2

質問者さんの理解度が把握しきれないのでヒントを出すのが難しい・・・
とくに考えるようなものでなく、配列に値を設定してください。という問題なので

既存のコードで

• 配列の生成方法
• 配列の値の変更
• 配列のループ
• コマンドライン引数の数の確認（0と比較して有無を確認している）
• コマンドライン引数をintに変換

と、サンプルはそろってるので、配列の使い方についてググらなくても解けるレベルだと感じます。

『初期化に関わる部分(initializeメソッド)を修正する事で、目的の出力結果が出る。（要は、この部分を修正すれば、初期化直後の在庫データの表示、つまりは目的の出力結果の表示が可能と考えた。）』事も分かっていますが『その部分の中でも何処を、どのように修正したら良いか。』が分かりません。

initializeメソッドの戻り値が何か、わかりますか？
initializeメソッドの戻り値に対して現状から引数の値を設定すればよいです。
既存のコードで書かれていますが、引数はString(String[])なのでintへの変換が必要です。
配列の値の変更は、ちょっと読みにくく感じるかもしれませんが、changeメソッドですでにやっています。

端的にかけば

int[] hoge = {0, 1, 2};
hoge[0] = 123;

とすると、配列hogeは{123, 1, 2}に変わります。

要は、この123にあたる部分をコマンドライン引数から受け取って(int変換して)変更すればよいわけです。

コマンドライン引数として、商品種類の数の代わりに、

”代わり”とあるので、商品種類の数の指定はなくなるように感じますが・・・
商品種類の数は、どのように決めるのか不明です（コマンドライン引数の数＝種類数？）

Comments

[IMT2174]

回答ありがとうございます。
ご指摘がありました"商品種類の数"ですが、担当の先生に確認してみたいと思います。(分かり次第、このコメント欄にてお知らせ致します。)