Почему не сказано, что я выигрываю, когда я играю в камне, бумаге, ножницах, ящерице, споке?

Question

Почему не сказано, что я выигрываю, когда я играю в камне, бумаге, ножницах, ящерице, споке?

0

Моя игра отлично работает, за исключением случаев, когда я побеждаю, когда участвует ящерица или спок, я не могу понять, почему, потому что, когда я теряю ее, я вижу, что теряю, что правильно. Я пытался искать опечатки, но не мог найти. Если кто-нибудь может понять, что я ошибаюсь, я бы это оценил.

Вот скрипка, чтобы вы могли увидеть проблему: http://jsfiddle.net/5RGwv/

Вот мой HTML:

<div id="center">

<button id="button" onclick="play()">Play</button>
<p>You rolled:</p><span id="myroll"></span>
<p>Your opponent rolled:</p><span id="opproll"></span>

<p id="outcome"></p>
</div>

Javascript:

function play() {

var ROCK = 0;
var PAPER = 1;
var SCISSORS = 2; 
var LIZARD = 3;
var SPOCK = 4;

var choices = ['rock', 'paper', 'scissors', 'lizard', 'spock'];

var myRoll = Math.floor(Math.random()*choices.length);
var opponentRoll = Math.floor(Math.random()*choices.length);

document.getElementById("myroll").innerHTML=choices[myRoll];
document.getElementById("opproll").innerHTML=choices[opponentRoll];

if (myRoll == opponentRoll) {
document.getElementById("outcome").innerHTML="It a draw.";
return;
} //end of if

switch(myRoll) {

case ROCK:
    document.getElementById("outcome").innerHTML=(opponentRoll == SCISSORS ? 'You win!' : 'You lose!');
    return;

case ROCK:
    document.getElementById("outcome").innerHTML=(opponentRoll == LIZARD ? 'You win!' : 'You lose!');
    return;

case PAPER: 
    document.getElementById("outcome").innerHTML=(opponentRoll == ROCK ? 'You     win!' : 'You lose!');
    return;

case PAPER:
    document.getElementById("outcome").innerHTML=(opponentRoll == SPOCK ? 'You win!' : 'You lose!');
    return;

case SCISSORS:
    document.getElementById("outcome").innerHTML=(opponentRoll == PAPER ? 'You win!' : 'You lose!');
    return;

case SCISSORS:
    document.getElementById("outcome").innerHTML=(opponentRoll == LIZARD ? 'You win!' : 'You lose!');
    return;

case LIZARD:
    document.getElementById("outcome").innerHTML=(opponentRoll == SPOCK ? 'You win!' : 'You lose!');
    return;

case LIZARD:
    document.getElementById("outcome").innerHTML=(opponentRoll == PAPER ? 'You win!' : 'You lose!');
    return;

case SPOCK:
    document.getElementById("outcome").innerHTML=(opponentRoll == SCISSORS ? 'You win!' : 'You lose!');
    return;

case SPOCK:
    document.getElementById("outcome").innerHTML=(opponentRoll == ROCK ? 'You win!' : 'You lose!');
    return;

} //end of switch(myRoll)


} //end of play()

Duplo W 18 фев. 2014, в 11:12

Источник

0

выглядит поклонником теории большого взрыва
Standin.Wolf 18 фев. 2014, в 09:39
0

«Я не могу понять, почему, потому что, когда я проигрываю, это показывает, что я проигрываю, и это правильно». Чем в чем конкретно проблема?
PeeHaa 18 фев. 2014, в 09:44
0

Хотел добавить больше элементов, чем камень, ножницы, бумага. Так что это был бы хороший способ сделать это: P
Duplo W 18 фев. 2014, в 09:44
1

При переключении опции появляются несколько раз. JS просто выполнит первое обнаружение и пропустит остальные.
Sirko 18 фев. 2014, в 09:44
0

@PeeHaa проблема в теме, это не показывает, что я выигрываю, когда я делаю.
Duplo W 18 фев. 2014, в 09:45
0

@ Сирко, так как бы это исправить?
Duplo W 18 фев. 2014, в 09:45
2

Комбинировать различные case команды для того же случая в один , а затем использовать , if для различения дальше.
Sirko 18 фев. 2014, в 09:47
0

@ Сирко хорошо, я попробую это
Duplo W 18 фев. 2014, в 09:48
0

@ Сирко, я бы так написал? case ROCK: document.getElementById ("исход"). innerHTML = (оппонент Ролл == НОЖНИЦЫ? "Вы выиграли!": "Вы проиграли!"); if (leaderRoll == LIZARD) {document.getElementById ("исход"). innerHTML = (оппонентRoll == LIZARD? 'Вы выиграли!': 'Вы проиграли!'); } вернуть;
Duplo W 18 фев. 2014, в 09:51
0

@DuploW Посмотрите на данный ответ. На первый взгляд, это почти то, что я имел в виду.
Sirko 18 фев. 2014, в 09:54
0

@ Сирко Я попробовал то, что ты сказал, и мой код работал как шарм. Большое спасибо
Duplo W 18 фев. 2014, в 10:08
0

возможный дубликат Рока, Бумаги, Ножниц, Ящерицы, Спока в JavaScript
Mke Spa Guy 09 сен. 2014, в 15:02

Показать ещё 10 комментариев

Теги:

javascript

html

1 ответ

Ещё вопросы

выглядит поклонником теории большого взрыва
«Я не могу понять, почему, потому что, когда я проигрываю, это показывает, что я проигрываю, и это правильно». Чем в чем конкретно проблема?
Хотел добавить больше элементов, чем камень, ножницы, бумага. Так что это был бы хороший способ сделать это: P
При переключении опции появляются несколько раз. JS просто выполнит первое обнаружение и пропустит остальные.
@PeeHaa проблема в теме, это не показывает, что я выигрываю, когда я делаю.
Комбинировать различные case команды для того же случая в один , а затем использовать , if для различения дальше.
@ Сирко, я бы так написал? case ROCK: document.getElementById ("исход"). innerHTML = (оппонент Ролл == НОЖНИЦЫ? "Вы выиграли!": "Вы проиграли!"); if (leaderRoll == LIZARD) {document.getElementById ("исход"). innerHTML = (оппонентRoll == LIZARD? 'Вы выиграли!': 'Вы проиграли!'); } вернуть;
@DuploW Посмотрите на данный ответ. На первый взгляд, это почти то, что я имел в виду.
@ Сирко Я попробовал то, что ты сказал, и мой код работал как шарм. Большое спасибо
возможный дубликат Рока, Бумаги, Ножниц, Ящерицы, Спока в JavaScript

Cerbrus · Accepted Answer · 2014-02-18T08-21-00.000Z

Замените оператор switch следующим:

var result = false;
switch(myRoll) {
    case ROCK:
        result = opponentRoll == SCISSORS || opponentRoll == LIZARD;
        break;
    case PAPER: 
        result = opponentRoll == ROCK     || opponentRoll == SPOCK;
        break;
    case SCISSORS:
        result = opponentRoll == PAPER    || opponentRoll == LIZARD;
        break;
    case LIZARD:
        result = opponentRoll == SPOCK    || opponentRoll == PAPER;
        break;
    case SPOCK:
        result = opponentRoll == SCISSORS || opponentRoll == ROCK;
        break;
}

document.getElementById("outcome").innerHTML = result ?  'You win!' : 'You lose!';

Этот кусок кода использует switch только для проверки, выиграли вы или нет. Затем в конце устанавливается значение.

Поскольку вы побеждаете, если вы бросаете "рок", а противник "lizard" ножницы противников, я объединил эти заявления.

Вы можете сократить код еще больше, просто проверив для случаев win, например:

var result =  myRoll == ROCK     && (opponentRoll == SCISSORS || opponentRoll == LIZARD) ||
              myRoll == PAPER    && (opponentRoll == ROCK     || opponentRoll == SPOCK)  ||
              myRoll == SCISSORS && (opponentRoll == PAPER    || opponentRoll == LIZARD) ||
              myRoll == LIZARD   && (opponentRoll == PAPER    || opponentRoll == SPOCK)  ||
              myRoll == SPOCK    && (opponentRoll == ROCK     || opponentRoll == SCISSORS);

document.getElementById("outcome").innerHTML = result ?  'You win!' : 'You lose!';

Если какая-либо из этих 5 строк true, вы выиграли.

Замените return на break или вы никогда не достигнете конца оператора
Я пошел с заявлением, но я уверен, что это также сработает, поэтому приму его в качестве ответа.
@DuploW: спасибо! Как обычно, есть несколько способов кодирования чего-либо, просто используйте то, что вам удобнее :)
Проще было добавить простое выражение if, чем менять весь Case: P