Błąd: Kontrola typów danych, pilne Opcje

[Aztech]

Pytałem się swego czasu w tym wątku o kontrolę typów danych i zostałem namówiony do jej używania, problem który pojawił się w prawktyce jest następujący:

w klasie bazowej użytkownika mam:

[PHP] pobierz, plaintext 
<?php
 
public function __construct($rbx_db, string $nick, string $email) {		
 
	#inicjacja połaczenia AdoDB poprzez wyłołanie konstruktora rodzica
	parent::__construct($rbx_db);		
 
	if (!empty($email)){
	  if (!empty($nick)){
	  /* wycęty kod */
	  }
	  else throw new UserException(CUE_NICK_EMPTY);
	}
	else throw new UserException(CUE_EMAIL_EMPTY);	
  }	
 
?>
[PHP] pobierz, plaintext 

w skypcie pobierającym dane z forumularza mam wywołąnie tego konstruktora w linii numer 6 (listing poniżej). Parser wywala mi taki błąd

Kod

Fatal error: Argument 2 must be an object of class string in C:\usr\krasnal\www\robotECS\classes\user.php on line 120

Uwaga: tutaj jest to linia numer 3 w listingu wyżej.
Dlaczego tak się dzieje mimo tego, że wartości wpisane w pole formularza są stringiem?

[PHP] pobierz, plaintext 
<?php
 
if (!$error_found) {	
	//if every field is set						
	try {						  
	  $user = new User($rbx_db, $_POST["reg_nick"],$_POST["reg_email"]);	  
	  $user->setName($_POST["reg_fname"]." ".$_POST["reg_lname"]);		
	  $user->setPassword($_POST["reg_pass"]);	   
	  $user->setGG($_POST["reg_gg"]);
	  $user->setPhone($_POST["reg_phone"]);
	  $user->setPhoto($_POST["reg_photo"]);					
	  $user->setIcq($_POST["reg_icq"]);
	  $user->setTlenId($_POST["reg_tlen_id"]);	  
	  //$user->setDescription("");			  
	  $user->createUserSQL(1/* getLastUserID()*/);	  
	  $correct = true;
	}
	catch (UserException $e){	   
	  switch ($e->getMessage()) {
/* wyciety kod */
	  }
	  $e_msg .= $e->getMessage();			   
	}
	catch (IllegalDataException $e) {
	  switch ($e->getMessage()) {
/* wyciety kod */
	  }
	  $e_msg .= $e->getMessage();			   
	}
	catch (SqlException $e) {
	  switch ($e->getMessage()) {
/* wyciety kod */
	  }
	  $e_msg .= $e->getMessage();			   
	}
	catch (Exception $e) {
	  echo 'Unknown error!'.$e->getMessage();
	}	  
  }	
 
?>
[PHP] pobierz, plaintext 

[matid]

Niestety, w php type hinting odnosi się tylko i wyłącznie do obiektów i AFAIR tablic.
Jeśli chcesz mieć dokładniejszą kontrolę typów danych musisz sobie zaimplementować klasy String, Integer, itd. i używać ich zamiast zwykłych stringów i integerów.
Ma to swoje wady i zalety, czy chcesz z tego korzystać - przemyśl to sam.

[dr_bonzo]

madit juz ci wyjasnil.
Ja tylko dodam:
1. skoro uzywasz wyjatkow to nie zagniezdzaj tak jakbys ich nie uzywal

[PHP] pobierz, plaintext 
<?php
if (!empty($email)){
	  if (!empty($nick)){
	  /* wycęty kod */
	  }
	  else throw new UserException(CUE_NICK_EMPTY);
	}
	else throw new UserException(CUE_EMAIL_EMPTY);   
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
if ( empty($email)) // << brak !
{
	throw new UserException(CUE_EMAIL_EMPTY);   
}
	if ( empty($nick)) // << brak !
	{
	  throw new UserException(CUE_NICK_EMPTY);
	}
 
?>
[PHP] pobierz, plaintext 

uzyskasz mniej zagniezdzen, spowodowanych TYLKO przez kontrole argumentow.

2. Nie przekazuj nie istniejacych zmiennych

[PHP] pobierz, plaintext 
<?php
$user->setName($_POST["reg_fname"]." ".$_POST["reg_lname"]);
?>
[PHP] pobierz, plaintext 

najpierw sprawdz czy $_POST[ '...' ] istnieje.

[Aztech]

A jest gdzieś jakiś artykuł na te temat? I przy okazji co to są te tablice AFAIR?

@dr_bonzo: sprawdzam czy nie są puste, tutaj by nie zaśmiecać kodu niepotrzebnymi informacjami wyciąłem (jak widać jest tam komentarz: if every field is set )

Ten post edytował Aztech 3.01.2006, 21:17:36

[dr_bonzo]

LOL
AFAIR - As Far As I Remember -- jak pamietam to...

http://pl.php.net/language.oop5.typehinting
http://www.gadgetopia.com/post/4667


IITYIWHTKU
http://home.tiscali.be/lexicon/chat.htm


Ten post edytował dr_bonzo 3.01.2006, 21:25:27

[Aztech]

[OT mode on]
IMHO, LOL, BTW, EOT itp itd ale AFAIR nie znałem i wyszedłem na niezłą lamę Człek się uczy całe życie
[OT mode off]

EDIT: @dr_bonzo: dzięki za linka uświadamiająco-edukacyjnego

Ten post edytował Aztech 3.01.2006, 21:30:22