[JavaScript]Sprawdzanie formularza Opcje

[malolat_ps]

Witam!

Tworzę sobie formularz rejestracji i chciałbym zrobić sprawdzanie w JS. Wymyśliłem, że jeżeli jakieś pole będzie wypełnione źle, bądź wogóle nie wypełnione, to nad formularzem pojawi się odpowiedni komunikat. Znalazłem taki skrypt, przerobiłem i niestety nie działa, pomimo, że wszystko jest praktycznie identyczne. Już straciłem pomysł, gdzie może być błąd, może Wy coś zauważycie  A więc:

formularz

[HTML] pobierz, plaintext 
<form method="post" action="" onsubmit="javascript:register();">
<div id="registerResult"></div>
<div class="rejestracja_naglowek">KONTO</div>
<div class="rejestracja_blok">E-mail: <input type="text" id="email" name="email" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_blok">Hasło: <input type="password" id="haslo" name="haslo" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_blok">Powtórz hasło: <input type="password" id="powtorzhaslo" name="powtorz_haslo" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_naglowek">DANE PERSONALNE</div>
<div class="rejestracja_blok">Imię: <input type="text" id="imie" name="imie" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_blok">Nazwisko: <input type="text" id="nazwisko" name="nazwisko" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_blok">Numer telefonu: <input type="text" id="telefon" name="numer_telefonu" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_blok">Ulica: <input type="text" id="ulica" name="ulica" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_blok">Numer domu: <input type="text" id="dom" name="numer_domu" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_blok">Kod pocztowy: <input type="text" id="kod" name="kod_pocztowy" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_blok">Miasto: <input type="text" id="miasto" name="miasto" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<?php $zapytanie_k = mysql_query("SELECT * FROM `kraje`");
?>
<div class="rejestracja_blok">Kraj: <select name="kraj" id="kraj" style="width: 254px; border: solid 1px #000000; color: #6c6c6c;">
<option value="0">-- wybierz --</option>
<?php while($fetchk = mysql_fetch_array($zapytanie_k))
{
$checked = ($fetchk['id'] == '120') ? 'selected' : '';
?>
<option value="<?php echo $fetchk['id']; ?>" <?php echo $checked; ?>><?php echo $fetchk['kraj']; ?></option>
<?php }
?>
</select></div>
<div class="rejestracja_naglowek">DANE DODATKOWE</div>
<div class="rejestracja_blok">Nazwa firmy: <input type="text" name="nazwa_firmy" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_blok">NIP: <input type="text" id="nip" name="nip" style="width: 250px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_blok" style="margin-right: 239px;">Subskrypcja: <input type="checkbox" name="subskrypcja" value="1" style="width: 15px; height: 15px; border: solid 1px #000000; color: #6c6c6c;" /></div>
<div class="rejestracja_blok" style="text-align: center; margin-top: 30px;"><input type="submit" value="Rejestracja" name="submit" style="width: 75px; height: 25px; background: #6c6c6c; border: solid 1px #000000; color: #E2E0E1;" /></div>
</form>
[HTML] pobierz, plaintext 

Oczywiście w nagłówku dokumentu znajduje się wpis:

[HTML] pobierz, plaintext 
<script language="javascript" type="text/javascript" src="js/register.js"></script>
[HTML] pobierz, plaintext 

A tutaj plik register.js:

[HTML] pobierz, plaintext 
function register() {
  
  var registerResult = document.getElementById("registerResult");
  var email = document.getElementById("email").value;
  var haslo = document.getElementById("haslo").value;
  var powtorzhaslo = document.getElementById("powtorzhaslo").value;
  var imie = document.getElementById("imie").value;
  var nazwisko = document.getElementById("nazwisko").value;
  var telefon = document.getElementById("telefon").value;
  var ulica = document.getElementById("ulica").value;
  var dom = document.getElementById("dom").value;
  var kod = document.getElementById("kod").value;
  var miasto = document.getElementById("miasto").value;
  var kraj = document.getElementById("kraj").value;
  var nip = document.getElementById("nip").value;
  
  var poprawnyemail = /^[a-zA-Z0-9._-]+@([a-zA-Z0-9.-]+\.)+[a-zA-Z0-9.-]{2,4}$/;
  var poprawnykod = /^\d{2}\-\d{3}$/;
  var poprawnynip = /^\(d{3}-\d{3}-\d{2}-\d{2})|(d{3}-\d{2}-\d{2}-\d{3}$/;
  
  var sprawdzemail = poprawnyemail.test(email);
  var sprawdzkod = poprawnykod.test(kod);
  var sprawdznip = poprawnynip.test(nip);
  
  if(!sprawdzemail) {
    registerResult.innerHTML = "Adres e-mail jest nieprawidłowy!";
    return false;
  }
  elseif(haslo == '') {
    registerResult.innerHTML = "Nie podano hasła!";
    return false;
  }
  elseif(powtorzhaslo == '') {
    registerResult.innerHTML = "Nie powtórzono hasła!";
    return false;
  }
  elseif(haslo != powtorzhaslo) {
    registerResult.innerHTML = "Podane hasła różnią się od siebie!";
    return false;
  }
  elseif(imie == '') {
    registerResult.innerHTML = "Nie wpisano imienia!";
    return false;
  }
  elseif(nazwisko == '') {
    registerResult.innerHTML = "Nie wpisano nazwiska!";
    return false;
  }
  elseif(telefon == '') {
    registerResult.innerHTML = "Nie podano numeru telefonu!";
    return false;
  }
  elseif(ulica == '') {
    registerResult.innerHTML = "Nie podano nazwy ulicy!";
    return false;
  }
  elseif(dom == '') {
    registerResult.innerHTML = "Nie wpisano numeru domu!";
    return false;
  }
  elseif(!sprawdzkod) {
    registerResult.innerHTML = "Podany kod pocztowy jest nieprawidłowy!";
    return false;
  }
  elseif(miasto == '') {
    registerResult.innerHTML = "Nie podano nazwy miasta!";
    return false;
  }
  elseif(kraj == '0') {
    registerResult.innerHTML = "Nie wybrano kraju z listy!";
    return false;
  }
  elseif(nip != '' && !sprawdznip) {
    registerResult.innerHTML = "Podany NIP jest nieprawidłowy!";
    return false;
  }
  else {
    return true;
  }
}
[HTML] pobierz, plaintext 

Gdzie jest błąd ? )

Ten post edytował malolat_ps 12.06.2009, 17:38:30

[athei]

Zrezygnuj ze sprawdzania poprawności danych za pomocą JS i rób to w php. Pomyśl co będzie gdy ktoś wyłączy js albo wyedytuje formularz...

[malolat_ps]

Czekałem aż ktoś to napisze. Czy ja gdziekolwiek stwierdziłem, że za pomocą php nie będę sprawdzał ? ;>  Formularz będzie sprawdzany dwukrotnie, za pomocą JS, a później jeszcze za pomocą PHP, na wypadek, gdyby ktoś miał właśnie wyłączony JS. Jednak chcę sprawdzać także przez JS, bo jest to wygodniejsze, gdy nie trzeba się męczyć z ciągłym przeładowywaniem strony.

Ten post edytował malolat_ps 12.06.2009, 18:03:00

[Filip_Łdz]

Wyłączone php ? Jeśli mówisz o userze to możesz sobie w przeglądarce js wyłączyć a nie php.

[malolat_ps]

Wyłączone php ? Jeśli mówisz o userze to możesz sobie w przeglądarce js wyłączyć a nie php.

A Ty mnie nie łap za słówka, bo można się było domyślić, że pomyliłem się z rozpędu i oczywiście o wyłączony JS mi chodzi. 

[Filip_Łdz]

Nie mam zamiaru łapać Cię za słówka. Zrób to tylko za pomocą php. Nie rozumiem po co robić sprawdzanie 2x.

[malolat_ps]

Bo to fajny bajer jest i mam taką wizję przy tworzeniu strony Uwierz, że dobrze się zastanowiłem czy chcę to wykorzystać i jestem pewny, że chcę. Więc jeżeli ktoś zna rozwiązanie problemu to będę bardzo wdzięczny

[Filip_Łdz]

Czasami chora upartość nie popłaca.

Może tutaj coś znajdziesz: http://marcinkaszynski.com/blog/index.php/...nie-formularzy/

Ten post edytował Filip_Łdz 12.06.2009, 18:29:35

[malolat_ps]

Ale to moja sprawa jak to zrobię i czy popłaca czy nie popłaca. Zwróciłem się na forum o pomoc i mam nadzieję, że ją otrzymam. Jeżeli nie jesteś w stanie mi pomóc to bądź tak miły i nie rozmywaj tego tematu. Amen

[Filip_Łdz]

Patrz link up.

ps. Odnoszę wrażenie, że jesteś jakoś strasznie podenerwowany. Spokojnie ^^

[malolat_ps]

Mogłeś odnieść takie wrażenie, bo rzeczywiście na chwilę obecną nie jestem najspokojniejszym człowiekiem świata, ale wynika to z faktu, że się rozchorowałem  więc bez urazy

Co do linka, to niestety nic mi to nie pomogło. Przy tym moim sprawdzaniu strona po kliknięciu się przeładowuje, więc wygląda to tak, jakby ta funkcja js nie była wogóle wywoływana. Nie mam zbyt dużego doświadczenia w JS, więc błąd może być bardzo prosty, jednak ja go nie zauważam

[Kshyhoo]

@malolat_ps, załącz skrypt przed Twoimi zmianami. Po za tym, nie karmić troli i nie spamować!

[malolat_ps]

Skrypt przed zmianami znajduje się tutaj