このコードは、java.util.Random
オブジェクトを作成して1つの乱数を生成するために使用して捨てています。
これはあまり良くない品質の乱数を作り出し、効率が悪いです。
できれば、Random
オブジェクトを1つだけ作成して保存されるようにコードを書き直してください。
そして、新しい乱数が必要なたびに既存の Random
オブジェクトでメソッドを呼び出してください。
生成された乱数が推測可能でないことが重要なら、乱数ごとに新しい Random
オブジェクトを作成してはいけません (値はあまりに簡単に推測可能です)。
その代わりに java.security.SecureRandom
の使用を強く検討するべきです (そして、必要とされる乱数ごとに新しい SecureRandom
のオブジェクトを作成することを回避します)。