Условия 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

Я вижу

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

Это не верно

EDIT

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

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 2

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

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

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

Answer 3

Вам понадобится еще один набор круглых скобок вокруг всей партии, т. Е. Если (a == b) {..}

Answer 4

Насколько я могу судить, вам не нужны любые круглые скобки (кроме тех, которые требуются синтаксисом 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 5

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

Answer 6

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

Answer 7

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

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'; 
    } 
} 

USE и IDE, он будет делать ур жизнь просто .. Приветствия в Eclipse IDE :)