ユーザーが選択するランダムなmaxを設定する方法

Question

私は、1つのアクティビティでランダムな評価の最大値を選択し、別のアクティビティで使用できるようにする簡単なコードを作成しようとしています。 （最小は1です）

私はアクティビティAからアクティビティBへのint変数を呼び出すことに成功しましたが、ランダムに使用するとアプリケーションがクラッシュしました。

ここで私はint型のために呼び出します。

Intent intent = getIntent(); 
highNum = intent.getIntExtra("highNum", 0); 

とランダムに置く：

Random random = new Random(); 
mainNum = random.nextInt(highNum) + 1; 

私はすでにintは本当にdefualtよりも高い数値に等しいことを確認 - 0

スタックフローの重要な部分が私の厚さです：

0私は mainNumを設定する場所

Caused by: java.lang.IllegalArgumentException: n <= 0: 0 
at java.util.Random.nextInt(Random.java:182) 
at com.appstudio.shirdn.sevenboom.MainActivity.randomNumber(MainActivity.java:148) 
at com.appstudio.shirdn.sevenboom.MainActivity.onCreate(MainActivity.java:40) 

ライン148であり、ライン40は、私は、ランダムなクラスを求めるonCreate クラスです。

ありがとうございました！

Answer 1

スタックトレースは、AndroidプラットフォームのAPIレベル23からのコードの実際の線であり、あなたがRandom.nextInt()に渡される数は、実際にここでは0であることを語っている：

public int nextInt(int n) { 
    if (n <= 0) { 
     throw new IllegalArgumentException("n <= 0: " + n); 
    } 
    ... 
} 

は、あなたがメッセージであることがわかりますnの値を例外エラーメッセージに挿入します。これは、あなたのhighNumの値に相当します。

このメソッドに渡される直前のhighNumの値を確認してください。ここ

：

mainNum = random.nextInt(highNum) + 1; 

あることを起こるあなたは乱数を生成しようとしているが、あなたは、パラメータ**として渡さバインドを検証していないので

Answer 2

この例外はあります変数highNumは正である必要があります

ソリューション：

nextIntメソッドに値を渡す前に、変数highNumを検証：あなたがそれを必要とする場合は、公式Documentationに見てみ

if (highNum > 0) { 
    mainNum = random.nextInt(highNum) + 1; 
} 

：

は、のような単純な何かを

パラメータ：上限（上限）を拘束します。正でなければなりません。