Map.putの使い方(java)

Question

初歩的な質問で申し訳ないのですが、下記の2つの書き方は問題ないでしょうか。
一般的な書き方なのか分かりません。

//通常（？）の書き方
Map<String, Object[]> map = new HashMap<String, Object[]>;
String key = "key";
Object[] ary = {"a", 1, 2.0L};
map.put(key, ary);

//省略した書き方
Map<String, Object[]> map = new HashMap<String, Object[]>;
map.put("key", new Object[]{"a", 1, 2.0L});

Answer 1

どちらも問題ないと思います。ただMapに突っ込むだけなら前者の例のようにわざわざ変数に入れる必要はないと思います。

Comments

[swordone]

よく見たら2.0Lは駄目ですね。

[masaya_ohashi]

確かにfloatならfですね

[uer03108]

有難うございました。

Answer 2

Map<String, Object[]> map = new HashMap<String, Object[]>; // ←最後に()が足りません。

重箱の隅をつつくような指摘だけではなんですし、すでにswordoneさんの回答ですでに問題は解決しているので、Map型の初期化時に値を突っ込む方法もあるということを書いておきます。

Map<String, Object[]> map = new HashMap<String, Object[]>(){
  {
   put("key", new Object[]{"a", 1, 2.0L});
  }
};

この書き方は「インスタンス初期化子」といいます。上記パターンでは「HashMapの無名クラス生成時にインスタンス初期化子を使って初期化処理を追加している」、ということになります。
【解説ページ】
http://d.hatena.ne.jp/satosystems/20120905/1346805620

Comments

[uer03108]

初めて知りました。
無名クラスの場合は、コンストラクタが使用できないので（名前からして）イニシャライザを使用するということですね。
https://teratail.com/questions/2935

Answer 3

挙げておられるコードの断片を見たときの意見はswordoneさんが答えておられる通りと思います。ご質問を拝見して感じたことを少しコメントしてみます。

putに限らず一般的にメソッド呼び出しの引数はどんな式でも書けますが、問題は「意図していることをよりよく実現する」ことです。それはそのコードがやろうとしていることやコードを人間が見たときの見易さなど様々な前提に左右されます。こうした前提はコードが現れる場面ごと、プログラマーの好みなどによっても様々に変化するので、その時々でどれがよいか判断するのが大事と思います。

例えばswordoneさんの回答にある「わざわざ変数を使う必要はない」というのは

• 代入する値を表す式が比較的単純
• 代入する値を他の箇所で用いない

といった「前提条件がはっきりとはわからないコードの断片を見たときの印象」からの判断だと思います。これが

Object[] ary = {
  "a", 1, "b", 2, ...,
  ...
  "y", 25, "z", 26,
};
map.put("key", ary);

ならどうだったでしょうか？例えaryが他で参照されることがなくても「変数へ代入してからputへ変数を指定する書き方を好む」という意見も出てくるかも知れませんよ？またmasaya_ohashiさんが挙げておられるような書き方がよいケースもあるでしょう。繰り返しになりますが、それぞれどれが望ましいかは状況によって変わるので一概にこの書き方がベストという単純な答えはないと思います。

以上、申し上げたかったのは「一般的にどうするのがよいかを漠然と問う」よりも、様々なコードを書く経験を通して「書き方の違いについての知見を積む」ことを忘れないでほしいということです。

Comments

[uer03108]

ご回答有難うございましたm(_ _)m
自分が知りたかったのは、「文法的には問題ないけど、実な内部で不具合が起こっている、または、起こりやすい記述」の可能性はあるのかなと言うことでした。
暗黙的に望ましい記述、望ましくない記述はあると思いますが、他の人のコードを見る機会は少ないので。

[KSwordOfHaste]

そういう意図でしたか。
「文法的には問題ないけど、実は内部で不具合が起こっている、または、起こりやすい記述」といったことは「通常はそんなことが起こらないように首尾一貫した考え方で言語が設計されている」と考えるのが普通です。もし質問意図がそういうことであれば質問文に明確にそう書いた方がよいですよ。一般的にどちらがよいか？では質問意図がぼやけてしまいます。

[uer03108]

有難うございました。
確かに「一般的な書き方」では分かりにくいですね。
もう少し詳しく書くように致します。