[PHP] Problem z walidacją php Opcje

[barthezpl]

Witam serdecznie. Mam mały problem z walidacją formularza. Otóż pisze skrypt dla hurtowni gdzie przy każdej pozycji wybieramy ilość sztuk danego towaru.Otóż nie wiem jak zrobić walidację tych pól( załóżmy że użytkownik wybiera 10 towarów i w dwóch zamiast liczb wpisał cyfry).Jak to rozgryźć, oto kody:
java script:

[HTML] pobierz, plaintext 
<script type="text/javascript">
function waliduj(formularz)
{
 
if (formularz.ILOSC.value.replace(/[0-9]+/,'')!='')
{
alert('Pole musi zawierac liczbe');
return false;
}
 
return true;
}
</script>
[HTML] pobierz, plaintext 

funkcja wyświetlająca towary:

[PHP] pobierz, plaintext 
<?php
function wyswietl_towary_z_cena()
 {
 
include("config.php");
include("db.php");
$zapytanie_towar = "select * from B2VTOWAR ('$FIRMA','".$_SESSION['KONTOODB']."','','','','',$TowStrMax*($TowNr-1)+1,$TowStrMax)   ";
$towar_odpowiedz=ibase_query($zapytanie_towar);
$max=2000;
 
	 echo '<table width =100% border=1 rules=rows align=center cellspacing=0 >';
	 echo '<tr>
	 <th width=30 align=left>Lp.</th>
	 <th width=20 align=left>Grupa</th>
	 <th width=50 align=left>Kod</th>
	 <th width=300 align=left>Nazwa</th>
	 <th width=30 align=center>Stan</th>
	 <th width=40 align=center>Jed</th>
	 <th width=50 align=center>Cena</th>
	 <th width=30 align=center>Cena<br> odbiorcy</th>
	 <th width=30 align=right>Ile sztuk</th></tr>';
	 echo '<form action="dodano_towary_do_kosza.php"  method="post" onSubmit="return waliduj(this)">';
 
 
for($b=1; $b<=$max; $b++)
 {
while($rzad = ibase_fetch_assoc($towar_odpowiedz))
 {
 
 
if ($rzad['KOD'] > '')
 {
 
 
 
	 echo '<tr valign=left><td width=30  align=left>'.$rzad['REKORD'].'</td>
	 <td width=20  align=left>' .$rzad['GRUPA'].'</td>
   <td width=50  align=left><input type=hidden name=KOD[] value=' .$rzad['KOD'].' /><a href="pokaz_notatnik.php?kodproduktu='.$rzad['KOD'].'" onclick="popUp(this.href, 'elastic', 300, 600); return false;" rel="external">'.$rzad['KOD'].'</a></td>
	 <td width=300 align=left>' .$rzad['NAZWA'].'</td>
	 <td width=30 align=right>';
	if($rzad['STAN'] > 0) {
	echo '<b><font color=blue>'. number_format ($rzad['STAN'], 2, ',', ' ').'</b></font>'; }
	 if($rzad['STAN'] <= 0) {
	 echo '<b><font color=red>'.number_format ($rzad['STAN'], 2, ',', ' ').'</b></font>' ;}
	 echo '</td><td width=40  align=center><input type=hidden name=JM[] value=' .$rzad['JM'].' />' .$rzad['JM'].'</td>
	 <td width=30 align=right>' .$rzad['CENA'].'</td>
	 <td width=30 align=right>' .$rzad['CENAODB'].'</td>
	 <td width=30 align=right><input type=text  size=5 name=ILOSC[] id= "'.$rzad['KOD'].'" value="" ></td></tr>';
 }
else
 {
$ile = $rzad['REKORD']-1;
 
break;
 
 }
 }
 }
	 echo '</table>';
 
	  echo '<table border=0 bgcolor=#0000FF width=100% height=20 align=center><tr><td align=right><input type="submit" value="zapisz " ></form></td></tr></table><br>';
	 echo '<table border=0 bgcolor=#0000FF width=100% height=20 align=center><tr><td> <p align=right><b><font size=2 color=red>Wyszukano</b> '.$ile.' rekordów</b></font></p></td></tr></table> ';
 
 
 
 
$iloscstron=ceil($ile/$TowStrMax);
 
echo '<table border=0 bgcolor=#COD9D9 width=80% height=20 align=center>';
 echo '<tr><td>';
if($_GET['page']>1){
 
	 echo '<a href="?page='.($_GET['page']-1).'">Poprzednie</a> ';
 }
echo '</td><td>';
for($a=1; $a<=$iloscstron; $a++)
 
 {
 
	 echo '[<a href=index2.php?page='.$a.'>'.$a.'</a>]';
 
 }
 echo '</td><td>';
if($_GET['page']<$iloscstron)
 
 {
	 echo '<a href="?page='.($_GET['page']+1).'">Następne</a> ';
 }
  echo '</td></tr></table>';
 }
?>
[PHP] pobierz, plaintext 

Zrobiłem takie małe javascript ale mi nie chodzi...

Ten post edytował barthezpl 22.10.2007, 11:54:38

[kossa]

Masz błąd w funkcji javascript bo ona zawsze Tobie zwróci true i formularz zostanie wysłany

[HTML] pobierz, plaintext 
function waliduj(formularz)
{
 
ret=true;
 
if (formularz.ILOSC.value.replace(/[0-9]+/,'')!='')
{
alert('Pole musi zawierac liczbe');
ret=false;
}
 
return ret;
}
[HTML] pobierz, plaintext 

a dalej kodu nie analizowałem i w sama funkcje js się też nie zagłębiałem

no i to nie php a javascript więc nie ta grupa na forum

Łukasz

[tsharek]

@kossa: wcale nie, funkcja javascript jest dobrze napisana (przynajmniej ta część z returnem). Return to return - nie jest wykonywana dalsza część kodu, tylko wychodzi z funkcji. Tak więc takie przepisanie jest nadmiarowe.

Po krótkiej analizie kodu chyba wiem co jest grane. Otórz w formularzu definiujesz tablicę ILOSCI[], a sprawdzasz jakby to była zwykła zmienna. Dlatego potrzeba będzie dołożenia pętli:

[HTML] pobierz, plaintext 
function waliduj(formularz)
{
 
for(i=0; i<formularz.ILOSC.length; i++)
{
if (formularz.ILOSC[i].value.replace(/[0-9]+/,'')!='')
{
alert('Pole musi zawierac liczbe');
return false;
}
}
 
return true;
}
</script>
[HTML] pobierz, plaintext 

Mogą byś też inne błędy, daj znać jak to nie będzie działać.
Aha: nie muszę chyba mówić że walidacja po stronie przeglądarki jest NIC nie warta? Może ona pełnić rolę "user-friendly", czyli wyświetla się błąd bez przeładowania strony, natomiast jest to żadne zabezpieczenie