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

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

新規登録して質問してみよう
ただいま回答率
85.48%
Java

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

ループ

ループとは、プログラミングにおいて、条件に合致している間、複数回繰り返し実行される箇所や、その制御構造を指します

ArrayList

Java用のタグです。arrayListはListインターフェースを実装した、リサイズ可能な配列用クラスです。

Q&A

解決済

1回答

1090閲覧

Java List<String>のループがうまくいかない

meex

総合スコア63

Java

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

ループ

ループとは、プログラミングにおいて、条件に合致している間、複数回繰り返し実行される箇所や、その制御構造を指します

ArrayList

Java用のタグです。arrayListはListインターフェースを実装した、リサイズ可能な配列用クラスです。

0グッド

1クリップ

投稿2021/07/12 22:42

ソース

Java

1List<List<List<String>>> masterData = new ArrayList<>(); 2List<List<String>> graphData = new ArrayList<>(); 345 6//----------------------------- 7//各グラフの値を昇順に並べていく 8//----------------------------- 9int idxOf; 10List<String> value = new ArrayList<>(); 11for (int i = 0; i < GRAPH_DATA_MAX_NUM/*=8*/; ++i){ //データの種類分 12 value.clear(); 13 //timestamp 14 for (String time:timeStampData) { 15 idxOf = masterData.get(i).get(0).indexOf(time); //各値の日付データ[0]==timestampを比較 16 //同じ日が見つかった場合 17 if (-1 != idxOf){ 18 value.add(masterData.get(i).get(1).get(idxOf)); 19 } 20 //見つからなかった場合 21 else{ 22 value.add("null"); 23 } 24 } 25 // Collections.addAll(graphData, value); 26 graphData.add(value); 27 System.out.println(i+":::::"+ value); 28}

躓いているところ

ソースで

Java

1System.out.println(i+":::::"+ value);

となっている箇所のコンソール出力を見ると、

リアルタイムループ: 0:::::[null, null, null, null, null, null, 19462, 22934, 30752, 9120, 41141, null, null, 42785, null, 34826, 45279, 47078, 39571, null, null, 49175, 51560] 1:::::[null, null, null, null, -2, null, null, null, null, null, null, 4620, 4470, null, -2, null, null, null, null, 4470, null, null, null] 2:::::[null, null, null, null, null, 7, null, null, 6, null, null, null, null, 5, null, null, null, null, null, null, null, null, null] 3:::::[null, null, null, 2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null] 4:::::[null, null, null, null, null, -2, null, null, null, null, 4380, 4680, 4380, null, null, null, null, null, null, 4380, 4680, null, null] 5:::::[null, null, -2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null] 6:::::[null, -2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null] 7:::::[-2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null] ```となります。 ⇒これは想定通りの結果です。 しかし、すべてのforが終わったあとに、graphDataを見ると、 ```ここに言語を入力 graphData [[-2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null], [-2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null], [-2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null], [-2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null], [-2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null], [-2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null], [-2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null], [-2, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null, null]] ```となってしまいます。 見たところ、最後の「7」番目のループ時の値で、すべて更新されてしまっているようです。 なぜ、こうなるのでしょうか? やりたいことは、 ```ここに言語を入力 リアルタイムループの8個の配列を、正しくgraphDataに格納したい ```です。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

y_waiwai

2021/07/12 23:03

どうなるはずなんでしょうか 問題が見えません
guest

回答1

0

ベストアンサー

List<String> value = new ArrayList<>();をループの外側で1つしか作っていないので、graphDataの各要素は、全て同じオブジェクトになっています。

投稿2021/07/12 23:04

maisumakun

総合スコア145184

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

meex

2021/07/13 09:38

的確なご指摘ありがとうございます! for内で定義したところ、動かしたい挙動になりました。素早いご回答大変感謝しております。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問