javaのMapの記述の仕方について

Question

いつもお世話になっております。
Javaの記述の仕方で分からない事があります。
ご存知の方がいらっしゃれば教えて頂けますでしょうか。

【ご質問】
以下のプログラムを記述すると
☆1で警告が表示されます。
警告を解消する為には、どのように修正すればよいのでしょうか。

JAVAのバージョンは1.8です。

public Map<Object,Object> getTantousya(){
	String cacheKey = "tantousya:N";
	Map<Object,Object> result = (Map)masterMap.get(cacheKey); ☆1
	if(result == null){
		result = key_value_service.getTantousya();
        masterMap.put(cacheKey, result);
	}
	return result;
}

(警告の内容)
☆1
型の安全性: 型 Map の式は、未検査の型変換を使用して Map<Object,Object> に準拠するようにする必要があります

(補足)
1.masterMapは、以下の様に宣言しております。

private final HashMap<String,Object> masterMap = new HashMap<>();

2.key_value_service.getTantousya()、以下の様に宣言しております。

public Map<Object,Object> getTantousya()
```　
3.☆1の箇所を
```ここに言語を入力
　Map<Object,Object> result = (Map<Object,Object>)masterMap.get(cacheKey);
```　
に、変更すると、以下の警告が表示されます。

この行に複数マーカーがあります
- Map は raw 型です。総称型 Map<K,V> への参照は、パラメーター化する必要があります
- 型の安全性: Object から Map<Object,Object> への未検査キャスト

以上です。
よろしくお願いいたします。

(追記)
Map<Object, Object>には、
以下のどちらかの戻り値を設定いたします。

```ここに言語を入力
public Map<Object,Object> getQuerykeyValue(Integer renban){
  List<KeyValue> list = keyValueMapper.querykeyValue(renban); 
  Map<Object,Object> result = new HashMap<>();
  for(int i=0;i<list.size();i++){
    KeyValue keyvalue = (KeyValue)list.get(i);
	result.put(keyvalue.getKey(), keyvalue.getValue());
  }
  return result;
}

public Map<Object,Object> getQuerykeyValueInteger(Integer renban){
  List<KeyValue> list = keyValueMapper.querykeyValueInteger(renban);
  Map<Object,Object> result = new HashMap<>();
  for(int i=0;i<list.size();i++){
    KeyValue keyvalue = (KeyValue)list.get(i);
    result.put(keyvalue.getKey(), keyvalue.getValue());
  }
  return result;
}

【KeyValue】
public class KeyValue implements Serializable {
  static final long serialVersionUID = 1L;
  private Object key;
  private Object value;
  getter/setter
}

【querykeyValueで実施するSQL】
<select id="querykeyValue" resultType="KeyValue">
  SELECT key AS "key",
         value AS "value"
    FROM key_value_tbl
   WHERE renban = #{renban}
   ORDER BY sort_no
</select>


【querykeyValueIntegerで実施するSQL】
<resultMap id="ResultMapInteger" type="KeyValue">
  <result property="key" column="key" javaType="Integer" />
  <result property="value" column="value" javaType="String" />
</resultMap>
<select id="querykeyValueInteger" resultMap="ResultMapInteger">
  SELECT key AS "key",
         value AS "value"
    FROM key_value_tbl
   WHERE renban = #{renban}
   ORDER BY sort_no
</select>

【key_value_tblテーブルの型】
key   VARCHAR
value VARCHAR

Comments

[swordone]

masterMapの値にはMap<Object, Object>型以外のモノが入っていますか?(masterMapの型をMap<String, Map<Object, Object>>として問題がないか？)

[a0841_1974]

ご連絡ありがとうございます。masterMapの値にはMap<Object, Object>型以外のものは入りません。

Answer 1

こういう警告ですね?(コンパイルオプション-Xlint:uncheckedを指定してます)

T.java:6: 警告:[unchecked] 無検査キャスト
    Map<Object,Object> result = (Map<Object, Object>)masterMap.get(cacheKey); //☆1
                                                                  ^
  期待値: Map<Object,Object>
  検出値:    Object

これはObjectをMap<Object, Object>へダウンキャストしているためですが、コンパイラーはこのダウンキャストが安全かどうか判断できません。"tantousya:N"というキーの値が必ずMap<Object,Object>になっているはずというのは(キャストが問題ないというのは)論理全体を見渡せる人間のみが判断できることです。

この警告を消すには「論理的にこのダウンキャストが問題ないと判断してるので警告は不要ですよ」とコンパイラーに伝える必要があり、それは@SuppressWarningsアノテーションを用いるとできます。

public Map<Object,Object> getTantousya(){
  String cacheKey = "tantousya:N";
  @SuppressWarnings("unchecked") // これ
  Map<Object,Object> result = (Map<Object, Object>)masterMap.get(cacheKey); //☆1
  if (result == null) {
    result = key_value_service.getTantousya();
    masterMap.put(cacheKey, result);
  }
  return result;
}

追記：ちなみに型引数の指定なしでMap型の値をMap<Object,Object>型の変数へ代入するのは警告の対象になりますが、それは型引数も含めてきっちりキャストする必要があります。上のコードではその点も修正しています。

参考：
https://docs.oracle.com/javase/jp/8/docs/api/java/lang/SuppressWarnings.html

Comments

[KSwordOfHaste]

ちょっと補足します。これはmasterMapの値はMap<Object,Object>以外に色んな型の値が格納されるという前提での回答です。
全ての値がMap<Object,Object>ならmasterMapの型はMap<String,Map<Object,Object>>とすべきですが、そうはできないんですよね？

[a0841_1974]

今回は全ての値がMap<Object,Object>ですので、
masterMapの型を
private final HashMap<String,Map<Object,Object>> masterMap = new HashMap<>();
としたら解決しました。
ありがとうございました。

[KSwordOfHaste]

おっと意外な結末です・・・
そういうことならもちろんSuppressWarningsアノテーションは付けないのが正解になりますね。
なお、型引数としてMap<Object, Object>が妥当なのかも若干気になります。
少なくともMapのキーの型をObjectにするというのは普通は避けると思うのです。キーや値の型は本当は何か特定の型ではないのですか？

[a0841_1974]

ご回答ありがとうございます。

基本的な事が分かっていないので、詳細に記述させて頂きます。
※詳細は、質問の(追記)をご確認して頂けますでしょうか。

頂きましたご質問ですと、Keyには、Integer or Stringを設定しますが、
この状態でmasterMapの型を
HashMap<String,Map<Object,Object>>
とするのは、よろしくないでしょうか。

[KSwordOfHaste]

querykeyValueとquerykeyValueIntegerを見る限りKeyはString型とすべきように見えます。KeyValueというクラスでなぜkeyがObject型になっているかわかりませんが・・・ひょっとしてquerykeyValueとquerykeyValueInteger以外にも用いる汎用のクラスなんでしょうか？

プログラムの設計がよく見えませんが、例えば自分なら
class KeyValue<KEYTYPE, VALUETYPE> {
KEYTYPE key;
VALUETYPE value;
}
のようにジェネリクスにするかも知れません。繰り返しになりますが、そうすべきかどうかはプログラムの設計次第なところがあります・・・

[a0841_1974]

ご回答ありがとうございます。
KeyValueは汎用クラスの目的で作成しております。
今のところKeyにはInteger,String,Longが来ることを想定しております。

[KSwordOfHaste]

04/05 18:51にコメントしましたように、KeyValueクラスの定義のしかたによってキーをObject型にせずにInteger,String,Longそれぞれとして扱うこともできますが・・・
本質問の主題からはなれてしまいますし、コメント欄で話せるほど単純な話題でもないのでキーワードだけコメントしておきますと「generics, generic method」あたりになると思います。機会があればその辺りを掘り下げてみるとよいかと思います。

[a0841_1974]

ご回答ありがとうございます。
ジェネリクスについて詳しく分かっていないので、調べてみます。
ありがとうございます。

Answer 2

多少割愛しましたが、警告はなくなりました。

	public Map<Object, Object> getTantousya() {
		String cacheKey = "tantousya:N";
		Map<Object, Object> masterMap = new HashMap<>();
		Map<?, ?> result = (Map<?, ?>) masterMap.get(cacheKey);
		return new HashMap<>(result);
	}

ご参考までに
List<Object>とList<?>の違い

Comments

[a0841_1974]

ご連絡ありがとうございます。
ご連絡頂きました方法で出来ましたが、以下の場合はどのように記述したらよいのでしょうか。
public Map<Object,Object> getTantousyaWithNULL(){
// キャッシュキー名を取得するデータ毎に設定する。
String cacheKey = "tantousya:NULL";
Map result = (Map)masterMap.get(cacheKey);
if(result == null){
result = new HashMap();
result.put((Object)new Integer(-1), (Object)"(指定なし)");
result.putAll(key_value_service.getTantousya());
masterMap.put(cacheKey, result);
}
return result;
}

Answer 3

こうやな・ｗ・

public static void main(String[] args) {
	Map<Object, Object> omap = new HashMap<>();
	omap.put("a", "9");
	Object map = omap;
	Map<? extends Object, ? extends Object> xmap = (Map<?, ?>) map;
	System.out.println(xmap.get("a"));
}

map型じゃないケースを想定するとこうなる「

public static void main(String[] args) {
	Map<Object, Object> omap = new HashMap<>();
	omap.put("a", "9");

	// Map<Object,Object>#get を課程
	Object map = omap;

	// 以下 説明ソース
	Map<? extends Object, ? extends Object> xmap;
	if (map instanceof Map) {
		// 設定時
		xmap = (Map<?, ?>) map;
	} else {
		// 未設定時（任意の別のマップを設定するなりしてください）
		xmap = omap;
	}
	System.out.println(xmap);

}

Comments

[a0841_1974]

ご連絡ありがとうございます。
ご連絡頂きました方法で出来ましたが、以下の場合はどのように記述したらよいのでしょうか。
public Map<Object,Object> getTantousyaWithNULL(){
// キャッシュキー名を取得するデータ毎に設定する。
String cacheKey = "tantousya:NULL";
Map result = (Map)masterMap.get(cacheKey);
if(result == null){
result = new HashMap();
result.put((Object)new Integer(-1), (Object)"(指定なし)");
result.putAll(key_value_service.getTantousya());
masterMap.put(cacheKey, result);
}
return result;
}