このコードは、java.util.Random オブジェクトを作成して1つの乱数を生成するために使用して捨てています。 これはあまり良くない品質の乱数を作り出し、効率が悪いです。 できれば、Random オブジェクトを1つだけ作成して保存されるようにコードを書き直してください。 そして、新しい乱数が必要なたびに既存の Random オブジェクトでメソッドを呼び出してください。

生成された乱数が推測可能でないことが重要なら、乱数ごとに新しい Random オブジェクトを作成してはいけません (値はあまりに簡単に推測可能です)。 その代わりに java.security.SecureRandom の使用を強く検討するべきです (そして、必要とされる乱数ごとに新しい SecureRandom のオブジェクトを作成することを回避します)。