Javascript getElementById 行为奇怪

Question

我使用 for 循环来循环访问一些具有起始值的元素（飞机上的座位）。

这是：
SeatNum - 要循环的座位数
startSeat - 开始骑行的座位

我从“onsubmit”表单中调用该函数。

当我尝试获取具有“s1”“s2”“s3”等 id 命名约定的元素...“s45”“s46”等...基于添加到的循环计数器时，问题出现在 for 循环中起始座位。从 0（起始座位）数到 SeatNum（多少个座位）。

知道为什么 by 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

我不确定，但也许 startSeat+i 会连接字符串，而不是进行您期望的数学加法。
尝试向屏幕发出警报：

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

是字段名称吗？

Answer 3

试试这个：

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

不要将 i 添加到 startSeat 值来获取座位 ID，而是在循环初始化中直接使用 startSeat 值。我相信发生的事情是您收到off-by-one 错误 因为您的 startSeat 值已经设置，然后您向其中添加了 i ，这使您领先了一位。

Answer 4

当你说循环遍历座位时，我假设你想在到达最大座位后继续从1开始计数？假设 numberOfSeats 是在某处定义的（我看不到它，但你必须在某处有它），你可以这样做：

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

所以该行完整：

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

Answer 5

您真正得到的是字符串连接，所以真正发生的是这样的：

i = 10
seatNum = 1

(seatNum+i) = "110"

尝试使用 parseInt() 函数将变量转换为整数类型：

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

Answer 6

在我看来，startSeat 是一种字符串类型。尽管 JavaScript 是无类型的，但 DOM 对象的值将默认为字符串。因此，您得到的是串联而不是加法。

使用 CMS 编写的内容。这应该可以解决你的问题。

Answer 7

您可能会重复太多次。尝试 i < for 循环中的 SeatNum - 1