[PHP]Dwa wyjątki czy konieczne? Opcje

[Fred1485]

Witam znów

Postanowiłem jednak stworzyć tą nakładkę dla PDO, i mam taki problem:

[PHP] pobierz, plaintext 
 public function prepare($query = FALSE) {
 
        $this->query = $query;
 
        try {
 
            if($this->connection == FALSE || $this->query == FALSE) { //sprawdzam czy jest połączenie oraz czy przekazano query
 
                throw new Exception('blad', 1); //tutaj jakiś przykładowy błąd jeśli nie spełnione warunki
            }
            $this->stmt = $this->connection->prepare($this->query); //tu wiadomo chyba
            $this->stmt->execute(); //TUTAJ TYLKO CHWILOWO żeby sprawdzić
        }
        catch(PDOException $e) {
            $this->error(array('text' => $e->getMessage(), 'code' => $e->getCode())); //Przekazanie do funkcji (w niej następuje na razie proste echo)
 
        }
catch(Exception $e) {
            $this->error(array('text' => $e->getMessage(), 'code' => $e->getCode())); //jak wyżej
 
        }
    }
[PHP] pobierz, plaintext 

Jak widać tutaj wyłapuję dwa wyjątki, pierwszy od PDO jeśli w execute coś by nie poszło z zapytaniem(dlatego dałem tymczasowo w tej funkcji), natomiast drugi własny jeśli jakieś tam warunki nie zostaną spełnione, i tutaj pytanie czy ja dobrze myślę teraz z tym przechwytywaniem dwóch wyjątków, jest to poprawne, czy jakoś inaczej to powinienem rozwiązać, zauważyłem też że jak PDOException zamienię na Exception to wyjątek od PDO też zostanie wychwycony(niby logiczne wiadomo) i praktycznie wszystko to mogę rozwiązać tylko

[PHP] pobierz, plaintext 
catch(Exception $e) {
            $this->error(array('text' => $e->getMessage(), 'code' => $e->getCode())); //jak wyżej
 
[PHP] pobierz, plaintext 

Czy mam jakoś kombinować i tworzyć n wyjątków na różne potrzeby czy inaczej powinienem to rozwiązać?

[olszam]

Wywal drugiego catch-a i po sprawie a w miejsce pierwszego daj exception to wyłapie ci wszystko, jak chcesz mieć coś więcej niż kod z pdo. Nigdy się nie bawiłem by dawać n wątków chyba że kiedyś raz zrobiłem tak że jak wyłapie błąd to wykona inny wątek

[PHP] pobierz, plaintext 
try{
////// jakiś tam kod
}catch(Exception $e){
try{
/////jakiś tam inny kod
catch(Exception $e){
//// wyświetl błąd
     }
}
[PHP] pobierz, plaintext 

coś na podstawie tego kiedyś robiłem ale to w sumie nie był mój pomysł by tak robić tylko profesor tak chciał, a kiedy takiego przykładu używać to sam nie wiem.

Ten post edytował olszam 6.04.2015, 17:22:58

[Pyton_000]

Tutaj musisz sobie sam określić poziom łapanych wyjątków. Albo chcesz wszystkie i zostawiasz Excepion albo tylko PDO.
Łapanie kilka wyjątków ma sens tylko gdy robisz inne akcje dla nich np. NotFoundException będzie robiło redirect 404 gdzieś tam, ale już WrongTokenException będzie Ci waliło np. 400 i fatal error w pysk.

Reasumując, zostaw sobie Albo albo.

[Fred1485]

O akurat w tym przypadku czy błąd zwrócony przez pdo czy rzucony przeze mnie ma byc obsłużony tak samo wiec sobie zostawię tylko jeden blok catch. Dzięki koledzy

Ten post edytował Fred1485 6.04.2015, 17:39:57