Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [js] walidacja radio
Forum PHP.pl > Forum > Przedszkole
agataperz
28.08.2011, 09:36:27
Witam,

mam taki skrypt 
[PHP] pobierz, plaintext 
  1. function checkradio(){
  2.    var radio=false;
  3.    for (var i=0; i<document.formularz.length; i++){
  4.       if (document.formularz.elements[i].type == "radio"){
  5.          var x = document.formularz.elements[document.formularz.elements[i].name].length;
  6.             for (var y=0; y<x; y++){
  7.                if (document.formularz.elements[i+y].checked) break;
  8.             }
  9.             if (y == x){
  10.                return false;
  11.             } else radio=true;
  12.          i = i+x-1;
  13.       } else radio=true;
  14.    }
  15.    return radio;
  16. } 
[PHP] pobierz, plaintext


oraz 

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


funkcja sprawdz działa poprawie - sprawdza pola input text
niestety funkcja checkradio nie chce działać. możecie mi powiedzieć co jest źle?
CuteOne
28.08.2011, 09:44:01
W poprzednim Twoim temacie napisałem jak sprawdzać pola radio...

ps. jeżeli nie wiesz dlaczego "coś" w JS nie działa - zobacz moją sygnaturę
agataperz
28.08.2011, 10:14:13
Witaj CuteOne,

niestety ten skrypt:
[PHP] pobierz, plaintext 
  1.     var obj = document.formularz.platnosc;
  2.     var len = obj.length;
  3.  
  4.     for(var i=0; i<len; i++) {
  5.  
  6.     if(obj[i].checked) {
  7.  
  8.     alert(i);
  9.     }
  10.     }
[PHP] pobierz, plaintext

nie działa.

w konsoli błędów nie ma żadnego błędu.

flashdev
28.08.2011, 10:37:57
wpisz do konsoli JS poniższą linijkę:
Kod
document.write('<button onclick="alert(\'hej\');alert(\'hej :)\')">hej</button>')


następnie przetestuj skrypt:
Kod
document.write('<button onclick="return alert(\'hej\');alert(\'hej :)\')">hej</button>')


Odrazu zroszumiesz swój błąd.

A poza tym nie wstrzykuj kodu JavaScript bezpośrednio do ciała strony. Do tego służą pliki js, które opisują zachowanie strony.
Magic WWW
28.08.2011, 10:41:20
Nie zagłębiam się bardziej w temat, tylko mam propozycję dla autora żeby zaczął korzystać z biblioteki jQuery smile.gif Jest to wiele wygodniejsze jak czyste Javascript.
kamil4u
28.08.2011, 12:08:39
@Magic WWW: Kłóciłbym się - szczególnie przy takiej drobnostce jak walidacja formularza.
@agataperz: Robisz dość podstawowy błąd.
Kod
onsubmit="return sprawdz(this);return checkradio(this)"

Funkcja checkradio się nie wykona, dlatego że zostanie zwrócona wartość funkcji sprawdź i ta część kody zostanie zakończona.
Przykład:
Kod
<span onclick="return true; alert(123);">KLIK</span>


Rozwiązaniem problemu jest stworzenie nowej funkcji "sprawdź wszystko" lub zapisanie tego w takiej formie:
Kod
return (sprawdz(this) && checkradio(this))


Lecimy dalej...
Jak w funkcji jako parametr przekazujesz "this": checkradio(this) to w funkcji nie musisz już korzystać z
document.formularz.length, a z przekazanej referencji:
[JAVASCRIPT] pobierz, plaintext 
  1. function checkradio( formularz ){
  2. //pod zmienną formularz masz to samo co wcześniej w document.formularz
  3. // jest to lepsze rozwiązanie, gdyż skrypt nie musi ponownie szukać tego elementu
[JAVASCRIPT] pobierz, plaintext


W ogóle ta Twoja funkcja mi się nie podoba... Poczytaj o getElementsByTagName, a następnie napisz taki kod(ja przedstawiłem, go w pseudokodzie):
Kod
function sprawdz( formularz ){ //sprawdza czy któryś z inputów został zaznaczony - prawda jeżeli został, fałsz jeżeli nie został
zmienna INPUT = formularz.pobierz_wszystkie_inputy;
PĘTLA po INPUT{
   JEŻELI( INPUT.type == radio i INPUT.checked ) ZWRÓĆ prawdę
}
ZWRÓĆ fałsz
}


Powodzenia
CuteOne
28.08.2011, 14:21:23
W ramach testów stwórz sobie nowy plik index.html a w nim:
[HTML] pobierz, plaintext 
  1. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
  2. <html xmlns="http://www.w3.org/1999/xhtml">
  3. <head>
  4. <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2" />
  5. <title>Untitled Document</title>
  6. </head>
  7.  
  8. <body>
  9.  
  10. 	<form name="formularz" id="formularz" action="ddd.thml" onsubmit="validate(); return false;">
  11. 		<input type="radio" name="platnosc"  value="1"/>
  12. 		<input type="radio" name="platnosc"  value="2"/>
  13. 		<input type="radio" name="platnosc"  value="3"/>
  14.  
  15. 		<input type="submit" name="submit"/>
  16. 	</form>
  17.  
  18. 	<script type="text/javascript">
  19.  
  20. 		function checkRadio(inputName) {
  21.  
  22. 			var obj = document.formularz[inputName];
  23. 			var len = obj.length;
  24.  
  25. 			for(var i=0; i<len; i++) {
  26.  
  27. 				if(obj[i].checked) return true;
  28.  
  29. 			}
  30. 			alert('Radio nie został wciśnięty');
  31. 			return false;
  32. 		}
  33.  
  34.  
  35. 		function validate(form) {
  36.  
  37. 			var noerror = true;
  38.  
  39. 			if(!checkRadio('platnosc')) { noerror = false; }
  40. 			// itd. np. sprawdzanie inputów if(checkInput()) { noerror = false; } 
  41.  
  42. 			if(noerror) {
  43.  
  44. 				document.formularz.submit();
  45. 			}		
  46. 		}
  47. 	</script>
  48.  
  49. </body>
  50. </html>
[HTML] pobierz, plaintext
agataperz
28.08.2011, 18:13:33
CuteOne masz rację Twój skrypt działa. Nie wiem czemu mi nie chciał działać. Widocznie mam za dużo zmiennych do sprawdzenia w jednym pliku i się już pogubiłam.

jeżeli chodzi o funkcje checkInput to niestety chyba żle cos robie bo przepuszcza mnie przez submit

[HTML] pobierz, plaintext 
  1. function checkRadio(inputName) {
  2.          var obj = document.formularz[inputName];
  3.          var len = obj.length;
  4.          for(var i=0; i<len; i++) {
  5.          if(obj[i].checked) return true;
  6.          }
  7.          alert('Nie zaznaczyłe? formy płatno?ci. Wybierz kuriera a następnie formę platno?ci: przelew, przelew on-line, płatno?ć przy odbiorze.');
  8.          return false;
  9. }
  10. function checkInput(inputName) {
  11.          var obj = document.formularz[inputName];
  12.          var len = obj.length;
  13.          for(var i=0; i<len; i++) {
  14.          if(obj[i].checked) return true;
  15.          }
  16.          alert('Niestety nie wiemy komu mamy wysłać zamówienie. Nie podałe? swoich danych.');
  17.          return false;
  18. }
  19.  
  20. function validate(form) {
  21.          var noerror = true;
  22.          if(!checkRadio('platnosc') || checkInput('zam_imie')) { noerror = false; }
  23.          if(noerror) {
  24.          document.formularz.submit();
  25.          }
  26. }
[HTML] pobierz, plaintext
flashdev
28.08.2011, 18:36:04
@agataperz mam do Ciebie niedyskretne pytanie.
Uznałaś mnie za idiotę skoro zupełnie zignorowałaś moją prawidłową odpowiedź (podpowiedź) na Twoje pytanie.

Wracam na to forum po długim czasie i niestety widzę, że długo tu nie zabawię. Poziom wiedzy tutaj jest żenująco niski.
Sorry za offtopic. Musiałem to z siebie wylać.
agataperz
28.08.2011, 18:49:00
@flashdev tak wiem że błąd był w tym moim skrypcie. I dzieki Tobie za zwrocenie uwagi.
Poza tym chciałabym Ci tylko nasunąć pewną myśl - dział przedszkole mówi samo za siebie. Ja niestety nie mam zielonego pojecia o js a jeszcze bardziej o jquery a nie mowiac o ajaxie. Dzieki @kamil4u wziełam się za studiowanie tych języków niestety nie jestem w stanie w ciągu jednego dnia przeczytać wszystkiego.
Wiem również że czasami wypisuje glupoty jezeli chodzi o skrypty js ale ...
Jedyne co znam to html, php, sql. Mam nadzieje że sie nie zrazisz @flashdev do tego forum przeze mnie.
CuteOne
28.08.2011, 18:56:28
własność .checked pole typu "text" nie posiada smile.gif

[HTML] pobierz, plaintext 
  1. function checkInput(inputName) {
  2.          var obj = document.formularz[inputName];
  3.          var len = obj.length;
  4.  
  5.          if(len > 5 && len < 50) { //sprawdzamy długość wpisanych danych przedział to 5-50 ale sama możesz to ustalić
  6.  
  7.                   return true;
  8.          }
  9.          alert('Niestety nie wiemy komu mamy wysłać zamówienie. Nie podałe? swoich danych.');
  10.          return false;
  11. }
[HTML] pobierz, plaintext


Tak ps. nie prościej było to komuś zlecić? Zaoszczędzisz sporo czasu i jak mniemam pieniędzy, w końcu czas to pieniądz smile.gif

EDIT: flashdev: chyba za bardzo bierzesz to do siebie. I nie wiem czego oczekujesz po dziale Przedszkole o.O Tymbardziej nie wiem co oznacza "coś wartościowego wśród tych wszystkich śmieci"...
flashdev
28.08.2011, 18:56:45
Może masz rację, może za dużo wymagam.
Zraziło mnie tylko to, że próbuję dać Ci wędkę, a Ty idziesz na łatwiznę i bierzesz od nich rybkę. Oni dobrze wiedzą, że ja już rozwiązałem Twój problem, ale mimo wszystko karmisz ich dając im tutejszego plusa (pomógł). I tak to się tutaj kręci. Czasem tu będę zaglądał ale po prostu przez taki układ jaki aktualnie tu panuje ciężko znaleźć coś wartościowego wśród tych wszystkich śmieci.
Z mojej strony koniec tematu. Powiedzialem wszystko co miałem do powiedzenia.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.