[JavaScript]Checkbox i js Opcje

[nikestylex7]

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 

[SmokAnalog]

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