- 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);
}
Угадайте, что будет из исключением?
istem 07.11.2014 00:21 # 0
Lure Of Chaos 07.11.2014 11:17 # 0
guest 07.11.2014 13:25 # 0
Анонимус 07.11.2014 16:59 # +2
Анонимус 07.11.2014 16:59 # 0
Так как примерно никто в мире не умеет правильно использовать эксепшены, то так почти никтоне делает
guest 07.11.2014 17:46 # 0
Анонимус 07.11.2014 17:53 # +2
* Тех, которые часть контракта и которые нужно ловить (checked exceptions, в терминологии Java)
* Тех, которые символизируют НАРУШЕНИЕ контракта, и ловить которые не нужно, а нужно дать им долететь до верху стека и сдохнуть (рантайм в терминологии джавы)
В первом случае нам нужно наследоваться, во втором -- нет.
Сравните (синтаксис пыха плохо помню, но кажется так):
guest 07.11.2014 17:43 # +1
guest 07.11.2014 13:20 # 0
Ничего.
guest 07.11.2014 17:44 # +1