Как сделать, чтобы при итерации прибавлялся 1 балл?

Question

[Alex931]

Здравствуйте. Есть такой код.

calc.onclick = function() {
    var myform = this.form,
        chbx = myform['mytest'],
        chnm = myform['myname'],
        sum = 0;
    for (var i = 0; i < chbx.length; i++) {
        if (chbx[i].checked && (chbx[i].value == 'Дождь' || chbx[i].value == 'Снег'))
            sum++;
    }
    for (var i = 0; i < chnm.length; i++)
        if (chnm[i].checked && (chnm[i].value == 'Ваня'))
            sum++;
    if (myform.street.value == 'Московская') sum++;

    myform.result.value = sum;
    if (sum > 2) {
        document.querySelector('.good-answer').style = "display: block;"
    }
    // if (sum < 2) {
    //     document.querySelector('.bad-answer').style = "display: block;"
    // }
    if (sum < 3 || sum == 2) {
        document.querySelector('.bad-answer').style = "display: block;"
    }
};

В этом условии
if (chbx[i].checked && (chbx[i].value == 'Дождь' || chbx[i].value == 'Снег'))
sum++;
он прибавляет 2 балла, если условие выполняется. Как его переписать, чтобы при выполнении этих 2 условий sum++ прибавлялось +1?

Answer 1

[Samad_Samadovic]

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

Comments

[Alex931]

Вот так переписал, теперь 1 вопрос вообще не считает, если ответ правильный
calc.onclick = function() {
var myform = this.form,
chbx = myform['mytest'],
chnm = myform['myname'],
sum = 0;
var i = 0;
i < chbx.length;
i < chnm.length;
// for (var i = 0; i < chbx.length; i++)
// {
if (chbx[i].checked && (chbx[i].value == 'Дождь' || chbx[i].value == 'Снег'))
sum++;
// }
// for (var i = 0; i < chnm.length; i++)
if (chnm[i].checked && (chnm[i].value == 'Ваня'))
sum++;
if (myform.street.value == 'Московская') sum++;

myform.result.value = sum;
if (sum > 2) {
document.querySelector('.good-answer').style = "display: block;"
}
// if (sum < 2) {
// document.querySelector('.bad-answer').style = "display: block;"
// }
if (sum < 3 || sum == 2) {
document.querySelector('.bad-answer').style = "display: block;"
}
};


Менял операторы с и на или || && Вот так

if (chbx[i].checked && (chbx[i].value == 'Дождь' && chbx[i].value == 'Снег'))
            sum++;

вообще при 2 правильных ответах +1 балл всего.

[Samad_Samadovic]

Alex931, Причём тут оператор, когда у вас цикл выполняет два действия вместо одно из-за того что i=0, а массив = 1. А если быть точнее у вас цикл вначале выполняет действие когда i=0(он добавляет единицу) и когда у вас i=1(он добавляет ещё единицу)