Jak sprawdzić pozostałe warunki if pomimo true?

Question

Witam, mam problem.. Kombinuje i kombinuję i póki co poza narastającą frustracją niczego nie osiągnąłem. Chcę sobie stworzyć "prostą" aplikację, listę produktów. Teraz chciałbym walidować pole input. oto krótki fragment kodu,

 

Mam taką funkcję:
 

function getItemValue() {    

    var arrValues = [];

    if( itemInput.value.length < 3 ) {    
        arrValues.push("minimum 3 znaki") ;
    } else if( itemInput.value.indexOf("#") !== -1 ) {    
        arrValues.push("nie mozna uzywac znaków specjalnych");
    } else {
        arrValues.push(itemInput.value);
    }

    return arrValues;
}

W tym drugim warunku to wiadomo, ze musze pobawić się RegExpami żeby nie przyjmował żadnych specjalnych znaków, ale to póki co mało istotne. założeniem jest wrzucenie wszystkich błedów jakie sobie tam wymyślę do tablicy arrValues i potem pokazanie ich użytkownikowi. Sek w tym, że chciałbym pokazać wszystkie, które tam sobie wpiszę (tzn. jeśli użytkownik wpisze coś w stylu "##" to dostanie dwie wiadomości, "minimum 3 znaki" oraz "nie wolno uzywac znakow !@#!%% itd". Problem jest taki, że nie moge wrzucić wszystkich potrzebnych w danej chwili błedow do tablicy, gdyż jesli pierwszy if wskazuje true to dalej już to wyrażenie nie idzie.. Czyli wrzucam jednorazowo tylko 1 błąd.. Mam wrażenie, że to jest dość proste ale tak mieszam i mieszam, troche ze switchem, troche z pętlami, i w sumie jestem coraz bardziej zakręcony.

Answer 1

usuń else if, i zostaw samego if

if( itemInput.value.length < 3 ) {    
        arrValues.push("minimum 3 znaki") ;
    } if( itemInput.value.indexOf("#") !== -1 )

 

Comments

Tylko wtedy jest problem inny..  Napisze to w ten sposób:

if(mniej niz 3 znaki) {

dupa

} else if( znaki specjalne) {

dupa

} else (wszystko super) { zwroc to co wpisal uzytkownik } ..

 

jesli usune else to ostatni warunek zwroci mi wszystko niezaleznie od powyzszych warunkow rowniez tresc typu "#@"

---

chodziło mi o susnięcie else if i zastąpienie na zwykły if. To ostatnie else zostaw

---

To dalej nie rozwiązywało mi mojego problemu, chyba, że coś robiłem źle. Natomiast poradziłem sboie w taki sposób:

 

function getItemValue() {	
	var arrValues = [];

	if( itemInput.value.length < 3 ) {	
		arrValues.push("minimum 3 znaki");
	} 

	if( itemInput.value.length > 10 ) {	
		arrValues.push("maximum 10 znaków");
	} 

	if( itemInput.value.indexOf("#") !== -1 ) {	
		arrValues.push("nie mozna uzywac znaków specjalnych");
	} 

	if( arrValues.length === 0 ) {
		arrValues.push(itemInput.value);
	}

	return arrValues;
}

 

Mam nadzieję, ze jest to zgodne z dobrymi praktykami.

Pozdrawiam :)

---

właśnie o tym ci cały czas pisałem xD

---

to coś źle zrozumiałem, bo pisałeś "zostaw ostatnie else" a tutaj żadnego else nie ma :P

---

ostanie, na samym doje jest samo else, bez ifów, o twój kod mi chodziło xD