引数のString型からString[]型に格納

Question

public class DraemonPoket {
	//配列型のフィールドを作成
	private String[] itemList;

	//長さをのフィールド
	private int index;

	//コンストラクターで引数を受け取る。
	public DraemonPoket(int length){

		this.index =length;
		System.out.println(this.index);
	}
	//道具のメソッドで引数を受け取る。
	public void setItem(String args){

		System.out.println(args);

		this.itemList = new String[this.index];

		this.itemList[0] =args;

	}

	//アイテムを表示をするメソッド
	public void showItem(){

		for(String result : this.itemList){
			System.out.println(result);
		}
	}

public class Main14 {

	public static void main(String [] args){

		//インスタンスを作成してコマンドライン引数の長さを渡す。
		DraemonPoket item = new DraemonPoket(args.length);

		item.setItem(args[0]);
		item.setItem(args[1]);
		item.setItem(args[2]);
		item.setItem(args[3]);

		item.showItem();

	}
}

コマンドライン引数で４つ道具をargsに渡しています。
その道具をフィールドのprivate String[] itemListに一つずつ格納をしてshowItemで表示をたいです。

いろいろ試して自分で解決できないので質問をさせていただきました。
すみません。
よろしくお願いします。

Comments

[azuapricot]

ListなのかArrayなのかわかんなくなるんで配列にListって使わないほうがいいよーな

[y_waiwai]

その４つをどうargsに格納してますか？
実際のargsの内容を提示してください

[azuapricot]

String[] でやる必要があるのかも疑問でList<String>じゃだめなんですかね？（私が配列嫌いなだけ）

[tanaka002]

args の中身は
１タケコプター
２翻訳こんにゃく
３スモールライト
４タイムふろしき

になります。

[tanaka002]

自分での理解の為にString[]でやっています。。。

[azuapricot]

this.itemList[0] =args;　これいつまでたっても道具1個しか入らないよーにみえるんですけど・・・
今何に詰まっているのか詳細にかきましょう

[y_waiwai]

その４つをArgsにどう格納してるんでしょうか？

[tanaka002]

item.setItem(args[0]);
item.setItem(args[1]);
item.setItem(args[2]);
item.setItem(args[3]);
mainの方からメソッドで引数として渡してます。

[azuapricot]

mainも書いて回答者が再現しやすくすると良いですね

[y_waiwai]

なんかはなしが違ってますが。。

[tetutetu]

コンストラクタか。よく読んでませんでした。消します。

Answer 1

    //道具のメソッドで引数を受け取る。
    public void setItem(String args){

        System.out.println(args);

        this.itemList = new String[this.index];

        this.itemList[0] =args;

    }

このメソッドを何度起動しても、配列を生成してその0番目に引数要素を格納する、ということを繰り返すのみで、配列再生成の時点で前に入れた情報が消滅します。
新たにアイテムを入れるときのために、｢次はどこに入れればいいか｣の情報を持っておく必要があります。

Answer 2

原因は全てsetItemメソッドにあります

まず、質問者様のコードですと
this.itemList[0] =args;で
常に配列の0番目に引数であるargsを入れ続けます
恐らくitem.showItem()の出力結果が

・args[3]の中身
・null
・null
・null

になっているのではないでしょうか。
それを修正するために、まずはsetItemメソッドの引数に配列のインデックスを追加します。

item.setItem(args[0],0);
item.setItem(args[1],1);
...
//省略

public void setItem(String args, int hoge){
   //省略
   this.itemList[hoge] =args;
}

（正直for文でやった方が絶対良いのですが、まだ学ばれていないのでしょうか。）

ここまでで実行した場合
・null
・null
・null
・args[3]の中身
という出力になると思います。

原因はこれまたsetItemメソッド内で

this.itemList = new String[this.index];

と、毎回配列を初期化しているためです。

正常に出力する方法としては
・setItemメソッドにargsを配列として渡す
（この場合、前述の配列のインデックスを引数に追加する必要はないです）
・DraemonPoketで配列を初期化し、setItemメソッドでは初期化を行わない

のどちらかになります。

こんなですかね

Main

public class Main {

  public static void main(String[] args) {
    DraemonPoket item = new DraemonPoket(args.length);

    for(int i=0;i<args.length;i++) {
      item.setItem(args[i],i);
    }

    item.showItem();
  }

DraemonPoket

public class DraemonPoket {

  private String[] itemList;

  public DraemonPoket(int length){
    this.itemList = new String[length];
  }

  public void setItem(String args, int num){
    this.itemList[num] =args;
  }

  public void showItem(){
    for(String result : this.itemList){
      System.out.println(result);
    }  
  }
}

Comments

[azuapricot]

setItem()の引数にインデックス渡す必要ありますかね・・・（まぁかんたんですけど）

[tetutetu]

ローカル環境ではfor文で書いたので
forで回すなら中身が
item.setItem(args[i],i);
だけで済むので楽かなと思った次第です。
DraemonPoketクラス側で変数を増やすか引数を増やすか程度の違いかなと…

[tetutetu]

とりあえず今一番気になってるのは処理の中身より「DraemonPoket」のスペルなんですが…
蛇足か。

Answer 3

参考程度にどうぞ

Main
※めんどくさいのでコマンドラインからじゃなくて直接渡してます

public class Main {
    public static void main(String[] args) throws Exception {
        //インスタンスを作成してコマンドライン引数の長さを渡す。
        DraemonPoket item = new DraemonPoket(4);

        item.setItem("あんきぱん");
        item.setItem("たけこぷたー");
        item.setItem("どこでもどあ");
        item.setItem("とうめいまんと");

        item.showItem();
    }
}

どらえもん

public class DraemonPoket {
    private String[] itemList;

    private int index;
    
    // 配列格納用のカウント 0スタート
    private static int count = 0;

    public DraemonPoket(int length){
        // コンストラクタで配列のサイズを決定
        this.itemList = new String[length];
        this.index =length;
    }

    public void setItem(String args){

        // index = countにつめる
        this.itemList[this.count] =args;

        // count のかうんとあっぷ
        this.count++;
    }

    public void showItem(){

        for(String result : this.itemList){
            System.out.println(result);
        }
    }
}

出力結果

あんきぱん
たけこぷたー
どこでもどあ
とうめいまんと

Comments

[azuapricot]

（もっとかっこいいやりかたありそう）

[tetutetu]

＞インスタンスを作成してコマンドライン引数の長さを渡す。
DraemonPoket item = new DraemonPoket(args.length);
の方がコメントには合ってる気がします。
追：いや面倒で直接渡してるからか。失礼しました。

＞this.itemList = new String[length];
で書くなら変数index自体が不要な気もしますね…
他でindex使ってるわけでもないですし

[tanaka002]

皆様ありがとうございます！
フィールドに渡ってきた引数を代入していたことが間違いでした。

[tetutetu]

>フィールドに渡ってきた引数を代入していたことが間違いでした。

配列をnewする場所が問題　と認識していただければと思います。
私の回答の方で、配列を引数としてsetItemに渡すのであればここでnewしても問題ないとは記載しましたが、わざわざコンストラクタで受け取った数値をindexに代入するのであれば
そこでindexの長さ分の配列を定義
(this.itemList = new String[index];)
してしまった方が、処理の流れを読むのも簡単になるかと。

[退会済みユーザー]

マイナスつけたくなるstaticの扱い

Answer 4

Javaのプログラムは、下のプログラムのようにクラスを定義して、main関数の中で実行するコードを書きます。
このようなプログラムを、HelloWorld.javaというファイル（クラス名と一致する名前のファイル）に書き込んでおいて、javacコマンドでコンパイルし、javaコマンドで実行します。

public class HelloWorld{
   public static void main(String[] args){
     System.out.println("Hello World!!");
   }
}

ところが、質問に書かれているプログラムには、クラスの定義も、main関数もありません。

Javaのプログラミングの初歩のところで躓いているように思われますので、「はじめてのJava」的なタイトルのMookを２，３冊買って、Javaの勉強をしてみては如何でしょうか？

まずは、コンパイルできるようなJavaのコードを書けるようになりましょう！

Comments

[azuapricot]

書いてるけど省いてるってコードよく見ればわかるんで回答としては間違ってるかなーってことで低評価押しました