Forum PHP.pl

Witam. Piszę walidację formularza a, że uczę się dopiero js to napotkałem błąd. Skrypt nie reaguje.

[PHP] pobierz, plaintext 
$('#rules').keyup(function() {
			var rules = $('#rules').val();
			if(rules.checked = false) {
			var komunikat_liczba_znakow7 = "Zaakceptuj regulamin.";
				$('#komunikat_liczba_znakow7').removeClass("poprawny");
				$('#komunikat_liczba_znakow7').addClass("niepoprawny");
				$('#komunikat_liczba_znakow7').html(komunikat_liczba_znakow7);	
			} else { 
				var komunikat_liczba_znakow7 = "Zaakceptowałeś regulamin."; 
				$('#komunikat_liczba_znakow7').removeClass("niepoprawny");
				$('#komunikat_liczba_znakow7').addClass("poprawny");
				$('#komunikat_liczba_znakow7').html(komunikat_liczba_znakow7);
			} 
		});
[PHP] pobierz, plaintext 

znalazłem jeszcze problem tutaj. Jak miałem mniej rozbudowane to działało lecz jak rozbudowałem trochę to już nie. Nie reaguje na submit.

[PHP] pobierz, plaintext 
	$('form').submit(function() {
			var name_czy_pusty = $('#name').val();
			var surname_czy_pusty = $('#surname').val();
			var email_czy_pusty = $('#email').val();
			var phone_czy_pusty = $('#phone').val();
			var cname_czy_pusty = $('#cname').val();
			var cstreet_czy_puste = $('#cstreet').val();
			var ccode_czy_pusty = $('#ccode').val();
			var cplace_czy_pusty = $('#cplace').val();
			var nip_czy_pusty = $('#nip').val();
			var rules_czy_pusty = $('#rules');
			var kumunikat_walidacji = '';
			var punkty_walidacji = 0;
			if(name_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = "Pole imię jest wymagane. <br>"; }
			if(surname_czy_puste.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole nazwisko jest wymagane. <br>"; }
			if(email_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole email jest wymagane."; }
			if(phone_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = "Pole telefon jest wymagane. <br>"; }
			if(cname_czy_puste.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole nazwa firmy jest wymagane. <br>"; }
			if(cstreet_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole ulica jest wymagane."; }
			if(ccode_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = "Pole kod pocztowy jest wymagane. <br>"; }
			if(cplace_czy_puste.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole miejscowość jest wymagane. <br>"; }
			if(nip_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole nip jest wymagane."; }
			if(rules_czy_pusty.checked == false) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Akceptacja regulaminu jest wymagana."; }
			if(punkty_walidacji == 10)
			{
				kumunikat_walidacji = "Wszystkie pola zostały wypełnione poprawnie. ";
			}
			$('#kumunikat_walidacji').fadeIn();
			$('#kumunikat_walidacji').html(kumunikat_walidacji);
			return false;
		});
[PHP] pobierz, plaintext 

Ten post edytował nikestylex7 24.10.2013, 15:51:14

Sprawdź w konsoli JavaScript - ona na pewno Cię nakieruje na błąd.

Czysta jak łza - niestety.

Swoją drogą Twoja walidacja działa bez sensu. Sprawdzane są wszystkie warunki, a wypisywany jest tylko ostatni błąd. Zamiast tego może dodawaj błędy do tablicy, a warunkiem prawidłowo wypełnionego formularza niech będzie np.:

[JAVASCRIPT] pobierz, plaintext 
if(bledy.length === 0) {...}
[JAVASCRIPT] pobierz, plaintext 

Bo sumowanie prawidłowych pól i porównywanie ich do liczby (i to stałej!) jest bardzo, bardzo złym pomysłem.

tzn nie rozumiem możesz prościej bądź jakiś przykład ?

albo może to przerobić tak aby jak jest źle podświetlało na czerwono

Bo zawsze zwracasz false:return false
FALSE ma byc gdy cos jest zle, gdy wszystko dobrze to ma byc return true;

Przeciez dodaje do siebie komunikaty, wiec wyswietla wszystko co bylo zle a nie tylko ostatni....

Tak ma byc koncowka:


Podstawy logiki sie klaniają....

tak tylko to nic nie zmienia dalej pokazuje ostatni błąd a nie wszystkie naraz.

[PHP] pobierz, plaintext 
	$('form').submit(function() {
			var name_czy_pusty = $('#name').val();
			var surname_czy_pusty = $('#surname').val();
			var email_czy_pusty = $('#email').val();
			var phone_czy_pusty = $('#phone').val();
			var cname_czy_pusty = $('#cname').val();
			var cstreet_czy_pusty = $('#cstreet').val();
			var ccode_czy_pusty = $('#ccode').val();
			var cplace_czy_pusty = $('#cplace').val();
			var nip_czy_pusty = $('#nip').val();
			var rules_czy_pusty = $('#rules');
			var kumunikat_walidacji = '';
			var punkty_walidacji = 0;
			if(name_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = "Pole imię jest wymagane. <br>"; }
			if(surname_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole nazwisko jest wymagane. <br>"; }
			if(email_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole email jest wymagane."; }
			if(phone_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + "Pole telefon jest wymagane. <br>"; }
			if(cname_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole nazwa firmy jest wymagane. <br>"; }
			if(cstreet_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole ulica jest wymagane."; }
			if(ccode_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + "Pole kod pocztowy jest wymagane. <br>"; }
			if(cplace_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole miejscowość jest wymagane. <br>"; }
			if(nip_czy_pusty.length != 0) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Pole nip jest wymagane."; }
			if(rules_czy_pusty.checked == false) { punkty_walidacji++; } else { kumunikat_walidacji = kumunikat_walidacji + " Akceptacja regulaminu jest wymagana."; }
			if(punkty_walidacji == 0)
            {
                kumunikat_walidacji = "Wszystkie pola zostały wypełnione poprawnie. ";
            }
            $('#kumunikat_walidacji').fadeIn();
            $('#kumunikat_walidacji').html(kumunikat_walidacji);
 
if(punkty_walidacji == 0) return true;
else return false;
		});
[PHP] pobierz, plaintext 

Na szybko czytalem i nie zwrocilem uwagi ze ty masz tam ELSY a punkty walidacji sluza do czegos innego przez to..... faktycznie porąbany sposob zes wymyslil

czyli musze się ich pozbyć ? chciałbym wiedzieć jak zrobić tak jak powyżej kolega napisał czyli dodanie do jednej tablicy wszystkich potem wyrzucenie na ekranie

Coś nadal jest nie tak wpisuje wszystko wyskakuje ze poprawnie kilkam submit i nic się nie dzieje tak jakby było coś źle wypełnione...

[PHP] pobierz, plaintext 
$('form').submit(function() {
			var name_czy_pusty = $('#name').val();
			var surname_czy_pusty = $('#surname').val();
			var email_czy_pusty = $('#email').val();
			var phone_czy_pusty = $('#phone').val();
			var cname_czy_pusty = $('#cname').val();
			var cstreet_czy_pusty = $('#cstreet').val();
			var ccode_czy_pusty = $('#ccode').val();
			var cplace_czy_pusty = $('#cplace').val();
			var nip_czy_pusty = $('#nip').val();
			var rules_czy_pusty = $('#rules');
			var kumunikat_walidacji = '';
			var punkty_walidacji = 0;
			if(name_czy_pusty.length != 0) { punkty_walidacji++;  kumunikat_walidacji = "Pole imię jest wymagane. <br>"; }
			if(surname_czy_pusty.length != 0) { punkty_walidacji++;  kumunikat_walidacji = kumunikat_walidacji + " Pole nazwisko jest wymagane. <br>"; }
			if(email_czy_pusty.length != 0) { punkty_walidacji++;  kumunikat_walidacji = kumunikat_walidacji + " Pole email jest wymagane. <br>"; }
			if(phone_czy_pusty.length != 0) { punkty_walidacji++;  kumunikat_walidacji = kumunikat_walidacji + "Pole telefon jest wymagane. <br>"; }
			if(cname_czy_pusty.length != 0) { punkty_walidacji++;  kumunikat_walidacji = kumunikat_walidacji + " Pole nazwa firmy jest wymagane. <br>"; }
			if(cstreet_czy_pusty.length != 0) { punkty_walidacji++;  kumunikat_walidacji = kumunikat_walidacji + " Pole ulica jest wymagane. <br>"; }
			if(ccode_czy_pusty.length != 0) { punkty_walidacji++;  kumunikat_walidacji = kumunikat_walidacji + "Pole kod pocztowy jest wymagane. <br>"; }
			if(cplace_czy_pusty.length != 0) { punkty_walidacji++;  kumunikat_walidacji = kumunikat_walidacji + " Pole miejscowość jest wymagane. <br>"; }
			if(nip_czy_pusty.length != 0) { punkty_walidacji++;  kumunikat_walidacji = kumunikat_walidacji + " Pole nip jest wymagane.<br>"; }
			if(rules_czy_pusty.is(':checked') == false) { punkty_walidacji++;  kumunikat_walidacji = kumunikat_walidacji + " Akceptacja regulaminu jest wymagana.<br>"; }
			if(punkty_walidacji == 0)
            {
                kumunikat_walidacji = "Wszystkie pola zostały wypełnione poprawnie. ";
            }
            $('#kumunikat_walidacji').fadeIn();
            $('#kumunikat_walidacji').html(kumunikat_walidacji);
 
if(punkty_walidacji == 0) return true;
else return false;
		});
[PHP] pobierz, plaintext 

O wiele, wiele bardziej elegancki kod:

[JAVASCRIPT] pobierz, plaintext 
$('form').submit(function () {
    var messages = {
            name:    "Pole imię jest wymagane.",
            surname: "Pole nazwisko jest wymagane.",
            // pozostałe pola i komunikaty na tej samej zasadzie
        },
        errors = [];
 
    $.each(messages, function (id, message) {
        if ($("#" + id).val() === "") {
            errors.push(message);
        }
    });
 
    if (!$('#rules').is(':checked')) {
        errors.push("Akceptacja regulaminu jest wymagana.");
    }
 
    if (errors.length > 0) {
        $('#kumunikat_walidacji')
            .html(errors.join("<br>"))
            .fadeIn();
        return false;
    }
});
[JAVASCRIPT] pobierz, plaintext 

Uzupełnij pozostałe pola według schematu.

P.S. Po co ustawiałeś komunikat przy wysyłaniu prawidłowo wypełnionego formularza? Przecież on się przeładowuje od razu (wysyła).

EDIT

Jeśli wszystkie pola mają mieć komunikat "Pole X jest wymagane", to równie dobrze możesz jeszcze uprościć i w obiekcie z komunikatami przekazywać tylko nazwę pola, np. name: "imię", a resztę komunikatu dopisywać dopiero przy wrzucaniu go do tablicy z błędami.

Ten post edytował SmokAnalog 24.10.2013, 23:00:39

Smok wiem że już dużo mi pomogłeś uprościłeś mój kod. Znalazłem też błąd jaki miałem a mianowicie w inputach zrobiłem tak że nie ma do nich labela tylko nazwy podawane są w środku czyli w każdym inpucie jest Imię Nazwisko i napisałem to tak:

[PHP] pobierz, plaintext 
<input  size="12" type="text" name="name" id="name" value="Imię"  onfocus="if(this.value == 'Imię') {this.value=''}" onblur="if(this.value == ''){this.value ='Imię'}" title="min. 3 znaki max. 16"/>
[PHP] pobierz, plaintext 

Wiem, że to jest bez sensu powiązując to z kodem js powyżej dlatego chciałbym jakoś te dwa skrypty połączyć czyli walidacje która sprawdza czy pola nie są puste oraz to żeby w inputach wyświetlały się poszczególne value.

Ten post edytował nikestylex7 25.10.2013, 00:40:58

A jeszcze mam pytanie jak chciałbym zrobić rejestracje 3 etapową 1 etap już mam czyli sprawdzanie danych. Potem chciałbym jak kliknie jakaś osoba na submit przenosi ją do działu płatności czyli dotpay. Płaci przekierowywuje ją na stronę główną wraz z tym wysyła dane na serwer. Chyba że zrobienie tego można jakoś ułatwić. Głównie chodzi o płatność za usługi handlowe.

Zapisuj dane z rejestracji do bazy do jakiejś tabeli tymczasowych użytkowników (nie opłaconych), a po dokonaniu płatności przenieś rekord, za który zapłacono do właściwej tabeli. DotPay wysyła odpowiedzi "w tle" tak więc musisz gdzieś przechować dane użytkownika by wykonać jakieś operacje po pozytywnej wpłacie. Oczywiście do DotPaya przesyłasz też informację o tym, jakiego ID użytkownika dotyczy płatność. Kombinuj.

a jak myślisz lepiej będzie zrobić takie coś, że po wypelnieniu formularza przechodzi na stronę płatności, gdy osoba zapłaci ma pełen dostęp i się może zalogować czy może lepiej zrobić tak, że rejestruje się potwierdza e-mail. Ma dostęp ale ograniczony i w zakładce aktywuj konto może zapłacić za korzystanie ?