CODEWITHSUNDEEP
javascripthtmlformsif-statementverification
mastaSabin
mastaSabin

Reputation: 141

JavaScript if and else if statement not working properly

I have an input type='number' and a button to indicate function testfun() and an if statement in the function that is supposed to validate that the number is within range and provide clear instructions.

when I enter a number within the range of 1-1000 it works fine.

when I enter a number that is <1 or >1000 the <div><p> does not populate.

HTML:

<div class="preT" id="preT">
  <form>
    <label for="voipLines">VoIP Lines</label>
    <input type="number" id="voipLines" name="voipLines" requiered min="1" max="1000" value="1">
  </form>
  <div class="voipbutton" id="voipbutton">
    <button class="button" onclick="testfun()">Test Clicker</button>
  </div>
</div>
<div id="duringT">
  <p Id="clickerTest"></p>
  <div Id="prBar" data-label=""></div>
  <div id="canvascon"></div>
</div>

JS:

function testfun() { //used in HTML
  var voip = document.getElementById("voipLines").value;
  if (voip < 1) { //voiplines verification 
    return document.getElementById("clickerTest").innerHTML = "<div><p>Make Sure VoIP Lines is within the range of 1 to 1000</p></div>";
  } else if (voip > 1000) {
    return document.getElementById("clickerTest").innerHTML = "<div><p>Make Sure VoIP Lines is within the range of 1 to 1000</p></div>";
  } else {
    var voipGuide = document.getElementById("clickerTest").innerHTML = '<h2>Clicker Successful!!!</h2>\
    <p>Please click the link below to start <b>' + voip + ' test</b></p>\
    <p>To change the amount of tests to run <a href= "">Click Here</a></p>';
  document.getElementById('preT').style.display = 'none';
    document.getElementById("canvascon").innerHTML = '<div id="canvas-container"><canvas id="canvs3" style="background-color:#F0F0F0; ">Your browser does not support the HTML5 canvas tag.</canvas><br><br></div>';
    voipsims();
    bootfun();
  }
}

Upvotes: 0

Views: 117

Answers (4)

Lewis
Lewis

Reputation: 4595

There was a typo: the property is .innerHTML, not .innerhtml. Working example:

function testfun() { //used in HTML
  var voip = document.getElementById("voipLines").value,
      clickerTest = document.getElementById("clickerTest");
        
  if (voip < 1) //voiplines verification 
    return clickerTest.innerHTML = "<div><p>Make Sure VoIP Lines is within the range of 1 to 1000</p></div>";
  
  else if (voip > 1000)
    return clickerTest.innerHTML = "<div><p>Make Sure VoIP Lines is within the range of 1 to 1000</p></div>"; 
  
  else {
    
    var voipGuide = clickerTest.innerHTML = '<h2>Clicker Successful!!!</h2>\
    <p>Please click the link below to start <b>' + voip + ' test</b></p>\
    <p>To change the amount of tests to run <a href= '
    '>Click Here</a></p>';
    document.getElementById('preT').style.display = 'none';
    document.getElementById("canvascon").innerHTML = '<div id="canvas-container"><canvas id="canvs3" style="background-color:#F0F0F0; ">Your browser does not support the HTML5 canvas tag.</canvas><br><br></div>';
    
  }
}
<div class="preT" id="preT">
  <form>
    <label for="voipLines">VoIP Lines</label>
    <input type="number" id="voipLines" name="voipLines" min="1" max="1000" value="1" required>
  </form>
  <div class="voipbutton" id="voipbutton">
    <button class="button" onclick="testfun()">Test Clicker</button>
  </div>
</div>
<div id="duringT">
  <p id="clickerTest"></p>
  <div id="prBar" data-label=""></div>
  <div id="canvascon"></div>
</div>

Upvotes: 1

Riza Khan
Riza Khan

Reputation: 3158

Here is a working version in Codepen. https://codepen.io/riza-khan/pen/YzwwRLQ?editors=1011

There were some bad practices in your code. Namely, it is better to use eventListeners then to call a function via a onclick in the HTML tags.

Secondly, there were some spelling mistakes, which are corrected in the codepen version.

With the corrections, the IF/ELSEIF block wouldn't even be required but I left it there anyway. 

<div class="preT" id="preT">
    <form>
        <label for="voipLines">VoIP Lines</label>
        <input type="number" id="voipLines" name="voipLines" required min="1" max="1000" value="1">
    </form>
    <div class="voipbutton" id="voipbutton">
        <button class="button">Test Clicker</button>
    </div>
</div>
<div id="duringT">
    <p Id="clickerTest"></p>
</div>

document.querySelector(".button").addEventListener("click", () => {
    let voip = document.querySelector("#voipLines").value;
    if (voip < 1) {
        document.querySelector("#clickerTest").innerHTML =
            "<div><p>Make Sure VoIP Lines is within the range of 1 to 1000</p></div>";
    } else if (voip > 1000) {
        document.querySelector("#clickerTest").innerHTML =
            "<div><p>Make Sure VoIP Lines is within the range of 1 to 1000</p></div>";
    } else {
        var voipGuide = (document.getElementById("clickerTest").innerHTML =
            "<h2>Clicker Successful!!!</h2>\
    <p>Please click the link below to start <b>" +
            voip +
            ' test</b></p>\
    <p>To change the amount of tests to run <a href= "">Click Here</a></p>');
        document.getElementById("preT").style.display = "none";
        document.getElementById("canvascon").innerHTML =
            '<div id="canvas-container"><canvas id="canvs3" style="background-color:#F0F0F0; ">Your browser does not support the HTML5 canvas tag.</canvas><br><br></div>';
        voipsims();
        bootfun();
    }
});

Upvotes: 2

ry kim
ry kim

Reputation: 103

I am not sure of this answer, but I think innerhtml should be innerHTML and there shouldn't be spaces between 'if', 'else if', 'else' and "{" and "}". So it should be "

if (voip<1){

...

}else if (voip>1000){

...

}else{

...

} "

There is also a \ after the paragraph ending after the Please click the link... which I think is incorrect.

Upvotes: 0

siralexsir88
siralexsir88

Reputation: 418

In the else body, you use innerHTML, but in the if and else if you use innerhtml. It should be innerHTML

Upvotes: 2

Related Questions