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

[dzastin]

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

A to czasem nie jest php'owy fatal? Jeśli tak, to żeby to przechwycić musiałbyś zrobić swój error_handler

[Piotrbaz]

Ok, doszedłem do tego, że miałem w połączeniu z bazą ustawione zrzucanie wyjątków klasy PDOException, a nie mogę ich jawnie zrzucać.

Po usunięciu

[PHP] pobierz, plaintext 
setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION)
[PHP] pobierz, plaintext 

mogę normalnie zrzucać wyjątki Exception. Tylko nie bardzo rozumiem sens PDOException, skoro nie mogę wpisać tam własnego komunikatu.

Ten post edytował Piotrbaz 24.03.2013, 14:05:08

[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

[Crozin]

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

Chyba źle zrozumiałeś. Klucz typu UNIQUE ma nadal pozostać . Po prostu zamiast:

[PHP] pobierz, plaintext 
<?
 
try {
 // próba dodania rekordu i operowanie jakgdyby był on unikalny
} catch (... $e) {
 // sprawdzenie czy wyjątek dotyczy duplikatu
 // jeśli tak: zrób coś
 // jeśli nie: throw $e; // rethrow
}
 
Powinieneś zrobić
 
// próba pobrania dupliaktu
// jeśli nie ma: operujesz na unikalnym rekordzie
// jeśli jest: zrób coś z duplikatem
[PHP] pobierz, plaintext 

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

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

Wyjątki PDO mają sobą reprezentować jakieś problemy z samym połączeniem z bazą danych i właściwie tylko "wnętrzności" PDO powinny rzucać wyjątek tego typu. Twoje wyjątki nie mają z tym nic wspólnego.