Rejestracja - problem z nadpisywaniem i dodawiem niepotrzebnych pól Opcje

[dzesi]

Witam , mam problem ze sktyptem rejestracji który zaczołem dokanczać i mam kilka problemów z tym zwiazanych .Za bardzo czasu nie mam na doskonalenie swoich umiejetnosci w php , ale zawsze po pracy 1 godzinke poświecam:)
1.Nawet jak nic nie wpisze w formularze , to i tak ddaje mi puste pola ,chciałbym usunąć tą niedokonałośc w moim skrypcie
2.o to ze juz od tego dłubania w tym skrypcie juz mi sie myli , czy dobrze robie mechanizm błedów , bo niby dobrze ale one nie do końca działaja dobrze. Mam nadzieje ze mi pomozecie . Prosił bym o podanie przykładów kodu i co robie zle

[PHP] pobierz, plaintext 
<?php
header("Content-type: text/html; charset=utf-8");
class Rejestracja
{
		   private $login;
			private $haslo;
			private $haslotwo;
 
 function __construct()  {
   include_once('config_database.php');
  //odbieranie danych
 
 
				  $this->login=$_POST['login'];
				  $this->haslo=$_POST['haslo'];
				  $this->haslotwo =$_POST['haslotwo'];				
   
		}
   
		public function sprawdzanie_formularza()
		{
			try {
				if(empty($this->login))
				{
					throw new  Exception("b�ad nie poda�es loginu");
				}
				elseif(empty($this->haslo))
				{
					throw  new Exception("B��d nie poda�e� hasla");
				}
				if (empty($this->haslotwo)) {
					throw  new Exception("B��d nie poda�e� hasla");
				}
			}
			catch(Exception $e)
			{
				echo $e->getMessage();
			}
		}
   public function polaczenie()   {
	$polaczenie = mysql_connect(CONNECT,USER,PASSWORD) or die(connect_error);
	 mysql_select_db(DATABASE) or die(database_error);					
	 }
	public function spraw_user()  {
   $zapytanielog = "SELECT * FROM `user` WHERE login='" .$this->login . "'";
   $sprawdzlog = mysql_query($zapytanielog) or die(mysql_error());
   /* return*/$num_rows= mysql_num_rows($sprawdzlog) > 0;   
	 $ob=mysql_fetch_object($sprawdzlog);			 
	   }
	public function spradzanie_hasla()  {
 
   if ($this->haslo!==$this->haslotwo);
 
	}
	public function sprawdzanie_loginu()
	{
	if($this->login==$ob->login);
 
	}
	public function rej_user()  {
   $zapisrej="INSERT INTO `user`(`login`,`haslo`)VALUES('" . $this->login . "','" . $this->haslo . "')";
	return mysql_query($zapisrej);
	}
}
$start=new Rejestracja();
if (isset($_POST['rej']))
{
	$start->sprawdzanie_formularza();
}
if(!$start->spradzanie_hasla())
{
	print "Podaje� z�e has�o !!!";
}
if(!$start->sprawdzanie_loginu())
{
	print  "Taki user istnieje";
}
$start->polaczenie();
if ($start->spraw_user())
{
 echo "strona rejestracji";  
}
 
if (!$start->rej_user())
{
	   echo "Nie udalo sie dodac";
}
 
 
?>
[PHP] pobierz, plaintext 

[Cysiaczek]

To jest źle

[PHP] pobierz, plaintext 
<?php
public function sprawdzanie_formularza()
		{
			try {
				if(empty($this->login))
				{
					throw new  Exception("b�ad nie poda�es loginu");
				}
				elseif(empty($this->haslo))
				{
					throw  new Exception("B��d nie poda�e� hasla");
				}
				if (empty($this->haslotwo)) {
					throw  new Exception("B��d nie poda�e� hasla");
				}
			}
			catch(Exception $e)
			{
				echo $e->getMessage();
			}
		}
?>
[PHP] pobierz, plaintext 

powinno być tak:

[PHP] pobierz, plaintext 
<?php
public function sprawdzanie_formularza()
		{
 
				if(empty($this->login))
				{
					throw new  Exception("b�ad nie poda�es loginu");
				}
				elseif(empty($this->haslo))
				{
					throw  new Exception("B��d nie poda�e� hasla");
				}
				if (empty($this->haslotwo)) {
					throw  new Exception("B��d nie poda�e� hasla");
				}
		}
?>
[PHP] pobierz, plaintext 

i na zewnątrz

[PHP] pobierz, plaintext 
<?php
try
{
   $obiekt->sprawdzanie_formularza()
}
catch(Exception $e)
{
   echo $e->getMessage();
}
?>
[PHP] pobierz, plaintext 

Pozdrawiam.

[dzesi]

Dzinx za odpwoiedz ale mam problem z tym ze nioe wiem jak zablokować dodawanie pustych rekordów do bazy , tzn np jak wywołam strone rejestruj.php lub wysle pusty formularz t i tak sie dodaje . Szukałęm w necie ale nic sesownego co by mi pomogło nie znalzłem

[c0cr0ach]

Przed dodaniem rekordu do bazy sprawdzaj, czy zmienne w $_POST (tj. login, haslo) są ustawione. Możesz to najprościej zrobić używając instrukcji warunkowej.

[matix]

Najlepiej wykorzystaj FilterIterator, wrzuć po prostu całą tablicę $_POST i sprawdź, co wywala. I potem:

[PHP] pobierz, plaintext 
<?php
if ($oFilter->hasRefused())
echo 'Error: Nie wypełniłeś wszystkich formularzu, lub wypełniłeś je niepoprawnie';
?>
[PHP] pobierz, plaintext 

I tyle,
Pozdro

[dzesi]

matix możesz dać przykład bo nie za bardzo rozumiem o co ci chodzi i jak mam sie za to zabrać ?

[matix]

W takim razie wstawiam Kawałek kodu z FilterIterator (ja go wykorzystuję w moim fw, aby sprawdzić, aby pousuwać kilka konkretnych plików z mapy używanej do autoloadu):

[PHP] pobierz, plaintext 
<?php
class Matix_FilterMap extends FilterIterator {
 
	public
			$aDisAllowed = array ('.', '..', ''),
						$aArrayRefused = array();
 
 
	public function __construct($oInstance)
	{
		parent::__construct($oInstance);
	}
 
	public function accept()
	{
		if ( (in_array($this->current(), $this->aDisAllowed)) ) {
			$this->aArrayRefused[] = $this->current();
			return false;
		 }
 
		return true;
	}
 
	public function getRefused()
	{
		   return $this->aArrayRefused;
	}
 
	public function hasRefused()
	{
		   return $this->aArrayRefused != array();
	}
 
}
?>
[PHP] pobierz, plaintext 

Wykorzystujesz to np, tak:

[PHP] pobierz, plaintext 
<?php
private function _CreateMap()
	{
		$oFiles = new Matix_FilterMap(new RecursiveIteratorIterator(new RecursiveDirectoryIterator('library')));
 
		foreach ($oFiles as $sFile)
				$aMap[preg_replace('/(.*)..*/is', '$1', $sFile->getFileName())] = str_replace('', '/', $sFile->getPathName('/));
 
		$sMap = serialize($aMap);
 
		return (bool) file_put_contents(self::MAP_TEMP_FILE_NAME, $sMap);
	}
?>
[PHP] pobierz, plaintext 

W twoim przypadku byłoby to:

[PHP] pobierz, plaintext 
<?php
$oFiles = new Matix_FilterMap($_POST);
?>
[PHP] pobierz, plaintext 

Pokombinuj. Bardzo fajne, wydajne i przede wszystkim obiektowe i przejrzyste rozwiązanie.

Pozdro,
Matix.

Ten post edytował matix 14.12.2007, 15:07:48

[dzesi]

Cysiaczek .Twoje rozwizanie może jest poprawne ale prawde mówiac wywala mi dziwny błąd
Fatal error: Uncaught exception 'Exception' with message 'b�ad nie poda�es loginu' in C:\AppServ\www\log_rej\rejestracja.php:25 Stack trace: #0 C:\AppServ\www\log_rej\rejestracja.php(69): Rejestracja->sprawdzanie_formularza() #1 {main} thrown in C:\AppServ\www\log_rej\rejestracja.php on line 25
o to kawałek tej klasy

[PHP] pobierz, plaintext 
<?php
public function sprawdzanie_formularza()
		{
 
				if(empty($this->login))
				{
					throw new  Exception("b�ad nie poda�es loginu");
				}
				elseif(empty($this->haslo))
				{
					throw  new Exception("B��d nie poda�e� hasla");
				}
				if (empty($this->haslotwo)) {
					throw  new Exception("B��d nie poda�e� hasla");
				}
 
		}
 
 
//////////////////////////////////////////////
$start=new Rejestracja();
if (!isset($_POST['rej']))
{
	$start->sprawdzanie_formularza();
}
try{
   $start->sprawdzanie_formularza();
}
catch(Exception $e)
{
   echo $e->getMessage();
}
if(!$start->spradzanie_hasla())
{
	print "Podaje� z�e has�o !!!";
}
if(!$start->sprawdzanie_loginu())
{
	print  "Taki user istnieje";
}
$start->polaczenie();
if ($start->spraw_user())
{
 echo "strona rejestracji";  
}
 
if (!$start->rej_user())
{
	   echo "Nie udalo sie dodac";
}
?>
[PHP] pobierz, plaintext 

Dodaje też spradzanie czy login i haslo zostało wpisane , ale to tak nie działa i tak mi sie puste rekordy dodaja do bazy

[Cysiaczek]

[PHP] pobierz, plaintext 
<?php
$start=new Rejestracja();
if (!isset($_POST['rej']))
{
	$start->sprawdzanie_formularza(); // a gdzie blok try catch? Dlatego wywala uncatched
}
 
// dlaczego to jest dopiero tu?
try{
   $start->sprawdzanie_formularza();
}
catch(Exception $e)
{
   echo $e->getMessage();
}
?>
[PHP] pobierz, plaintext 

Pozdrawiam.

[dzesi]

Dzin za odpowiedz, ale sam sie tego dopatrzyłem bo dwa razy wywołanie tej samej funkcji poza klasa

Witam ponownie .Już mam takie nerwy że szok z niczym nie maiłem wiekszych problemów jak z tym skryptem rejestracji Jedno co mi nie działa to ze jak nawet wpisze dane w formularz czy ich nie wpisze to tak mi sie pokazuja błedy tzn nie podałes loginu nie podałes hasła , taki user istnieje .i nie do konca catch i try dziaął tzn jak nic nie wpisze to mi tylko podaje Nie podałes loginu a reszta jak by gdzieś gineła i sie nie wyświetlała .A i zrobiłem an ifach ten system zeby sprawzdał czy dany user podał haslo i login ale to teraz nie do konca działa bo i tak puste pola mi sie dodaja w abzie czy ktoś może mi w końcu pokazać na przykładzie jak to powinoo działać i wygladać

[PHP] pobierz, plaintext 
<?php
header("Content-type: text/html; charset=utf-8");
class Rejestracja
{
		   private $login;
			private $haslo;
			private $haslotwo;
 
 function __construct()  {
   include_once('config_database.php');
  //odbieranie danych
 
 
				  $this->login=$_POST['login'];
				  $this->haslo=$_POST['haslo'];
				  $this->haslotwo=$_POST['haslotwo'];	   
			   
				  }
   
 
   
		public function sprawdzanie_formularza()
		{
 
				if(empty($this->login))
				{
					throw new  Exception("b�ad nie poda�es loginu");
				}
				if(empty($this->haslo))
				{
					throw  new Exception("B��d nie poda�e� hasla");
				}
				if (empty($this->haslotwo)) {
					throw  new Exception("B��d nie poda�e� hasla");
				}
			  exit();  
		}
 
			 
 
 
 
   public function polaczenie()   {
	$polaczenie = mysql_connect(CONNECT,USER,PASSWORD) or die(connect_error);
	 mysql_select_db(DATABASE) or die(database_error);					
	 }
	public function spraw_user()  {
   $zapytanielog = "SELECT * FROM `user` WHERE login='" .$this->login . "'";
   $sprawdzlog = mysql_query($zapytanielog) or die(mysql_error());
  /* return*/ $num_rows= mysql_num_rows($sprawdzlog) > 0;   
	 $ob=mysql_fetch_object($sprawdzlog);			 
	   }
	public function spradzanie_hasla()  {
 
   if ($this->haslo!=$this->haslotwo);
 
	}
	public function sprawdzanie_loginu()
	{
	if($this->login==$ob->login);
 
	}
	public function rej_user()  {
   $zapisrej="INSERT INTO `user`(`login`,`haslo`)VALUES('" . $this->login . "','" . $this->haslo . "')";
	return mysql_query($zapisrej);
	}
}
$start=new Rejestracja();
if (!isset($_POST['rej']))
{
try{
   $start->sprawdzanie_formularza();
}
catch(Exception $e)
{
   echo $e->getMessage();
}
}
 
if(!$start->spradzanie_hasla())
{
	echo"Podajeś złe hasło";
}
if(!$start->sprawdzanie_loginu())
{
	echo "Taki user istnieje";
}
$start->polaczenie();
if ($start->spraw_user())
{
 echo "strona rejestracji";  
}
 
if (!$start->rej_user())
{
	   echo "Nie udalo sie dodac usera";
}
 
 
?>
[PHP] pobierz, plaintext 

Bardzo bym prośił o pokazenie tego jako kodu .troche mi gupio że w php pisze już ponad 1,5 roku a i tak mało wiem i nie do końca wiem jak cos zrobić, ale człowiek uczy sie przeż całe zycie

Witam już wyrywam sobie włosy z głowy i już trace cierpliwość kk....
Czy któryś z bardziej doświadczonych programistów niż ja może mi wkońcu pomóc.
Prubuje zrrobić system błedów ale wogole mi nie wychodzi ,też wogóle nie dodaje wartości do bazy nie wiem może juz za długo przy nim siedze i nie widze błędów .Bardzo prosze o pomoc. , i żeby ktoś pokazał mi to w kodzie jak to powinno być .
ps:Jeszcze tyle zcasu sie nie męczyłem z takim skryptem

[PHP] pobierz, plaintext 
<?php
header("Content-type: text/html; charset=utf-8");
class Rejestracja
{
		   private $login;
			private $haslo;
			private $haslotwo;
 
 function __construct()  {
   include_once('config_database.php');
  //odbieranie danych
 
 
				  $this->login=$_POST['login'];
				  $this->haslo=$_POST['haslo'];
				  $this->haslotwo=$_POST['haslotwo'];	   
			   
				  }
   
 
		public function sprawdzanie_formularza()
		{
				if(empty($this->login))
				{
					throw new  Exception("błąd nie podałeś loginu");
				}
				if(empty($this->haslo))
				{
					throw  new Exception("B��d nie podałesś hasla");
				}
				if (empty($this->haslotwo)) {
 
					throw  new Exception("Błąd nie podałeś hasla");
				}
			 if($this->haslo != $this->haslotwo) {
						$formOk = false;
						throw  new Exception("Hasla musza byc identyczne.");				
				}
 
		}
 
 
   public function polaczenie()   {
	$polaczenie = mysql_connect(CONNECT,USER,PASSWORD) or die(connect_error);
	 mysql_select_db(DATABASE) or die(database_error);					
	 }
	public function spraw_user()  {
   $zapytanielog = "SELECT * FROM `user` WHERE login='" .$this->login . "'";
   $sprawdzlog = mysql_query($zapytanielog) or die(mysql_error());
  return $num_rows= mysql_num_rows($sprawdzlog) > 0;   
	 $ob=mysql_fetch_object($sprawdzlog);			 
	   }
   /* public function spradzanie_hasla()  {
 
   if ($this->haslo!=$this->haslotwo); // 
  
	}
	public function sprawdzanie_loginu()
	{
	if($this->login!=$ob->login);
 
	}*/
	public function rej_user()  {
		if( $this->sprawdzanie_formularza() && $this->spraw_user()){
   $zapisrej="INSERT INTO `user`(`login`,`haslo`)VALUES('" . $this->login . "','" . $this->haslo . "')";
   return mysql_query($zapisrej);
		}
 
	}
}
$start=new Rejestracja();
$start->polaczenie();
if (!isset($_POST['rej']))
{
try{
   $start->rej_user();
}
catch(Exception $e)
{
   echo $e->getMessage();
}
}
 
 
/*if(!$start->spradzanie_hasla())
{
	echo"Podajeś złe hasło";
}
if(!$start->sprawdzanie_loginu())
{
	echo "Taki user istnieje";
}*/
 
//$start->rej_user();
/*
if ($start->spraw_user())
{
 echo "strona rejestracji";  
}
 
if (!$start->rej_user())
{
	   echo "Nie udalo sie dodac usera";
}
*/
?>
[PHP] pobierz, plaintext