配列への連番の格納(Java)

Question

Java勉強し始めた初心者です。ご教授ください。

###前提・実現したいこと
配列へ連番の格納
1から100までの値を配列に格納したいとき
int[] array = {1.2....100};
と書くのは非常に面倒に思い試行錯誤していましたが
詰まってしまいましたので質問しました。

###試したこと
for文を使って格納できないか試していたのですが、中々上手くいきませんでした。
無理やり（ここでは1から10までの連番）

public class Test{
	public static void main(String[] args){
		int i;
		int[] array;
		array = new int[10];
		for(i=0;i<9;i++){
			array[i]=i+1;
		}
	}
}	
```と格納することは出来たのですがどうにもスマートさにかけるような気がしてなりません。

他にこのような方法がある、もしくはこうすれば短く書ける、など
よろしければご教授ください。
###補足情報(言語/FW/ツール等のバージョンなど)
java

Answer 1

maisumakunさんの回答は一番スマートだと思います。
しかしながらこれは変数の宣言や初期化、ループの書き方、様々なAPIを使いこなせるという段階になってから書くようにしても遅くないと思います。

最初は以下のようなコードが書ければまずはよいのでは？

public class Test{
  public static void main(String[] args){
    int[] array = new int[10];
    for (int i = 0; i <= array.length; i++) {
      array[i] = i + 1;
    }
  }
}   

1. 初期値を与えないローカル変数宣言は避ける

そのような変数宣言は往々にして「必要になるより以前に宣言」することになり、不必要に広いスコープを持つことになりがちです。本件でいえばiはforループ以外に必要ありませんので上記のように書く習慣にした方がよいでしょう。またarrayの宣言直後に代入するより宣言と同時に初期値を与える方がたいていの場合、短く分かり易いコードになると思います。

1. for文の終端判定

forループの終了判定をi < 9としていますが、これでは最後の要素が0のままです。N回の繰り返しにforループを用いるならfor (i = 0; i < N; i++)と書く人が多いと思います。

1. 配列の要素数で決まる値にリテラルを使わない

本件のループは本質的に(A)「配列の要素数だけ繰り返す」のが目的であって(B)「10回繰り返す」とやってしまうと「繰り返したいのが配列の要素数なのか、先頭の10要素だけなのか」が不明瞭になります。本件の場合は(A)が目的なのですから、i < array.lengthと書く方が合理的です。

このような点に配慮できているコードが「スマート」の第一段階ではないでしょうか？

---

ちなみに目的により「どう書くべきか」は様々に変化します。なぜそう書いたかを自分や他者に納得いくように説明できるようになるとコードは自然にスマートになっていく気がします。

Comments

[退会済みユーザー]

ありがとうございます！
i < array.length
というやり方はなるほどその通りだと思いました…！
i < 9では間違ってましたね…
人間の記述ミスを防いで行くよう気をつけるのもポイントなんですね…

後々の修正も考えるとスマートだと思います…
レベルに合わせていただき助かりました…！

スコープ、初期化に関してまだ理解が浅いので気をつけて勉強したいと思います！
ありがとうございました！

Answer 2

Java8であれば、Stream APIで生成できます。

// 1～100まで
int[] arr = IntStream.rangeClosed(1, 100).toArray();

Comments

[退会済みユーザー]

ありがとうございます！
APIの使用についてはまだ分からない部分も多かったですが、無事思ったとおり格納することができました。

格納する際、配列の宣言と初期化を同時に行いたかったということが上手く言い表せていませんでした。
勉強を続けたいと思います。