Javascript getElementById действует нечетно

Question

Я использую цикл for для циклического перехода через некоторые элементы со стартовым значением (места на плоскости).

Здесь:
seatNum - Количество мест, подлежащих циклическое через
startSeat - сиденье, чтобы начать езду на велосипеде

Я вызываю функцию из формы «onsubmit».

Проблема возникает в цикле for, когда я пытаюсь получить элементы с соглашением о присвоении имен «s1» «s2» «s3» и т. Д. «S45» «s46» и т. Д. ... на основе счетчик циклов, добавленный к стартовому месту. Подсчет от 0 (начальное место) до местаNum (сколько мест).

любая идея, почему по id не разрешается правильно? Все остальные отлично работают, за исключением последнего внутри цикла for.

Да, я новичок в программировании, поэтому у меня, вероятно, нет лучших практик, пожалуйста, прощайте стилистически.

function check() { 
    var startSeat; 
    var fName = document.getElementById('fName').value 
    var lName = document.getElementById('lName').value 
    var address = document.getElementById('address').value 
    var city = document.getElementById('city').value 
    var state = document.getElementById('state').value 
    var zip = document.getElementById('zip').value 
    var phone = document.getElementById('phone').value 
    var seatNum = document.getElementById('seatNumber').value 
    var y=document.getElementById('seatList1').value; 
    var z=document.getElementById('seatList2').value; 

    if (z >= y) { 
     startSeat = y; 
    } 
    else { 
     startSeat = z; 
    } 

    if ((fName == "") || (lName == "") || (address == "") || (phone == "") || (zip == "") || (state == "") || (city == "")) { 
     alert("You must fully complete the form"); 
     return false; 
    } 

    for (var i = 0; i < seatNum; i++) { 
     if (document.getElementById("s"+(startSeat+i)).className=="taken"){ 
      alert("Selected seat(s) already booked."); 
      return false; 
     } 
    else { 
      continue; 
     } 
    } 
} 

Answer 1

Преобразования y и z переменных номер:

var y = +document.getElementById('seatList1').value; 
var z = +document.getElementById('seatList2').value; 

var startSeat = (z >= y) ? y : z; // or simply startSeat = Math.min(z,y); 

Это решит проблему, что @Faruz указал.

Answer 2

Попробуйте это:

for (var i = startSeat; i < seatNum; i++) { 
    if (document.getElementById("s"+i).className == "taken") { 
     alert("Selected seat(s) already booked."); 
     return false; 
    } 
} 

Вместо добавления i к значению startSeat, чтобы получить идентификатор места, используйте startSeat's значение прямо в инициализации цикла. Я считаю, что произошло то, что вы получали off-by-one error, так как ваше значение startSeat было уже установлено, а затем вы добавляли i к нему, который натолкнул вас вперед на один.

Answer 3

Я не уверен, но, возможно, startSeat + i конкатцирует строки и не выполняет математическое ожидание. Попробуйте оповещения на экране:

alert(document.getElementById("s"+(startSeat+i))); 

Является ли это имя поля?

Answer 4

Возможно, вы повторяете слишком много раз. Попробуйте i < seatNum - 1 в вашем цикл

Answer 5

Когда вы говорите, проезжая через сиденья, я предполагаю, что вы хотите продолжать отсчет с 1 после достижения максимального места?Предполагая, что numberOfSeats определяется где-то (я не мог видеть его, но вы должны иметь его где-то), вы можете сделать это:

"s"+((startSeat + i - 1) % numberOfSeats + 1) 

поэтому линия в полном объеме:

if (document.getElementById("s"+((startSeat + i - 1) % numberOfSeats + 1)).className=="taken"){ 

Answer 6

Что вы действительно получают конкатенации, так что на самом деле происходят так:

i = 10 
seatNum = 1 

(seatNum+i) = "110" 

Попробуйте это с помощью функции parseInt() для литья переменных целочисленных типов:

if (document.getElementById("s"+(parseInt(startSeat)+parseInt(i))).className=="taken") 

Answer 7

Похоже, что startSeat - это строковый тип. Несмотря на то, что JavaScript не имеет значения типа, значение объекта DOM будет использоваться как строка. Итак, вы получаете конкатенацию вместо добавления.

Используйте то, что написал CMS. Это должно решить вашу проблему.