[js] funkcja sprawdz Opcje

[agataperz]

Witam,

mam pytanie jak zrobic z zapytania poniżej azeby sprawdziło czy a i b są wypelnione lub czy c jest wypełnione

[PHP] pobierz, plaintext 
<script type="text/javascript">
function Sprawdz(spr){
y=true;
if(spr.a.value=='' || spr.b.value==''){ komunikat=false; alert('Nie wypełniłeś wymaganych pól'); }
return komunikat;
}
</script>
[PHP] pobierz, plaintext 

Z góry dziekuje za pomoc

[peter13135]

a=='' && b='' || c=''

[kamil4u]

Warto jeszcze zadbać o nawiasy - dobry nawyk, żeby później nie zastanawiać się, który operator miał pierwszeństwo - nawet, gdy one nie są potrzebne. Czyli:

Kod

a && b || c  //nie
(a && b) || c // tak

[peter13135]

[PHP] pobierz, plaintext 
a && b || c  //tak
(a && b) || c // tak
[PHP] pobierz, plaintext 

To kwestia gustu. Jeśli programista wie, że koniunkcja ma pierwszeństwo nad alternatywą to nie musi pisać, jeśli nie koniecznie wie, to dać nawiasy nie zaszkodzi.

analogicznie: czy uważasz, że zapis 2+2*2 jest gorszy od 2+(2*2)

[agataperz]

programista wie ale soft nie wie i trzeba mu pokazać

Dzięki chłopaki.
Wersja poprawna o którą mi chodziło to:

[HTML] pobierz, plaintext 
(a || b) && c
[HTML] pobierz, plaintext 

Nie wiedziałam że w js można korzystać z tych samych warunków co ma sql


Nie wiem czemu ale niestety jak ktoś wpisze tylko jedna zmienna (np. imie) i wysle formularz wtedy wyskoczy blad. Jak poprawi to formularz sie nie chce wyslac - jaki może być problem?
Możecie pomóc?

[PHP] pobierz, plaintext 
<script type="text/javascript">
function sprawdz(spr){
y=true;
if((spr.zam_imie.value=='' || spr.zam_nazwisko.value=='') &&  spr.zam_firma.value==''){ komunikat=false; alert('Nie wypełniłeś wszystkich pól'); }
return komunikat;
}
</script>
[PHP] pobierz, plaintext 

później

[PHP] pobierz, plaintext 
<form id="formularz" action="finalizacja.php" method="post" onsubmit="return sprawdz(this)">
[PHP] pobierz, plaintext 

bez tych wierszy wszystko działa poprawnie a jak tylko wpisze to od razu jest problem przy poprawianiu danych. Jeżeli ktos wpisze wszystkie dane poprawnie to dziala.

Ten post edytował agataperz 25.08.2011, 19:30:40

[gargamel]

[JAVASCRIPT] pobierz, plaintext 
y=true;
[.....]
komunikat=false;
[.....]
return komunikat;
[JAVASCRIPT] pobierz, plaintext 

y to pewnie mialo byc komunikat ?

I swoją drogą - jeśli nie ma potrzeby stosowania zmiennych globalnych wewnątrz funkcji, lepiej stosować zmienne lokalne.
A najprościej to zrobić tak:

[JAVASCRIPT] pobierz, plaintext 
<script type="text/javascript">
function sprawdz(spr){
    if((spr.zam_imie.value=='' || spr.zam_nazwisko.value=='') && spr.zam_firma.value==''){
    alert('Nie wypełniłeś wszystkich pól');
    return false;
    }
    else{
    return true;
    }
}
</script>
[JAVASCRIPT] pobierz, plaintext 

Ten post edytował gargamel 25.08.2011, 19:39:43

[agataperz]

gargamel masz racje

jeszcze jedno ostatnie pytanie: Co trzeba wpisać ażeby sprawdził czy przycisk radio jest wciśnięty?

[kamil4u]

"checked JS" w google

[agataperz]

Poszukałam i zaczełam kombinować i wykombinowałam:

[PHP] pobierz, plaintext 
    <script type="text/javascript">
    function sprawdz(spr){
    komunikat=true;
    if((spr.zam_imie.value=='' || spr.zam_nazwisko.value=='' || spr.platnosc.checked==true ) && spr.zam_firma.value==''){ komunikat=false; alert('Nie wypełniłeś wszystkich pól'); }
    return komunikat;
    }
    </script>
[PHP] pobierz, plaintext 

ale niestety nie działa. Co robie źle?

Ten post edytował agataperz 26.08.2011, 12:01:00

[kamil4u]

Pokaż jeszcze kod HTML.
Kod wygląda OK, pewnie zły warunek napisałaś - aby to spr. trzeba mieć kod HTML(fragment z formularzem)

[agataperz]

[PHP] pobierz, plaintext 
<form name="formularz" action="finalizacja.php" method="post" onsubmit="return sprawdz(this)">
<table>
<tr><td align=right>Imię: </td><td><input type=text name=wys_imie size=30></td></tr>
<tr><td align=right>Nazwisko: </td><td><input type=text name=wys_nazwisko size=30></td></tr>
<tr><td align=right>Nazwa firmy: </td><td><input type=text name=wys_firma size=30></td></tr>
<tr><td align=right>Adres: </td><td><input type=text name=wys_adres size=30></td></tr>
<tr><td align=right>Kod pocztowy: </td><td><input type=text id=wys_kod_poczt name=wys_kod_poczt size=30></td></tr>
<tr><td align=right>Miejscowo?ć: </td><td><input type=text name=wys_miejscowosc size=30></td></tr>
<tr><td align=right>Telefon: </td><td><input type=text id=wys_tel name=wys_tel size=30></td></tr>
</table>
<table>
<tr><td align=right>Imię: </td><td><input type=text id=eyescript name=zam_imie size=30></td></tr>
<tr><td align=right>Nazwisko: </td><td><input type=text name=zam_nazwisko size=30></td></tr>
<tr><td align=right>Nazwa firmy: </td><td><input type=text name=zam_firma size=30></td></tr>
<tr><td align=right>Adres: </td><td><input type=text name=zam_adres size=30></td></tr>
<tr><td align=right>Kod pocztowy: </td><td><input type=text id=zam_kod_poczt name=zam_kod_poczt size=30></td></tr>
<tr><td align=right>Miejscowo?ć: </td><td><input type=text name=zam_miejscowosc size=30></td></tr>
<tr><td align=right>Telefon: </td><td><input type=text id=zam_tel name=zam_tel size=30></td></tr>
<tr><td align=right>e-mail: </td><td><input type=text name=zam_mail size=30></td></tr>
<tr><td align=right>Powiadomienie o statusie zamowienia: </td><td><input type="radio" name=powiadomienie value=sms>SMS <input type="radio" name=powiadomienie value=mail>e-mail</td></tr>
</table>
<input type="radio" name=platnosc value=pobranie > Płatno?ć przy odbiorze (+6,00 zł)<br>
<input type="radio" name=platnosc value=online > Platno?ci on-line (+<? echo $t_pro; ?> zł)<br>
<input type="radio" name=platnosc value=przelew > Przelew<br>
</td></tr></table>
<input type=submit value=dalej>
[PHP] pobierz, plaintext 

Ten post edytował agataperz 26.08.2011, 12:48:25

[kamil4u]

Musisz stworzyć odpowiedni warunek. Podpowiem, że:
spr.platnosc[0].checked - spr pierwsze radio
spr.platnosc[2].checked - spr trzecie radio

[agataperz]

a to nie można sprawdzić jedną komendą wszystkie radio name=platnosc?

[CuteOne]

[JAVASCRIPT] pobierz, plaintext 
var obj = document.formularz.platnosc; 
var len = obj.length;
 
for(var i=0; i<len; i++) {
 
   if(obj[i].checked) {
 
        alert(i);
   }
}
[JAVASCRIPT] pobierz, plaintext