- 1
- 2
- 3
- 4
- 5
if($record->getTitle()) {
$request->setAttribute('title', $record->getTitle());
}else{
$request->setAttribute('title', $record->getName());
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+145.5
if($record->getTitle()) {
$request->setAttribute('title', $record->getTitle());
}else{
$request->setAttribute('title', $record->getName());
}
определение существования метода у объекта $record
guest 18.09.2009 17:12 # +1
guest 18.09.2009 17:15 # +3
Аффтар поста, учи матчасть
Shock 19.09.2009 19:16 # +1
$request->setAttribute(\'title\', $title);
guest 19.09.2009 19:53 # 0
Имхо, не такой уж и говнокод.
WGH 19.09.2009 22:00 # 0
Shock 20.09.2009 13:48 # 0
но не критично — согласен. думаю, достаточно часто в рабочих проектах такое встречается.
if($record->getTitle()) {
$title = $record->getTitle();
}else{
$title = $record->getName();
}
$request->setAttribute('title', $title);
Shock 20.09.2009 13:49 # +1
$title = $record->getTitle();
if(!$title) {
$title = $record->getName();
}
$request->setAttribute('title', $title);
единственный вариант, при котором метод getTitle не вызывается дважды)
TWINc 20.09.2009 20:09 # +2
if (!($title = $record->getTitle()) {
$title = $record->getName();
}
...
Shock 26.09.2009 02:17 # 0
Bartelby 20.09.2009 10:47 # 0
0m3r 22.09.2009 17:50 # 0
guest 24.09.2009 16:36 # 0
guest 30.09.2009 19:36 # 0