Условия JavaScript

Question

Если бы кто-то мог указать, что я делаю неправильно, я был бы вечно благодарен! Кажется, я не могу получить правильную комбинацию скобок - как объединить несколько условий в одном выражении ?? Очевидно, я не ожидаю, что кто-то изменит код ниже, я просто хочу показать, чего я пытаюсь достичь.

Если бы кто-то мог объяснить мне логику, это было бы замечательно

Спасибо

function ChangeButton()
{
if 
((document.forms[0].IPR.value == "") && (document.forms[0].FNM1.value == "") && (document.forms[0].FNM1.value == "") && (document.forms[0].SURN.value == "") && (document.forms[0].GEND.value == "") && (document.forms[0].DOB.value == "") && (document.forms[0].CRIM.value == "") && (document.forms[0].ETHC.value == "") && (document.forms[0].DSBC.value == "") && (document.forms[0].MARK1.value == "") && (document.forms[0].NATC.value == "") && (document.forms[0].COBC.value == "") && (document.forms[0].COD.value == "") && (document.forms[0].FIVE.value == "") && (document.forms[0].PERM.value == "") && (document.forms[0].VISF.value == "") && (document.forms[0].USD.value == "") && (document.forms[0].HAD1.value == "") && (document.forms[0].HAD3.value == "") && (document.forms[0].HTEL.value == "") && (document.forms[0].HAEM.value == "") && (document.forms[0].FEES.value == "") && (document.forms[0].REF1TIT.value == "") && (document.forms[0].REF1ORG.value == "")     && (document.forms[0].REF1POS.value == "") && (document.forms[0].REF1AL1.value == "") && (document.forms[0].REF1AL3.value == "") && (document.forms[0].REF1AL5.value == "") && (document.forms[0].REF1EMA.value == "") && (document.forms[0].DISC.value == ""))
&&
((document.forms[0].PERM.value == "") && (document.forms[0].FIVE.value == "N")) 
&&
((document.forms[0].AGNT.value == "") && (document.forms[0].USD.value == "Y")) 
&&
((document.forms[0].CSTRT.value == "") && (document.forms[0].USD.value == "N") && (document.forms[0].CENDD.value == "") && (document.forms[0].CAD1.value == "") && (document.forms[0].CAD3.value == "") && (document.forms[0].CAD4.value == "") && (document.forms[0].CAPC.value == "") && (document.forms[0].CTEL.value == ""))
&&
((document.forms[0].AWDB.value == "") && (document.forms[0].FEES.value == "") && (document.forms[0].FEES.value == "Private Funds Self or Family") && (document.forms[0].AWDS.value == ""))
&&
((document.forms[0].RESEARCH.value == "Y") && (document.forms[0].RESSRT.value == "") && (document.forms[0].RESMOA.value == "") && (document.forms[0].RESAR.value == "") && (document.forms[0].RESDIS.value == ""))
{
document.getElementById('submitbutton').className = 'enabled'; 
}
else {
document.getElementById('submitbutton').className = 'disabled'; 
}
}

Answer 1

Вот исправленный код.

function ChangeButton()
{
    if 
    ((document.forms[0].IPR.value == "") && (document.forms[0].FNM1.value == "") && (document.forms[0].FNM1.value == "") && (document.forms[0].SURN.value == "") && (document.forms[0].GEND.value == "") && (document.forms[0].DOB.value == "") && (document.forms[0].CRIM.value == "") && (document.forms[0].ETHC.value == "") && (document.forms[0].DSBC.value == "") && (document.forms[0].MARK1.value == "") && (document.forms[0].NATC.value == "") && (document.forms[0].COBC.value == "") && (document.forms[0].COD.value == "") && (document.forms[0].FIVE.value == "") && (document.forms[0].PERM.value == "") && (document.forms[0].VISF.value == "") && (document.forms[0].USD.value == "") && (document.forms[0].HAD1.value == "") && (document.forms[0].HAD3.value == "") && (document.forms[0].HTEL.value == "") && (document.forms[0].HAEM.value == "") && (document.forms[0].FEES.value == "") && (document.forms[0].REF1TIT.value == "") && (document.forms[0].REF1ORG.value == "")     && (document.forms[0].REF1POS.value == "") && (document.forms[0].REF1AL1.value == "") && (document.forms[0].REF1AL3.value == "") && (document.forms[0].REF1AL5.value == "") && (document.forms[0].REF1EMA.value == "") && (document.forms[0].DISC.value == "")
        &&
        ((document.forms[0].PERM.value == "") && (document.forms[0].FIVE.value == "N")) 
        &&
        ((document.forms[0].AGNT.value == "") && (document.forms[0].USD.value == "Y")) 
        &&
        ((document.forms[0].CSTRT.value == "") && (document.forms[0].USD.value == "N") && (document.forms[0].CENDD.value == "") && (document.forms[0].CAD1.value == "") && (document.forms[0].CAD3.value == "") && (document.forms[0].CAD4.value == "") && (document.forms[0].CAPC.value == "") && (document.forms[0].CTEL.value == ""))
        &&
        ((document.forms[0].AWDB.value == "") && (document.forms[0].FEES.value == "") && (document.forms[0].FEES.value == "Private Funds Self or Family") && (document.forms[0].AWDS.value == ""))
        &&
        ((document.forms[0].RESEARCH.value == "Y") && (document.forms[0].RESSRT.value == "") && (document.forms[0].RESMOA.value == "") && (document.forms[0].RESAR.value == "") && (document.forms[0].RESDIS.value == "")))
        {
        document.getElementById('submitbutton').className = 'enabled'; 
        }
    else {
        document.getElementById('submitbutton').className = 'disabled'; 
    }
}

ИСПОЛЬЗОВАНИЕ и IDE, это сделает вашу жизнь простой .. Приветствия Eclipse IDE:)

Answer 2

Я вижу

...&&...  document.forms[0].FIVE.value == ""
...&&... document.forms[0].FIVE.value == "N"

Это никогда не будет правдой

РЕДАКТИРОВАТЬ

Я думаю, что вы должны изменить подход, попробуйте что-то вроде этого:

function ChangeButton()
{
    var frm = document.forms[0];
    var neverEmpty = ['field1','field2','field3'];
    var mustBe     = {field3:'Y', field4:'N'};

    var status = 'ok';

    for(var i = 0; i<neverEmpty.length; i++) {
        if(frm[neverEmpty[i]] == '') {
        status = 'ko';
        break;
    }
}

for(myField in mustBe) {
    if(frm[myfield] != mustBe[myField]) {
        status = 'ko';
        break;
    }
}
document.getElementById('submitbutton').className = status=='ok'? 'enabled' : 'disabled';

}

Answer 3

Вам понадобится еще 1 пара до и до первой фигурной скобки, если ( ... ) {...}

Answer 4

Дайте элементам ввода, которые должны быть непустыми, атрибут "class".Затем найдите все эти элементы, используя их вместо написания этого безумно уродливого кода.

Answer 5

Насколько я вижу, вам не нужны любые круглые скобки здесь (кроме тех, которые требуются для синтаксиса if).

if(document.forms[0].IPR.value == "" && document.forms[0].FNM1.value == "" &&
   document.forms[0].PERM.value == "" && document.forms[0].FIVE.value == "N" &&
   ...
   ) {
    document.getElementById('submitbutton').className = 'enabled'; 
} else {
    document.getElementById('submitbutton').className = 'disabled'; 
}

Answer 6

Вам понадобится еще один набор скобок вокруг всего лота, т.е.

Answer 7

вы не закрываете скобки

if (document.forms[0].IPR.value == "" && document.forms[0].FNM1.value == "" && ect...)

это так просто