JS function 'not defined' after adding for loop

Question

I'm trying to create a calculator that takes a number as input, decides in which range this number is, and adds a corresponding price to the basic price (currently set to 0). However, I'm stuck at a rather simple for-loop I inserted to iterate over the ranges (in seizprijs). Without the for-loop, the code appears to do just fine. However, if I run the code including the for-loop, the function returns 'not defined'.

I'm completely new to coding, so I expect it to be a simple issue. However, I tried every suggestion I could find online, and nothing seems to work.

const seizprijs = [
{tijd : "winter", bdatum : 1, edatum : 5, prijs : 45},
{tijd: "lente", bdatum: 5, edatum: 9, prijs: 50}
];

function myFunction() {
  var prijs = 0;

  var vdat = document.getElementById("datum");
  var vdatum = vdat.value;

  for (var x = 0, x < seizprijs.length, x++){

    if (vdatum > seizprijs[1].bdatum && vdatum < seizprijs[1].edatum)
    {
      prijs += seizprijs[1].prijs;
    }
  }

 document.getElementById("demo").innerHTML = "Totaalprijs:" + prijs;

}

<h1>Boeking</h1>
<p id="demo">Totaalprijs: </p>
<form id="boeking">
  <input type="number" name="datum" id="datum">
</form>
<button type="button" onclick="myFunction()">Prijs berekenen</button>

w3schools.com/js/js_loop_for.asp for (var x = 0; x < seizprijs.length; x++) — AlexOwl
– AlexOwl, Commented Mar 14, 2019 at 11:02

Shadab Ahmed · Accepted Answer · 2019-03-14 11:11:38Z

1

you are using , instead of ; in your for loop

try like this

for (var x = 0; x < seizprijs.length; x++)

answered Mar 14, 2019 at 11:11

Shadab Ahmed

57610 silver badges20 bronze badges

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

1 Comment

AmLam Over a year ago

Thanks, that was it! I changed that around before, but apparently made another mistake at first, and didn't change it back. It works now! :)

brk · Accepted Answer · 2019-03-14 11:06:57Z

The problem seems to be the data type of the value from the input. Use + unary operator to convert the string to number. Also if you are iterating a loop use the index , x in your code to access the value from the array of object

const seizprijs = [{
    tijd: "winter",
    bdatum: 1,
    edatum: 5,
    prijs: 45
  },
  {
    tijd: "lente",
    bdatum: 5,
    edatum: 9,
    prijs: 50
  }
];

function myFunction() {
  var prijs = 0;

  var vdat = document.getElementById("datum");
  var vdatum = +vdat.value;

  for (var x = 0; x < seizprijs.length; x++) {

    if (vdatum > seizprijs[x].bdatum && vdatum < seizprijs[x].edatum) {
      prijs += seizprijs[x].prijs;
    }
  }

  document.getElementById("demo").innerHTML = "Totaalprijs:" + prijs;

}

<h1>Boeking</h1>
<p id="demo">Totaalprijs: </p>
<form id="boeking">
  <input type="number" name="datum" id="datum">
</form>
<button type="button" onclick="myFunction()">Prijs berekenen</button>

Collectives™ on Stack Overflow

JS function 'not defined' after adding for loop

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related