Co zwraca konstruktor w razie błędu? Opcje

[czarek1986]

Witam mam do was takie banalne pytanie, na którym niedawno sie zaciełęm. Tworzę sobie nowy obiekt jakiejś klasy. Ma ona w konstruktorze parametr id, na podstawie którego pobierane są dane z bazy które wypełniają reszte pól w obiekcie. Do tej pory myślałem, że aby przerwać tworzenie obiektu w wypadku gdy wpisu o podanym id nie ma w bazie za pomocą return false. Okazał się jednak, że tak nie jest w moim przypadku, gdyż obiekt tworzy się mimo to tylko z pustymi polami. Ogólnie jego kod wygląda tak

Kod

class Menu extends ObjectModel {
    
    protected $id_menu;
    protected $nazwa_menu;
    
    
    public function __construct($id=null) {
        parent::__construct();
        if(!empty($id))
        {
            if(!is_numeric($id))
                return false;
            $sql = "SELECT * FROM menu WHERE id_menu = '$id'";
            $menu = $this->dbhandle->select($sql);
            
            if(!empty($menu))
                $this->setAll($menu[0]);
            else
                return false;
        
          
        }
    }

}

To jest tylko kod contruktora ale powinno wystarczyć. Co mam zrobić aby w przypadku podania nieprawidłowego id obiekt nie został utworzony?

[marcio]

Wydaje mi sie ze nie mozna uzyskac to czego ty chcesz poniewaz obiekt musi sie stworzyc by kod kontruktora sie wykonal.

By uzwglednic czy dany obiekt ma sie utworzyc lub nie musialbys to sprawdzic przed jego inicjalizacja.

[Crozin]

Wyrzuć wyjątek.

[kefirek]

Może to pomoże
http://php.net/manual/pl/exception.getmessage.php

Powód edycji: [batman]: Pomorze to region Polski :)

[pp-layouts]

Konstruktor nie zwraca nic. Jakakolwiek instrukcja return w konstruktorze jest ignorowana (a w każdym razie nic nie robi). Tak dzieje się zawsze. Kiedy obiekt jest tworzony dyrektywą new, konstruktor jest wykonywany, a do zmiennej przypisywana jest instancja obiektu (TO NIE JEST WARTOŚĆ ZWRACANA PRZEZ KONSTRUKTOR!).

Zamiast od razu kombinować z wyjątkami jest IMHO o wiele prostszy sposób sprawdzenia, czy konstruktor "dał radę":

[PHP] pobierz, plaintext 
class krowa {
  public $gotowa = false;
  public function __construct($trawa = false) {
    if ($trawa) $this->gotowa = true;
  }
  public function mu() {
    return $this->gotowa ? 'MUUUUUUU!' : 'A trawsko gdzie?!';
  }
}
[PHP] pobierz, plaintext 

Masz klasę krowa. Odpalasz konstruktor, w którym możesz podać trawę, lub nie. Jeśli nie podasz, to będzie nasz błąd. Przy właściwym utworzeniu, nasza krowa prawidłowo obsługuje metodę mu(). Bez trawy, krowa zwraca tekst, którego nie powinna zwracać. Chcemy temu zapobiec, więc robimy to tak:

[PHP] pobierz, plaintext 
$k = new krowa($dla_krowy);
if ($k->gotowa) echo $k->mu();
[PHP] pobierz, plaintext 

Wszystko zależy, czy błąd przy tworzeniu konstruktora ma tylko spowodować niewykonanie jakiś metod obiektu lub części kodu, czy ma położyć program. Bo jak ma położyć, a Ty albo user mają wiedzieć co się stało, to oczywiście throw new Exception('I tutaj co poszło nie tak....'); Na takie okazje przygotowałem sobie własne rozszerzenie klasy Exception, które wywala mi "pełen raport komisji śledczej".

[tvister]

Z konstruktorem nie powalczysz.
Możesz zrobić loader coś w tym stylu

[PHP] pobierz, plaintext 
klasa::zaladuj_obj()
[PHP] pobierz, plaintext 

Statyczna metoda zwracająca obiekt własnej klasy lub null-a.
Dodatkowo opisać phpdoc-em i vista wio.

[skowron-line]

Statyczna metoda zwracająca obiekt własnej klasy

Czy to nie brzmi jak singelton ?

[marcio]

Czy to nie brzmi jak singelton ?

Moze mu chodzi o cos a'la factory() z kohany.

[skowron-line]

Moze mu chodzi o cos a'la factory() z kohany.

A to tylko kohana stosuje wzorzec faktory ?
Edit a i factory nie zwaraca obiektu własnej klasy
factory

Ten post edytował skowron-line 1.12.2009, 10:38:46

[darko]

To może wywołuj najpierw funkcję, w której będziesz sprawdzał czy wpis o podanym id jest w bazie, jeśli jest, to utwórz obiekt i zwróć go, np.

[PHP] pobierz, plaintext 
class Test {
 
	private $param = null;
 
	public function __construct($param) {
		$this->param = $param;
	}
 
}
 
function create_class_conditionally($name, $arg = null) {
	$return = null;
	// zamień warunek na sprawdzanie w bazie
	if($arg > 0) {
		$return = call_user_func_array(array(new ReflectionClass($name), "newInstance"), $arg);
	}
	return $return;
}
 
//print_r(create_class_conditionally("Test", 1));
print_r(create_class_conditionally("Test", -1));
 
if(is_null(create_class_conditionally("Test", -1))) {
	echo "null!";
}
[PHP] pobierz, plaintext 

[Crozin]

Hmm... a mnie nadal zastanawia dlaczego wszyscy tak boją się tego wyjątku...?