- 1
- 2
- 3
- 4
- 5
- 6
- 7
public function isOverflowed(){
if(is_null($this->isOverflow)){
$this->isOverflow = ($this->getMemberCount() > $this->getMaxMembers());
}
return $this->isOverflow === true;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−23
public function isOverflowed(){
if(is_null($this->isOverflow)){
$this->isOverflow = ($this->getMemberCount() > $this->getMaxMembers());
}
return $this->isOverflow === true;
}
Придадим значимости результату.
вот как это делают программисты
https://habrahabr.ru/post/159099/
Первоначально же я хотел поделиться финтом с результатом.
$this->isOverflow = ($this->getMemberCount() > $this->getMaxMembers()) однозначно даст bool
а затем
return $this->isOverflow === true;
Это да, но вдруг в $this->isOverflow какой-то другой метод сунул не бул?
Нищеброд?