Forum PHP.pl

Mam problem z rejestracją. Niby wszystko dodaje do bazy jak wpisze dane, ale jak zrobić instrukcję warunkową sprawdzającą czy wszystkie pola zostały uzupełnione? Kombinowałem ale zawsze robie gdzieś błąd bo zawsze jak kombinuje z if elseif to po wejściu w link register.php odrazu dodaje pusty wpis do bazy. Jeżeli nie wpisze nic w żadne pole i klikne rejestruj też 'zarejestruje" jako puste pola w bazie. (IMG:http://forum.php.pl/style_emoticons/default/sadsmiley02.gif) Pomóżcie (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

[PHP] pobierz, plaintext 
<?php
 
 
echo"
<form action=\"?link=register.php\" method=post>
Imię:         <input class=login type=text name=imie></input><br>
Nazwisko:     <input class=login type=text name=nazwisko></input><br>
Klasa:         <input class=login type=text name=klasa></input><br>
Login:         <input class=login type=text name=login></input><br>
Hasło:         <input class=login type=password name=pass></input><br>
E-mail:     <input class=login type=text name=email></input><br>
O sobie:     <textarea class=login type=text name=osobie></textarea><br>
<input class=submit type=submit value= Rejestruj ></input>
</form>
";
 
//niby powinno to sprawdzać?
if( $_GET['link']=="register.php"  && isset($_POST['imie']) && isset($_POST['nazwisko']) && isset($_POST['login']) && isset($_POST['pass']) && isset($_POST['email']) && isset($_POST['klasa']) && isset($_POST['osobie']) ) 
{
 
 
$imie = $_POST['imie'];
$nazwisko = $_POST['nazwisko'];
$login = $_POST['login'];
$pass = md5($_POST['pass']);
$klasa = $_POST['klasa'];
$email = $_POST['email'];
$osobie = $_POST['osobie'];
$ip = $_SERVER['REMOTE_ADDR'];
 
$db = mysql_connect( "", "", "" ); mysql_query("SET NAMES 'latin2'");
mysql_select_db( "" );
$query = "SELECT `password` FROM `uczen` WHERE `login`='$login'";
$query = mysql_query( $query );
$haslo = mysql_fetch_row( $query );
 
 
if( isset($haslo[0]) ) { echo"<br><span style=\"color:red\">Taki login już istnieje! Podaj inny login!</span>"; }
else
{ 
 
 
    $query = "INSERT INTO `uczen` (`id`, `imie`, `nazwisko`, `login`, `password`, `email`, `klasa`, `osobie`, `ip`) VALUES ('', '$imie','$nazwisko','$login', '$pass','$email','$klasa','$osobie','$ip')";
    if( $log = mysql_query($query) ){ echo "<center><span style=\"color:green\"><br><br><br>Zostałeś zarejestrowany! <img src=http://strona/img/ok.png ></span></center>"; }
    else{ echo"<center><span style=\"color:red\">Wystąpił nieoczekiwany Błąd! Spróbuj poźniej!<BR></span></center>"; } 
}
 
mysql_close( $db );
}
 
?>
[PHP] pobierz, plaintext 

moze tak

[PHP] pobierz, plaintext 
<?php
if($_GET['link']=="register.php"  && $_POST['imie'] != "" && $_POST['nazwisko'] != "" && $_POST['login'] != "" && $_POST['pass'] != "" && $_POST['email'] && $_POST['klasa'] != "" && $_POST['osobie'] != "" )
?>
[PHP] pobierz, plaintext 

niestety nie działa! nadal po kliknięciu rejestruj(bez wpisywania żadnych danych) dodaje puste wpisy w bazie i pojawia się ze rejestracja zakonczona pomyślnie.

[PHP] pobierz, plaintext 
<?php
//to co wyżej zostaje poza sprawdzeniem tego warunku czy pola są wypełnione
 
if( isset($haslo[0]) ) { echo"<br><span style=\"color:red\">Taki login już istnieje! Podaj inny login!</span>"; }
 
elseif ($imie=='' || $nazwisko=='' || $login=='' || $pass=='' || $klasa=='' || $email=='' || $osobie=='' || $ip=='')
{
echo '<p>Wszystkie pola muszą być wypełnione</p>';
}
else
{
   $query = "INSERT INTO `uczen` (`imie`, `nazwisko`, `login`, `password`, `email`, `klasa`, `osobie`, `ip`) VALUES ('".$imie."','".$nazwisko."','".$login."', '".$pass."','".$email."','".$klasa."','".$osobie."','".$ip."')";
   if( $log = mysql_query($query) ){ echo "<center><span style=\"color:green\"><br><br><br>Zostałeś zarejestrowany! <img src=http://strona/img/ok.png ></span></center>"; }
   else{ echo"<center><span style=\"color:red\">Wystąpił nieoczekiwany Błąd! Spróbuj poźniej!<BR></span></center>"; }
}
 
mysql_close( $db );
 
?>
[PHP] pobierz, plaintext 

Trochę zmodyfikowałem.
Po piewsze sprawdzasz czy jakieś pole nie jest puste, jeśli jest to dajesz komunikat, że tak być nie może, jeśli nie to idziesz dalej.
Po drugie moim zdaniem w zapytaniu miałeś błąd, chodzi o id, które zapewne jest u Ciebie auto_increment, jeśli tak nie podajesz go w ogóle w zapytaniu.
Po trzecie zmienne w zapytaniu powinny być zapisane w takim formacie ".$zmienna." inaczej chyba źle jest to inerpretowane.

Ten post edytował drzalek 9.10.2008, 21:02:57