Forum PHP.pl > [php] poprawne zapisanie if i else

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [php] poprawne zapisanie if i else

pijanyadmin

24.10.2007, 13:11:40

odbieram z formularza dane które sa wykonywane poprzez plik:

[PHP] pobierz, plaintext 
<?php 
// odbieramy dane z formularza 
$dologowania = $_POST["user"]; // loginnazwa
$loginpass = $_POST['pass'];
 
 
//połączenie z bazą 
   include("connect.php");
 
	 /sprawdzanie loginu w bazie
$waliduser = mysql_query("SELECT * FROM userdata WHERE loginkod = '$loginnazwa'");
$waliduserkom = mysql_num_rows($waliduser);
 
if (!$waliduserkom == 1) {
   }
 
else {
  echo 'Nazwa użytkownika <b>'.$dologowania.'</b> już istnieje.<BR>Wybierz inna nazwe';
  exit;
 
 
}
 
//sprawdzamy pole dologowania
if (strlen($dologowania)<3 or strlen($dologowania)>50 or !eregi("^[a-zA-Z0-9_.]+$",$dologowania)) { 
 
}
 
 else {
  echo 'Login zawiera niepoprawne znaki lub jest ich za mała ilośc.<br>';
  exit;
} 
 
//sprawdzamy pole haslo
if (strlen($loginpass)<6 or strlen($loginpass)>50) { 
   
}
 
 else {
  echo 'Hasło zawiera niepoprawne znaki lub jest ich za mała ilośc.<br>';
  exit;
} 
 
 
//sprawdzanie czy pola zostały wypełnione
if(empty($_POST['user'])) {
 
}
 
 else {
  echo 'Polę Login niezostało wypełnione';
  exit;
}
 
if(empty($_POST['pass'])) {
 
}
 
 else {
  echo 'Polę Hasło niezostało wypełnione';
  exit;
}
 
//dodanie do bazy
 
 
$loinid = mysql_query("SELECT * FROM userdata");
$loginids = mysql_num_rows($loinid);
 
$loginnr = $loginids + 1;
 
mysql_query("INSERT INTO userdata VALUES ('$loginnr','$dologowania','$loginnazwa','$loginpass','$data','$adresip')");
 
echo "Użytkownik <b>". $dologowania ."</b> został zarejestrowany.<BR>Data rejestracji: ".$data.".";
 
 
  ?>
[PHP] pobierz, plaintext

Niestety coś jest nie tak, jeśli jeden if niezostanie spełniony skrypt wykonuje sie dalej, ale jesli np login bedzie juz w bazie, to i tak jest dodawany nawet nie zostanie sprawdzone pole hasło. Jak zapisać to poprawnie aby były sprawdzane wszytkie if'y jeśli jakiś jest wpełniony niepoprawnie, wyświetlaja sie komunikaty gdzie a skrypt doda wynik dopiero gdy wszytko bedzie spełnione?

tsharek

24.10.2007, 13:16:00

stwórz sobie jakąś flagę, którą na początku będzie true, a w przypadku błedu zamiast "exit" ustawi ją na false. Później przy wpisie do bazy danych robisz sprawdzenie tej flagi i jeżeli jest true to znaczy że jest ok, a inaczej nie dodajesz do bazy, proste:)

pijanyadmin

24.10.2007, 13:29:21

tzn? jak to zrobić?

Gonzo44

24.10.2007, 13:55:23

zamiast exit daj $flaga_bledu = 1 i wtedy na koncu dajesz if( $flaga_bledu = 0) i tu dopisanie do bazy

tsharek

24.10.2007, 14:12:27

Cytat(pijanyadmin @ 24.10.2007, 14:29:21 )

tzn? jak to zrobić?

W tym forum nie znajdziesz gotowej odpowiedzi. Możmy Cię jedynie nakierować.

pijanyadmin

7.11.2007, 15:52:06

[PHP] pobierz, plaintext 
<?php
$result = False;
 
 
if($submit)
{
  if (!$_POST['user'] || !$_POST['pass'])
  {
    echo "Wszystkie pola sš obowišzkowe.";
  }
  elseif (strlen($loginpass)<6 or strlen($loginpass)>50)
  {
    echo "<font color=\"#000000\">Hasło musi mieć od 6 do 50 znaków.</font><br>";
  }
  elseif (!eregi("^[_[:alnum:]-]{3,10}$", $_POST['pass']))
  {
    echo "Niepoprawne hasło.";
  }
  elseif ($waliduserkom == 1)
  {
    echo "Nazwa użytkownika ".$dologowania." już istnieje.<BR>Wybierz inna nazwe"; 
  }
  elseif (!$user->add_user (strtolower ($_POST['user']), md5 ($_POST['pass'])))
  {
 
 
  }
  else
  {
    $result = True;
 	 //dodanie do bazy
$loinid = mysql_query("SELECT * FROM userdata");
$loginids = mysql_num_rows($loinid);
$loginnr = $loginids + 1;
 
mysql_query("INSERT INTO userdata VALUES ('$loginnr','$dologowania','$loginnazwa','$loginpass','$data','$adresip')");
 
echo "Użytkownik <b>". $dologowania ."</b> został zarejestrowany.<BR>Data rejestracji: ".$data.".";
  }
}
?>
[PHP] pobierz, plaintext

chodzi o coś takiego? jednak to nie zabardzo działa...

Regyam

7.11.2007, 16:13:11

Podam tylko kawalek:

[PHP] pobierz, plaintext 
<?php
if(empty($_POST['user'])) {
 
}
 
 else {
  echo 'Polę Login niezostało wypełnione';
  exit;
}
?>
[PHP] pobierz, plaintext

zamiast tego np to:

[PHP] pobierz, plaintext 
<?php
if(empty($_POST['user'])) {
 
}
 
 else {
  echo 'Polę Login niezostało wypełnione';
  $blad = '1';
}
?>
[PHP] pobierz, plaintext

A przy wpisie do bazy:

[PHP] pobierz, plaintext 
<?php
if($blad != 1){
WYKONAJ ZAPYTANIE DO BAZY
}else{
echo 'blad';
}
?>
[PHP] pobierz, plaintext

Apropo twojego kodu to nie rozumiem kompletnie tego:

[PHP] pobierz, plaintext 
<?php
$loinid = mysql_query("SELECT * FROM userdata");
$loginids = mysql_num_rows($loinid);
$loginnr = $loginids + 1;
?>
[PHP] pobierz, plaintext

Przeciez wystarczylo by w tabeli w bazie dac loginid jako klucz i samo by sie ustawialo na nastepne a nie tak manualnie bo jak dwoch userow siadzie na stronie i bedzie chcialo sie zarejestrowac i jeden wejdzie a drugie bedzie wlasnie wpisywal dane to bedziesz mial takie same id userow. Tzn w przypadku twojego kodu jak tam zastosujesz ifa jak ci podalem i wezmiesz to w klamre to tak sie nie powinno przydazyc ale lepiej zrob jak powiedzialem.

pijanyadmin

25.11.2007, 19:58:55

[PHP] pobierz, plaintext 
<?php 
// odbieramy dane z formularza 
 
$login = $_POST["login"];
$haslo = $_POST['haslo'];
$haslo2 = $_POST['haslo2'];
$email = $_POST['email'];
 
//pobieranie adresu IP
$adresip = $_SERVER["REMOTE_ADDR"];
 
//data dodania rekordu
$data = date("H:i:s d-m-Y");
 
//zmiana liter na małe
$login=strtolower($login);
 
//walidacja loginu, czarna lista
$sprlogin = array('admin', 'administrator', 'admini', 'root', 'bin', 'daemon', 'adm', 'lp', 'sync', 'shutdown', 
'halt', 'mail', 'news', 'uucp', 'operator', 'games', 'mysql', 'httpd', 'nobody', 'dummy', 'www', 
'cvs', 'shell', 'ftp', 'irc', 'debian', 'ns', 'download',
			   ' ') ;
 
//walidacja
 if ((empty($_POST['login'])) || (empty($_POST['haslo'])) || (empty($_POST['email']))) {
	 
		if (in_array($login, $sprlogin)) { echo"Wybrany login jest zakazany.<br>"; $blad = '1'; }
		if (strlen($login)<3 or strlen($login)>30 or !eregi("^[a-zA-Z0-9_.]+$",$login)){ echo"Login musi mieć od 3 do 30 znaków bez polskich liter i spacji.<br>"; $blad = '1'; }
		if (strlen($haslo)<3 or strlen($haslo)>32 or !eregi("^[a-zA-Z0-9_.]+$",$haslo)){ echo"Hasło musi mieć od 3 do 32 znaków bez polskich liter i spacji.<br>"; $blad = '1'; }
		if (!preg_match('|^[_a-z0-9.-]*[a-z0-9]@[_a-z0-9.-]*[a-z0-9].[a-z]{2,3}$|e', $email)){ echo"błędny email<br>"; $blad = '1'; } 
	  if (!$login) { echo "Wypełnij pole login<br>"; $blad = '1'; }
	  if (!$haslo) { echo "Wypełnij pole hasło<br>"; $blad = '1'; }
	  if (!$email) { echo "Wypełnij pole email<br>"; $blad = '1'; }
 
	 $blad = '1';
  }
 
 
 
//połączenie z bazą	
include("connect.php");
 
//kodowanie MD5
$haslo = MD5($haslo);
 
//sprawdzanie czy login już istnieje
$sprawdzlogin = mysql_query("SELECT * FROM userdata WHERE login = '$login'");
$wyniksprawdz = mysql_num_rows($sprawdzlogin);
 
 
  if ($wyniksprawdz == 1) {
	  echo "Wybrany login już istnieje.wybierz inna nazwe";
	  $blad = '1';
  }
 
 
 
if($blad != 1) {
 
$zapytanie = "INSERT INTO `userdata` (`id`, `login`, `haslo`, `email` , `data` , `adresip`) VALUES ('', '$login','$haslo','$email','$data','$adresip')";
$idzapytania = mysql_query($zapytanie);
 
echo "Użytkownik <b>" . $login . "</b> został zarejestrowany.<BR>przejdz do panelu logowania.";
 
  }
 
 
?>
[PHP] pobierz, plaintext

poprawiłem kod mając na uwadze Wasze wskazówki, ale teraz w ogóle nie działa walidacja pół formularza...

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.