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

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

ただいまの
回答率

91.35%

  • Java

    10463questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Android

    5167questions

    Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

入れ子Listの中身が同じになる

解決済

回答 2

投稿 2017/11/30 14:28

  • 評価
  • クリップ 0
  • VIEW 63

miqa3893

score 4

前提・実現したいこと

自作APIからデータを取得し、その内容を入れ子Listに入れる処理を記述しています。
以下はそのリスト定義です。

class Hoge{
  private int mParam;
  private String mField;
  //コンストラクタなど以下略
}

//Hoge型のクラスを持つFugaクラス
class Fuga{
  private int mId;
  private String mName;
  private List<Hoge> mList;

  //コンストラクタなど中略
  public List<Hoge> getList(){
    return this.mList;
  }

  //以下略
}

//上記クラスを使ってListを定義したActivity
public class HomeActivity extends AppCompatActivity{
  //中略
  final List<Fuga> fugaList = new ArrayList<>();
  //以下略
}

発生している問題・エラーメッセージ

入れ子Listの中身が共通になり、参照しても同じデータが表示され、削除すると共通して削除され、何も残らなくなってしまいます。
もう少し具体的に申しますと、変数listに対してインデックスを指定してget()メソッドを実行しても、どのインデックスでもlist.get(index).getList()で同じ内容が表示されてしまいます。
Hoge・Fugaクラスでコンストラクタ、ゲッター・セッターは実装しておりますが、うまく行きません。

該当のソースコード

このソースコードはActivityの中に実装されており、バックグラウンド処理される部分です。
今回質問させていただいた内容で関連のある部分のみを、一部変更して抜粋しております。

//fugaListは、Activityのフィールドで以下のように定義しています
final List<Fuga> fugaList = new ArrayList<>();

private void addDataToList(JSONObject object) {

    String name,field;
        int id,param;

        try {
    //WebAPIから取得したJSONデータをパースし、Listに格納する
            JSONArray jsonArray = object.getJSONArray("history");
                for(int i = 0; i < jsonArray.length() ;i++) {
                    List<Hoge> hogeList = new ArrayList<>();
                        JSONObject temp = (JSONObject) jsonArray.get(i);
                        name = temp.getString("name");
                        idd = Integer.parseInt(temp.getString("machine"));

                        JSONArray jsonMeasureArray = temp.getJSONArray("measures");

                        for(int j = 0; j < jsonMeasureArray.length(); j++){
                            JSONObject w = (JSONObject) jsonMeasureArray.get(j);
                            param = Integer.parseInt(w.getString("param"));
                            field = String.valueOf(w.getString("field"));
                            hogeList.add(new Hoge(param,field));
                        }//for

                        fugaList.add(new Huga(id,name,hogeList));
                }//for
    }//try
    //以下略

}//adddataToList

試したこと

リストのスコープをメソッド内に変更したり、set()で上書きしたり、clear()で初期化してもうまく行きません。

補足情報

コンパイラ:Java 8
JVM実行環境:Android 5.0.1 Nexus 7
開発OS:Windows 10
IDE:Android Studio 3.0

初歩的な質問のことと思いますが、ご回答いただければ幸いです。よろしくお願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

+1

見た限り、何も問題無いように思います。
下記の行にブレークポイントを設置しhogeListの値が妥当かを確認してください。

fugaList.add(new Huga(id,name,hogeList));

投稿 2017/11/30 15:15

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/30 15:33

    ご回答ありがとうございます。
    自己解決内容に書いたとおり、WebAPIに問題があることが判明しました。Java側に問題がなかったと判明したことで、疑いを他にかけることができたことで進捗が捗りそうです。

    この度はありがとうございました。また機会がありましたら、よろしくお願いいたします。

    キャンセル

check解決した方法

0

閲覧頂きありがとうございます。

上記の質問ですが、yona様に頂いたご回答に従い、もう一度hogeListの中身を確認致しましたところ、やはり内容が同じでした。そこで疑いをWebAPIにかけたところ、WebAPIから返却されたデータが全て同一になっていたことが判明致しました。

いずれにしろ論理エラーですが、Javaの上では問題がないことが判明したのは進歩であると考えております。yona様に改めてお礼申し上げます。

投稿 2017/11/30 15:33

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

ただいまの回答率

91.35%

関連した質問

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

  • Java

    10463questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Android

    5167questions

    Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。