java - or オペランドの int を if ステートメントで使用する

Question

私の問題は、「彼」が意図したとおりにプログラムがコードを読み取っていないことです。

私は持っている

if (hero.getPos() == (6 | 11 | 16)) {
    move = new Object[] {"Up", "Right", "Left"};
} else {
    move = new Object[] {"Up", "Down", "Right", "Left"};
}

ヒーローの位置が 6 の場合でも、プログラムは別の場所に移動します。

何故ですか？オペランドのせいですか？はいの場合、どのように変更すればよいですか？

score 11 · Accepted Answer

使用する：

if (hero.getPos() == 6 || hero.getPos() == 11 || hero.getPos() == 16)) {

これはあなたが望むことをします。

あなたがしたことは、ビットごとに、またはそれらの数値の間で実行されるhero.getPos()結果と比較することです。(6|11|16)

score 4 · Accepted Answer

他の答えは正しいです、あなたがセットを使うかもしれない別の考えをするだけです。

static final Set<Integer> positions = new HashSet<Integer>();
static{
    positions.add(6);
    positions.add(11);
    positions.add(16);
}

if (positions.contains(hero.getPos())){
    move = new Object[] {"Up", "Right", "Left"};
} else {
    move = new Object[] {"Up", "Down", "Right", "Left"};
}

score 2 · Accepted Answer

あなたはそのようにそれをすることはできません。3つの数値をビット単位でORします。

あなたはこのようにしなければなりません：

if (hero.getPos() == 6 || hero.getPos() == 11 | hero.getPos() == 16)) {
    move = new Object[] {"Up", "Right", "Left"};
} else {
    move = new Object[] {"Up", "Down", "Right", "Left"};
}

違いがわかりますか？|はビット単位またはwhile||は論理またはです。また、毎回比較を書き直す必要があることにも注意してください。

score 2 · Accepted Answer

(6 | 11 | 16)は、最初に 31 (二項演算) に評価されます。これは6 != 31です。あなたが望むものではありません。

すべての単一の位置をチェックすることをお勧めします (3 つしかないため、インラインが適切です。さらにループの使用を検討してください)。

if (hero.getPos() == 6 || hero.getPos() == 11 | hero.getPos() == 16)) {
    move = new Object[] {"Up", "Right", "Left"};
} else {
    move = new Object[] {"Up", "Down", "Right", "Left"};
}

score 1 · Accepted Answer

いいえ、ci.getNumber() == ...各値を確認するか、コレクションに追加して確認する必要がありますmyCollection.contains(ci.getNumber())。ただし、いくつかの既知の値に対してメソッドをチェックする場合は、コードの構造を再考する必要があります。

score 1 · Accepted Answer

からの回答を使用：

配列に特定の値が含まれているかどうかをテストするにはどうすればよいですか?

数値の配列を作成し、ci.getNumber() がその中にあるかどうかを確認できます。

score 1 · Accepted Answer

そのようなオペレーターはいません。ただし、数値を比較している場合は、スイッチを使用してそれをシミュレートできます。方法は次のとおりです。

int aNumber = ci.getNumber();
swithc(aNumber) {
    case 6252001:
    case 5855797:
    case 6251999: {
        ...
        break;
    }
    default: {
        ... // Do else.
    }
}

お役に立てれば。

score 1 · Accepted Answer

boolean theyAretheSame = num1 == num2 ? (num1 == num3 ? true:false):false;

私はこれをチェックしていないことを認めなければなりませんが、ロジックは正しいようです。

score 1 · Accepted Answer

いいえ。Set<Integer>一度作成してからそれを使用するか、次のようにすることができます。

int number = ci.getNumber();
if (number == 6252001 || number == 5855797 || number == 6251999)

これらの数値を定数に変更して、より意味のあるコードを取得することも検討します。

score 0 · Accepted Answer

すべての数値をコレクションに入れてから、contains()メソッドを使用できます。それ以外には、あなたがやりたいように比較するための特別な構文はないと思います。

score 0 · Accepted Answer

Java ではそれができません。ハッシュルックアップ（これはやり過ぎです）またはcaseステートメント、または大きなクラクションの醜い複数比較を実行できます。

if ((n==1 ) || (n==2) || ...

score 0 · Accepted Answer

0

いいえ、個別に比較する必要があります。

于 2010-01-04T19:55:49.940 に答える

java - or オペランドの int を if ステートメントで使用する

12 に答える 12

Related

Reference