0

My code was working fine till I added the if statement. Any help is much appreciated.

function calc() {
   var a = parseInt(document.getElementById('eersteGetal').value); 
    var b = parseInt(document.getElementById('tweedeGetal').value); 
    var answer = parseInt(a + b);
    var quotient = a % b;
    var bigger = '';                                                
    if(a >= b) {                                
        bigger = "Number A is bigger";              
    } else if(a <= b) {                             
        bigger = "Number B is bigger";              
    } else(a === b) {                           
        bigger = "Number A en B are equal";     
    }
    var paragraaf = document.getElementById('answer'); 
    paragraaf.innerHTML = answer;
    var paragraafTwo = document.getElementById('quotient');  
    paragraafTwo.innerHTML = quotient;
    var paragraafThree = document.getElementById('bigger'); 
    paragraafThree.innerHTML = bigger;
}
4
  • what errors are you getting? Commented Jun 22, 2016 at 12:05
  • 3
    Remove condition from else part Commented Jun 22, 2016 at 12:05
  • 1
    if you want third condition then remove = from >= and <= Commented Jun 22, 2016 at 12:06
  • else is only exicuted if the other if/else if condition(s) do not return true so no condition is required for else. Commented Jun 22, 2016 at 12:07

3 Answers 3

1

Remove condition from else part.

function calc() {
  var a = parseInt(document.getElementById('eersteGetal').value); 
  var b = parseInt(document.getElementById('tweedeGetal').value); 
  var answer = parseInt(a + b);
  var quotient = a % b;
  var bigger = '';                                                
  if(a > b) {                               
    bigger = "Number A is bigger";              
  } else if(a < b) {                          
    bigger = "Number B is bigger";              
  } else {
    bigger = "Number A en B are equal";     
  }
  var paragraaf = document.getElementById('answer'); 
  paragraaf.innerHTML = answer;
  var paragraafTwo = document.getElementById('quotient');  
  paragraafTwo.innerHTML = quotient;
  var paragraafThree = document.getElementById('bigger'); 
  paragraafThree.innerHTML = bigger;
}
<input type="text" id="eersteGetal">
<input type="text" id="tweedeGetal">
<button type="button" onclick="calc()">Calculate</button>
<p id="answer"></p>
<p id="quotient"></p>
<p id="bigger"></p>

Sign up to request clarification or add additional context in comments.

1 Comment

Thank you so much, removing the condition did the trick. Very greatful.
0

You cant put a condition in an ELSE.

Change your IF/ELSE-STATEMENT to this:

if(a > b ) {                               
    bigger = "Number A is bigger";              
}
else if(a < b) {                          
    bigger = "Number B is bigger";              
}
else {
    bigger = "Number A en B are equal";     
}

3 Comments

last condition never reach
@FastSnail - Thanks for noticing.
remove = from first 2 conditions and add else
0

You Missed "if" in last else

else(a === b) {                            
bigger = "Number A en B are equal";     
}

it should be

else if(a === b) {                            
bigger = "Number A en B are equal";     
}

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.