if else if 语句没有按我想象的方式工作

Question

I was making a winner checking system where, if I get 2 same numbers from a die than the innerHTML of the result should change from 'result' to 'you win,', if I didn't get same numbers, then the innerHTML of the 'p' should keep showing 'keep trying' but the if else if statement I was working with didn't give me the output I needed and instead of showing 'You win' or 'keep trying' in different cases. it keeps showing 'you win' when clearly my both numbers don't match with each other.

here's the js code:

var dice1 = document.getElementById('you');
var dice2 = document.getElementById('cpu');
var result = document.getElementById('rslt');
var button = document.getElementById('btn');

// Button Interaction

function randomNumbers() {
  document.getElementById('you').innerHTML = Math.floor(Math.random() * 6);
  document.getElementById('cpu').innerHTML = Math.floor(Math.random() * 6);
}

// Result

function checkWin() {
  if (you.innerHTML == cpu.innerHTML) {
    return rslt.innerHTML = "You Win!";

  } else if (you.innerHTML != cpu.innerHTML) {
    return rslt.innerHTML = "Keep on trying!";

  } else {

    return rslt.innerHTML = "Keep on trying!";
  }
}

checkWin();

Please tell me what mistake I did. I'm a novice in programming with javascript, so please dont bully me.

Thank you if you read this far!

Answer 1

Wrap into variables and then try to use if else or as you already did with dice1 and dice2

  var you = document.getElementById('you').innerHTML = Math.floor(Math.random()*6);
  var cpu = document.getElementById('cpu').innerHTML = Math.floor(Math.random()*6);

if(you == cpu)

Answer 2

You should call checkwin() when you button is pressed

Try this code

  var dice1 = document.getElementById('you');

    var dice2 = document.getElementById('cpu');

    var result = document.getElementById('rslt');

    var button = document.getElementById('btn');

    // Button Interaction

    function randomNumbers(){

    document.getElementById('you').innerHTML = Math.floor(Math.random()*6);
    document.getElementById('cpu').innerHTML = Math.floor(Math.random()*6);
    checkWin();
    }

    // Result

    function checkWin(){

    if(you.innerHTML == cpu.innerHTML)

    { return rslt.innerHTML = "You Win!";

    } else if (you.innerHTML != cpu.innerHTML)

    { return rslt.innerHTML = "Keep on trying!";

    } else {

        return rslt.innerHTML = "Keep on trying!";
    }

    }