- 1
- 2
- 3
- 4
$this->template_model = BaseCommunicationType::model()->findByAttributes(array('template' => $this->template));
if(!$this->template_model === NULL) {
throw new Exception("Template name is wrong", 1);
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+154
$this->template_model = BaseCommunicationType::model()->findByAttributes(array('template' => $this->template));
if(!$this->template_model === NULL) {
throw new Exception("Template name is wrong", 1);
}
Угадайте, что будет из исключением?
Так как примерно никто в мире не умеет правильно использовать эксепшены, то так почти никтоне делает
* Тех, которые часть контракта и которые нужно ловить (checked exceptions, в терминологии Java)
* Тех, которые символизируют НАРУШЕНИЕ контракта, и ловить которые не нужно, а нужно дать им долететь до верху стека и сдохнуть (рантайм в терминологии джавы)
В первом случае нам нужно наследоваться, во втором -- нет.
Сравните (синтаксис пыха плохо помню, но кажется так):
Ничего.