Skrypt Walidacji w JS

pytanie zadane 12 września 2016 w JavaScript przez Damian Maksymilian B Nowicjusz (120 p.)
edycja 12 września 2016 przez Arkadiusz Waluk

Witam. Mam problem gdyż chce stworzyć skrypt który pozwoli wyłączyć w formularzu komunikat "Podana nazwa użytkownika lub hasło jest nieprawidłowe.Spróbuj ponownie." jeśli liczba znaków nie przekroczy 1 czyli jeśli będzie puste pole mam tutaj poniższy kod i nie wiem w czym tkwi problem. Proszę o pomoc.

       <script type="text/javascript">
            var modal = document.getElementById('LoginForm');
            
            var password = document.getElementsByName('password');
            
            var login = document.getElementsByName('login');
        
            var div = document.getElementsByClassName("form-error");
            
            window.length = function(event) {
                if(password.length==0 || login.length==0) {
                div.style.display = "none";
                }
            }
        </script>

2	komentarz 12 września 2016 przez niezalogowany Otwórz konsolę Popraw błędy z konsoli

1	komentarz 12 września 2016 przez Arkadiusz Waluk Ekspert (287,950 p.) 3. Wstawiaj kod na forum w specjalnych bloczkach.

4 odpowiedzi

odpowiedź 12 września 2016 przez kubaapk Nałogowiec (44,270 p.)

Skoro ta walidacja ma uniemożliwiać wysłania pustych inputów to daj im 'required'

<input typr="text" required>

komentarz 13 września 2016 przez efiku Szeryf (75,160 p.)

To powinno wystarczyć ! Zwłaszcza że walidacja jest też po stronie serwera;)

odpowiedź 12 września 2016 przez ScriptyChris Mędrzec (190,190 p.)

Kod umieść w odpowiednim bloczku.

jeśli liczba znaków nie przekroczy 1 czyli jeśli będzie puste pole

W takim razie pole nie będzie puste, bo ilość znaków równa jedności nie jest większa od jedynki.

document.getElementsByName('password')

document.getElementsByTagName( 'password' ); jeśli już

Jednakże, metoda ta zwraca Ci zbiór elementów (tak jak pozostałe metody ze zbioru document.getElementsBy*()), a nie pojedynczy element. Dlatego zapisz na końcu [0], albo pobierz element przez ID, albo skorzystaj z .querySelector();

window.length = function(event) {

Jeśli chcesz wychwytywać wpisanie pojedynczego znaku, to musisz nasłuchiwać zdarzenia 'input', to po pierwsze. Po drugie, skorzystaj z addEventListener(). Po trzecie, po co robić to aż na obiekcie window?

Nie za bardzo rozumiem logikę tego skryptu, ale może chodzi Ci o coś takiego: http://codepen.io/ChrissP92/pen/dpGrVB

komentarz 12 września 2016 przez Damian Maksymilian B Nowicjusz (120 p.)

Generalnie chodzi mi o to że jeśli nic nie wpisze w inpucie login lub hasło i kliknę submit to funkcja spowoduje wyłączenie określonego blocku, który pojawia mi się niepotrzebnie.

1	komentarz 12 września 2016 przez Comandeer Guru (601,590 p.) document.getElementsByTagName( 'password' ); jeśli już Niekoniecznie. document.getElementsByName zwraca elementy, które mają atrybut [name] równy podanej wartości.

komentarz 13 września 2016 przez ScriptyChris Mędrzec (190,190 p.)

Ok, ale Autor nie podał kodu HTML, więc stwierdziłem że po prostu źle zapisał tą metodę.

odpowiedź 12 września 2016 przez xmentor Nałogowiec (49,520 p.)
edycja 12 września 2016 przez xmentor

odpowiedź 12 września 2016 przez skrzatjedyny Gaduła (3,150 p.)
edycja 12 września 2016 przez skrzatjedyny

window.length = function(event) ...

Ten fragment nie ma sensu kompletnie. Spójrz, żeby wywołać jakas funkcje musisz ja do jakiegos zdarzenia przypisac (onload, onchange, itd.). W Twoim przypadku przyda sie zdarzenie onChange (http://www.w3schools.com/jsref/event_onchange.asp). Cos w ten desen:

password.onchange = function() {nazwaTwojejFunkcji}

function nazwaTwojejFunkcji() {
//jakis kod
}

Sam warunek tez nie ma prawa dzialac, popraw na cos w stylu:

if(password.length <=0) {
    alert("haslo jest zbyt krotkie");
}

Aha, zainteresuj sie biblioteka jQuery, duzo ulatwia ;)

komentarz 12 września 2016 przez ScriptyChris Mędrzec (190,190 p.)

http://forum.pasja-informatyki.pl/34559/w3schools-nie-szerzmy-falszywej-propagandy

I nie password.onchange tylko password.addEventListener( 'change', function );

zainteresuj sie biblioteka jQuery, duzo ulatwia ;)

Co w tym konkretnie przypadku może ułatwić?

komentarz 12 września 2016 przez skrzatjedyny Gaduła (3,150 p.)

No dobrze, moze z tym password.onchange poplynalem, ale nie wiedzialem ze na tamtej stronce robia takie buble. Dzieki za info ;)

Ale jQuery ulatwiloby np nasluch zdarzenia np. https://api.jquery.com/change/

1	komentarz 12 września 2016 przez ScriptyChris Mędrzec (190,190 p.) Ale jQuery ulatwiloby np nasluch zdarzenia np. Przy triggerowaniu może jest krótszy zapis, ale do podpinania zdarzeń możesz posłużyć się czystym JSem: http://youmightnotneedjquery.com/