Zrzucanie wyjątku w modelu z PDO Opcje

[Piotrbaz]

Witajcie,

Mam problem ze zrzuceniem wyjątku. W modelu mam funkcję save, oto jej ciało:

[PHP] pobierz, plaintext 
public function save($task, advertisement $advert, $id = null) {
 
(...)
          if ($task == "insert") {
               $result = $this->db->prepare("INSERT INTO ads_advertisement 
                                                                  VALUES('',:title, :description, :content, :miniature, :date, :type_id, :category_id)");
 
          }
 
          $result->bindValue(':title', $advert->getProperty('title'), PDO::PARAM_STR);
//reszta wywołań bindValue (...)
 
 
          $result->execute();
 
     }
[PHP] pobierz, plaintext 

Chcę zrzucić wyjątek, gdy podczas zapisywania wystąpi naruszenie unikalności jakiegoś pola w tabeli, np title.
No i nie wiem gdzie wrzucić throw new Exception('wiadomosc'); Próbowałem sprawdzenie, czy $result->execute() zwróci coś innego niż true, ale nie działa. Wciąż przy wyłapywaniu wyjątku dostaję wiadomość, jakbym w ogóle throw new Exception nie umieśił w Modelu.

SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'mmmmmmmmmmm' for key 'title'

Kod w kontrolerze:

[PHP] pobierz, plaintext 
    try {
 
$this->model->save("insert", $new_advert);
 
 } 
 catch (Exception $e) {
 
$this->view->set_Exception($e->getMessage());
 
  }
[PHP] pobierz, plaintext 

W widoku, wiadomo, wyświetlam komunikat wyjątku.

edit:

Teraz już nie wiem, czy w ogóle mogę własny komunikat wysłać, w manualu znalazłem, że nie powinno się jawnie zrzucać wyjątków klasy PDOException
Ale nie wychodziło nawet z Exception.

Ten post edytował Piotrbaz 23.03.2013, 09:13:41

[Crozin]

1. Powinieneś samodzielnie sprawdzać czy w bazie nie istnieje już dany rekord (unikalna wartość), a nie oczekiwać na wyjątek.
2.

Tylko nie bardzo rozumiem sens PDOException, skoro nie mogę wpisać tam własnego komunikatu.

Co to znaczy wrzucić własny komunikat? Jeżeli chcesz jakoś rozbudować zawartość rzucanego wyjątku przechwyć rzucony wyjątek PDO i "opakuj go" we własny wyjątek:

[PHP] pobierz, plaintext 
try {
  ...
} catch (PDOException $e) {
  throw new RuntimeException('Blah blah blah', 0, $e);
}
[PHP] pobierz, plaintext 

[Piotrbaz]

1. Powinieneś samodzielnie sprawdzać czy w bazie nie istnieje już dany rekord (unikalna wartość), a nie oczekiwać na wyjątek.

Rozwiązanie z UNIQUE jest prostsze, ale skoro to napisałeś, znaczy że gorsze. Dlaczego ?

2. Co to znaczy wrzucić własny komunikat?

Myślałem, że mogę po prostu zrobić jak ze zwykłą klasą Exception

[PHP] pobierz, plaintext 
throw new PDOException("Blah Blah");
[PHP] pobierz, plaintext 

Ten post edytował Piotrbaz 24.03.2013, 19:22:26