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 

[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