hashMap.getその２

質問

たびたび、ほんとすいませんが
またご質問させて下さい。

先ほど、以下の通り教えて頂きまして
訂正したところエラーは出なくなりました。

MyData myData = (MyData) hashMap.get("A");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);

データは以下の通りです。
－－－－－－－－－－－－－－－
A,2003/1/1,スイカ
B,2003/2/1,バナナ
C,2003/3/1,メロン
D,2003/4/1,イチゴ
－－－－－－－－－－－－－－－

正しい表示は、
「日付：2003/1/1　名前：スイカ」
と、表示されなければいけませんが
「日付：2003/4/1　名前：イチゴ」
と、一番最後のレコードが表示されてしまいます。

一行追加しても、やはり一番最後のレコードが表示されてしまいます。

System.out.printlnで確認したところ、hashMapのキーはきちんとセットされているようです。

ご教授して頂けたら幸いです。

回答No.2

okure_donchan · ベストアンサー

－－－－－－－－－－－－－－－

import java.io.*;
import java.util.*;

public class TestMap
{
/*HashMap生成(クラス変数)*/
class MyData{
public String sDate;
public String sName;
}

private static HashMap hashMap = new HashMap();

/*追加：入力キー(クラス変数)*/
public static String inputKey = null;
private String[] keys = {};

public static void main(String args[]){
try{

TestMap testMap = new TestMap();
testMap.setMap("test.csv");

/*Keyに対応する出力の判断*/
if(testMap.checkKey(args[0])){

MyData myData = (MyData) hashMap.get("A");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);
myData = (MyData) hashMap.get("B");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);
myData = (MyData) hashMap.get("C");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);
myData = (MyData) hashMap.get("D");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);

}
else{
System.out.println("対応するKeyが存在しません。");
}
}catch(Exception e){}
}

/*HashMapにKey and Valueをセット*/
public void setMap(String sFileName) throws Exception{
FileReader oFileReader = new FileReader(sFileName);
BufferedReader oBufferedReader = new BufferedReader(oFileReader);
String sLineData;
String Key;

****ここから変更************************************************
MyData oMyData = null;//new MyData();
while((sLineData = oBufferedReader.readLine()) != null) {
StringTokenizer oStringTokenizer = new StringTokenizer(sLineData, ",");
Key = oStringTokenizer.nextToken();
oMyData = new MyData();
oMyData.sDate = oStringTokenizer.nextToken();
oMyData.sName = oStringTokenizer.nextToken();

hashMap.put(Key,oMyData);
oMyData = null;
***************************************************
}
}
/*HashMap.put(Object,Object)でセット*/

/*セットされたキーがあるか判定*/
public boolean checkKey(String key){

boolean boolKey = true;

boolKey = hashMap.containsKey(key);
/*クラス変数のinpuKeyに判定結果によってセットするかどうか*/
if(boolKey){
inputKey = key;
}
else{
inputKey = null;
}

return boolKey;
}

}

ループの前でクラスをnewしてたんで一つしかインスタンスが生成されてません。したがってその一つしかないインスタンスに上書きで値が更新されているのが問題です。
ですのでループの中でインスタンス生成して、HashMapにputして下さい。

補足

ご返事有り難うございました。
お陰様で解決しました。
okure_donchanさんには、何から何まで本当にお世話になりました。

一つだけ質問したいことがあるんですが、
MyData oMyData = null;//new MyData();
の、「null;//」の意味というのは
オブジェクトを空にするという意味なんですか？

回答No.4

okure_donchan

「null;」と「new MyData(); 」は同じ意味と言うことなんでしょうか？
違います。
型宣言している際にクラス型の変数をメソッド内で宣言する際に初期化というのを明示化するためにnullというのを記述したのみです。

MyData myData; //この段階ではMydata型の変数myDataの中味はnull
myData = new MyData(); //インスタンス生成しmyDataの中にインスタンスを格納。
と
MyData myData = new MyData();

が一緒のなのはわかりますよね。
この上のやつの型宣言の初期化を明示的に行ってるだけです。実際には上記の方法でnullがmyDataに入っています。

でNO.2でやった方法というのはループの外で型宣言だけして、ループの中で一回りごとに新しいインスタンスを生成してそれをHashMapにputするという方法を取っています。
put後にnullを代入するのは初期化の明示化です。
こうしておくことによってイリーガルな値を持つことを防ぎます。
でMapからvalueを取り出す際にkeyに対応したvalue(この場合は作成されたインスタンス)を取り出すのですが。
A = MyData@436463
B = MyData@316303
C = MyData@317421
D = MyData@218230
のように四つのインスタンスを扱っています。

お礼

ご返事有り難うございました。

凄く丁寧に書いて頂きまして有り難うございました。

ただ、未熟な私には理解出来ないところもありますが
でもなんとなく分かりました。

有り難うございました。

回答No.3

okure_donchan

MyData oMyData = null;//new MyData();
//以降はコメントアウトしただけなので。気にせずに。
後この部分では初期化です。宣言しなければ暗黙的に行われますが、メソッド内では行った方が無難です。

補足

何度もすいません。

「null;」と「new MyData(); 」は同じ意味と言うことなんでしょうか？

回答No.1

hilo256

見たところいいような気がします。
hashMap.putに問題はありませんか？
MyData myData = (MyData) hashMap.get("A");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);
myData = (MyData) hashMap.get("B");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);
myData = (MyData) hashMap.get("C");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);
myData = (MyData) hashMap.get("D");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);
とやってみましたか？
hashMap.put()で間違いがあるとは考えられないでしょうか？

補足

ご返事有り難うございました。

早速試してみましたが、４つとも
一番最後のレコードが表示されてしまいます・・・。

以下が全コードなんですが・・・。

大変お忙しいところご面倒かとは思いますが
ご教授して頂けたら幸いです。

－－－－－－－－－－－－－－－

import java.io.*;
import java.util.*;

public class TestMap
{
/*HashMap生成(クラス変数)*/
class MyData{
public String sDate;
public String sName;
}

private static HashMap hashMap = new HashMap();

/*追加：入力キー(クラス変数)*/
public static String inputKey = null;
private String[] keys = {};

public static void main(String args[]){
try{

TestMap testMap = new TestMap();
testMap.setMap("test.csv");

/*Keyに対応する出力の判断*/
if(testMap.checkKey(args[0])){

MyData myData = (MyData) hashMap.get("A");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);
myData = (MyData) hashMap.get("B");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);
myData = (MyData) hashMap.get("C");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);
myData = (MyData) hashMap.get("D");
System.out.println("日付：" + myData.sDate + " 名前：" + myData.sName);

}
else{
System.out.println("対応するKeyが存在しません。");
}
}catch(Exception e){}
}

/*HashMapにKey and Valueをセット*/
public void setMap(String sFileName) throws Exception{
FileReader oFileReader = new FileReader(sFileName);
BufferedReader oBufferedReader = new BufferedReader(oFileReader);
String sLineData;
String Key;
MyData oMyData = new MyData();
while((sLineData = oBufferedReader.readLine()) != null) {
StringTokenizer oStringTokenizer = new StringTokenizer(sLineData, ",");
Key = oStringTokenizer.nextToken();

oMyData.sDate = oStringTokenizer.nextToken();
oMyData.sName = oStringTokenizer.nextToken();

hashMap.put(Key,oMyData);

}
}
/*HashMap.put(Object,Object)でセット*/

/*セットされたキーがあるか判定*/
public boolean checkKey(String key){

boolean boolKey = true;

boolKey = hashMap.containsKey(key);
/*クラス変数のinpuKeyに判定結果によってセットするかどうか*/
if(boolKey){
inputKey = key;
}
else{
inputKey = null;
}

return boolKey;
}

}