Forum PHP.pl > [php] Poprawność skryptu ;)

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [php] Poprawność skryptu ;)

falfin

2.07.2007, 06:50:40

Witam. Zastanawiam się czy taki skrypt (jak ten ten poniżej) jest prawidłowy...Czy warto tworzyć jeden duży plik zawierający razem skrypt i formularz, czy lepiej je robijać na dwa pliki?
Pozatym jeżeli komus sie chce to prosze o wypisanie mi jakie błedy (jeżeli chodzi o styl pisania) popełniam.
Ps. zaznaczam że w php jestem jeszcze baaardzo zielony.
Pozdrawiam i z góry dziękuje za odpowiedzi.

[PHP] pobierz, plaintext 
<?php ob_start(); ?>
<?php session_start();?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html>
<head>
<link rel="stylesheet" href="style.css" type="text/css" />
<meta http-equiv="Content-Type" content="application/xhtml+xml; charset=iso-8859-2" />
<title>
Panel administracyjny-Rejestracja
</title> 
</head>
<body>
 
 
 
	<?php
	if (!isset($wyslany))
	{
	?>
 
<div id="duzy">
 
				<h1>Rejestracja</h1>
 
					<form action="rejestracja.php?wyslany='tak'" method=post>
 
					<table border=0>
					<tr bgcolor=olive>
					  <td width="150">Wymagane informacje</td>
					  <td width="100"></td>
					  </tr>
					</tr>
					<tr>
					  <td>Nazwa użytkownika*:</td>
					  <td align="center"><input type="text" name="nazwa_uz" size="30" maxlength="30"></td>
					</tr>
					<tr>
					  <td>Hasło*:</td>
					  <td align="center"><input type="text" name="haslo" size="30" maxlength="30"></td>
					</tr>
					<tr>
					  <td>Powtórz hasło*:</td>
					  <td align="center"><input type="text" name="haslo2" size="30" maxlength="30"></td>
					</tr>
					<tr>
					  <td>Adres e-mail*:</td>
					  <td align="center"><input type="text" name="email" size="30" maxlength="30"></td>
					</tr>
					<tr>
					  <td colspan="2" align="right"><input type="submit" value="wyslij"></td>
					</tr>
 
 
 
					</table>
					</form>
 
 
</div>
 
</body>
</html>
 
 
 
<?php
}
elseif (isset($wyslany)  && !isset($prawidlowy))
{
 
$nazwa_uz= $_POST['nazwa_uz'];
$haslo= $_POST['haslo'];
$haslo2= $_POST['haslo2'];
$email= $_POST['email'];
$czas = time();
 
 
															// SPRAWDZENIE	   CZY	  WSZYSTKIE	  DANE		 ZOSTALY	  PODANE
	if (!$nazwa_uz || !$haslo || !$haslo2 || !$email)
			{
			echo ' Nie podales wszyszystkich danych. <br />Wróć do poprzedniej strony i uzupełnij brakujące pola .' ;			// SPRAWDZENIE	   CZY	  WSZYSTKIE	  DANE		 ZOSTALY	  PODANE
			exit;
			}
 
 
	else
	{
															//SPRAWDZENIE	 POPRAWNOSCI	DANYCH
		if(strlen($nazwa_uz)>16)
		{
		echo ' Podana nazwa użytkownika jest za długa. Powinna ona zawierać mniej niż 16 znaków.';
		exit;
		}
 
		if(strlen($haslo)<5)
		{
		echo 'Podane hasło jest za krótkie! Ze względów bezpieczeństwa,powinno ono zawerać conajmniej 5 znaków.';
		exit;
		}
 
			if ($haslo != $haslo2)
			{
			echo 'Podane hasła są różne!';
			exit;
			}
 
		if (!eregi ("[a-zA-z0-9_]+@[a-zA-z0-9-]+.[a-zA-z0-9-.]+$", $email))
			{
			echo 'Podany adres e-mail jest nie poprawny.';
			exit;
			}
	}		
 
 
		// JEZELI	 WSZYSTKO	 SIE	 ZGADZA	TO	 REJESTRACJA
 
									// SPRAWDZENIE	 CZY	 MECHANIZM	 DODAWANIA	 ZNAKOW	 JEST	 WLACZZONY	 I	 JEZELI	 NIE	 TO	 ICH	 DODANIE	
		if (!get_magic_quotes_gpc())
			{
			$nazwa_uz = addslashes($nazwa_uz);					
			$haslo = addslashes ($haslo);
			$email=addslashes ($email);
			}
 
									//POLACZENIE	 Z	BAZA	DANYCH
		require('polaczenie.php');	 //PLIK	 POLACZENIA	 W	 OSOBNYM	 PLIKU	 PONIEWAZ	BEDZIE
										//WYKORZYSTYWANY	W	INNYCH	 APLIKACJACH	:)
 
		mysql_select_db('test');
 
														//JEZELI	NAWIAZANO	POLACZENIE
														//SPRAWDZENIE	CZY	PODANA	NAZWA	 NIE	 JEST	JUZ	ZAJETA
		$zapytanie1= "select * from uzytkownicy where nazwa_uz='$nazwa_uz' ";
		$wynik1 = mysql_query($zapytanie1, $mysql) or die(mysql_error());	
		if(!$wynik1)
		{
			echo 'Przepraszamy rejestracja w tej chwili nie jest mozliwa. Prosze spróbowac pozniej
.<br/>';
			exit;	
		}
		else
		{
			if(mysql_num_rows($wynik1)>0)
			{
				echo 'Wybrany nick jest juz zajety. Wybierz inny i spróuj ponownie.<br/>';
				exit;
			}
 
			else
			{									//JEZELI	 NICK	 NIE	 JEST	 ZAJETY	REJESTRACJA
			$zapytanie2= "insert into  uzytkownicy values ('$nazwa_uz' , sha1('$haslo') , '$email' , 'administrator', '$czas' ) ";
			$wynik2 = mysql_query($zapytanie2, $mysql) or die(mysql_error());
		$prawidlowy='tak';
			}
 
			if(!$wynik2)
			{
				echo '2Przepraszamy rejestracja w tej chwili nie jest mozliwa. Prosze spróbowac poznie
j.<br/>';
				exit;
			}									
 
			else
			{
				echo'Rejestracja zakończona sukcesem. W celu zalogowania wróć na strone główną';
				echo '<a href="index.php">Powrót </a>';
				exit;
			}
		}
}
?>
[PHP] pobierz, plaintext

Ps. Jeszcze jedno

na razie ten skrypt rejestruje wszystkich domyślnie jako administratorów,
($zapytanie2= "insert into uzytkownicy values ('$nazwa_uz' , sha1('$haslo') , '$email' , 'administrator', '$czas' ) ";)
ale jest on tylko na localhoscie i tak jest po prostu mi wygodniej. Oczywiście niedługo zostanie to zmienione

strife

2.07.2007, 06:55:13

Cytat

.Czy warto tworzyć jeden duży plik zawierający razem skrypt i formularz, czy lepiej je robijać na dwa pliki?

Nie warto bo mieszasz, logikę z wyglądem. Umieść tą walidację w jakiejś klasie bądź funkcji, takich rzeczy nie robi się z poziomu szablonu. Poczytaj o założeniach MVC. Co do stylu pisania, wywal zbędne klamry

[PHP] pobierz, plaintext 
<?php if ( isset( $dupa ) ) : ?>
KOD HTML
<?php endif; ?>
[PHP] pobierz, plaintext

Tak jest bardziej przejrzyście, poza tym więcej wcięć, staraj się unikać liczb w nazwach zmiennych. To tyle ode mnie.

Pozdrawiam.

falfin

2.07.2007, 12:26:36

Dzięki strife za odpowiedz. Teraz juz chyba wiem co z tym zrobić.
Zkmylił mnie jeden z rozdziałów ksiązki Biblia PHP 4 gdzie zalecali używania samoprzetwarzających sie formularzy- rozumię ze to już nie aktualne?

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.