HashMap備忘録1

Question

###前提・実現したいこと
SQLから取得した情報をHashMapを使ってデータ整形したい。

###発生している問題・エラーメッセージ
HashMap<String,String[]>でputするときkeyが異なるはずだが
全てのValueが上書きされてしまう。

###該当のソースコード

String Result = executeInfo.toString();
String[] List = new String[12];

Map<String, String[]> Map = new HashMap<String, String[]>();
String[] matchList = null;

String KeyInfo = null;
String keyAddress = null;
String keyWebId = null;

String[] Recs = StringUtil.extractArrayString(Result, "\n"); //SQLから取得

for(String rec : Recs) {

String[] Columns = StringUtil.extractArrayString(rec, "\t"); //SQLから取得
keyWebId = resultSqlMailList[RESULT_SQL_WEBSITE_ID]; //SQLから取得
for(int i=0; i<3; i++) {

	keyAddress = Columns[i+1]; // SQLから取得

	if(StringUtil.isNull(keyAddress)) {
		continue;
	}

	matchList = Map.get(KeyInfo);

	if(Map.containsKey(KeyInfo)) {

		Map.put(KeyInfo, List);
	} 
} // End for

} // End for
// 出力データ整形
for (String[] resultCol : Map.values()) {
result.append(resultCol);
}

Answer 1

Value値を更新する場合、ループの外で変数宣言を行ってはいけない。
登録・更新する前で新規メンバ変数を作成しなければいけない

・・・・・・
for(~~~~
・・・・・・
if(Map.containsKey(KeyInfo)) {

String[] newList = ListなりmatchList なり登録・更新したいValue値
Map.put(KeyInfo, List);
}

Comments

[LouiS0616]

Javaの変数名はlowerCamelCaseにするのが原則です。
KeyInfoじゃなくてkeyInfoの方がいいと思いますよ。
MapとかListとか、一見独自クラスにしか見えないです。

[swordone]

「ループの外で変数宣言を行っている」ことが問題なのではなく、本質は「配列という参照型の扱いの不備」です。

[pilepierce]

LouiS0616さん
ご指摘ありがとうございます。
他の変数名はできているのでこういう凡ミスを無くしていきたいです・・。
命名センスが皆無なのも色んなソース見て直していきたいです。

[pilepierce]

swordoneさん
http://qiita.com/mdstoy/items/2ef4ada6f88341466783
こういうことですかね
もともとC++からやっていたので確かに参照渡しがなんたるかを理解しきれていないところはありました。
インスタンス生成をすることの重要性を肝に銘じて取り組んでいきたいと思います。