質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.10%

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

解決済

回答 4

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,100

score -2

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();

    }
}


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

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • azuapricot

    2019/08/28 16:34

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

    キャンセル

  • y_waiwai

    2019/08/28 16:34

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

    キャンセル

  • tetutetu

    2019/08/28 16:37 編集

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

    キャンセル

回答 4

+1

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

        System.out.println(args);

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

        this.itemList[0] =args;

    }


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

checkベストアンサー

0

参考程度にどうぞ

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);
        }
    }
}

出力結果

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/29 09:24

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

    キャンセル

  • 2019/08/29 09:48

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

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

    キャンセル

  • 2019/08/29 16:32

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

    キャンセル

0

原因は全て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);
    }  
  }
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/28 17:19

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

    キャンセル

  • 2019/08/28 17:27

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

    キャンセル

  • 2019/08/28 17:45

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

    キャンセル

-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を2,3冊買って、Javaの勉強をしてみては如何でしょうか?

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/28 16:45

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.10%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る