Зачем нужно всегда ставить фигурные скобки: { и }?
1,00
р.
Я не понимаю, зачем все говорят, что нужно всегда использовать { и }. Вот, например, код:
if(1>2) { test = true }
Зачем тут писать скобки? Многие говорят, что для читаемости, но любой программист знает о таком написании, зачем же увеличить код?
Также лично я не понимаю зачем открытие скобки писать на новой строке, чтобы занять ещё одну лишнюю строку?
На мой взгляд это неправильно, возможно вы меня сможете в этом переубедить?
Лично мне намного быстрее и удобнее разобрать вот такой код:
if ($a === $b) bar() elseif($a > $b) $foo->bar($arg1) else BazClass::bar($arg2, $arg3)
Чем:
if ($a === $b) { bar() } elseif ($a > $b) { $foo->bar($arg1) } else { BazClass::bar($arg2, $arg3) }
Ответ
Практика всегда ставить скобки избавляет вас от досадных глупых ошибок.
Страшилка 1. Жил-был метод с guard condition:
if(somethingBad()) return
В один прекрасный день вы собираетесь домой, вас просят быстренько добавить логгирование туда. Ткнули по кнопкам (может быть даже по хоткею для live template), закоммитили:
if(somethingBad()) log.error("Something bad happened") return
И ушли. А кто-то потом тратит пару часов, чтобы найти эту невзрачную ошибку.
Страшилка 2. Жил был код:
if (reallyRareCondition()) handleIt() doImportantThings()
Однажды джуниора попросили временно убрать обработку Очень Редкого Случая. Он смело закомментировал соответствующую строку и ушел играть в кикер с коллегами.
if (reallyRareCondition()) // handleIt() doImportantThings()
А потом вдруг Что-то Очень Важное перестало работать кроме редких случаев.
PS. Все совпадения не случайны, а действующие лица не выдуманы.
if(1>2) { test = true }
Зачем тут писать скобки? Многие говорят, что для читаемости, но любой программист знает о таком написании, зачем же увеличить код?
Также лично я не понимаю зачем открытие скобки писать на новой строке, чтобы занять ещё одну лишнюю строку?
На мой взгляд это неправильно, возможно вы меня сможете в этом переубедить?
Лично мне намного быстрее и удобнее разобрать вот такой код:
if ($a === $b) bar() elseif($a > $b) $foo->bar($arg1) else BazClass::bar($arg2, $arg3)
Чем:
if ($a === $b) { bar() } elseif ($a > $b) { $foo->bar($arg1) } else { BazClass::bar($arg2, $arg3) }
Ответ
Практика всегда ставить скобки избавляет вас от досадных глупых ошибок.
Страшилка 1. Жил-был метод с guard condition:
if(somethingBad()) return
В один прекрасный день вы собираетесь домой, вас просят быстренько добавить логгирование туда. Ткнули по кнопкам (может быть даже по хоткею для live template), закоммитили:
if(somethingBad()) log.error("Something bad happened") return
И ушли. А кто-то потом тратит пару часов, чтобы найти эту невзрачную ошибку.
Страшилка 2. Жил был код:
if (reallyRareCondition()) handleIt() doImportantThings()
Однажды джуниора попросили временно убрать обработку Очень Редкого Случая. Он смело закомментировал соответствующую строку и ушел играть в кикер с коллегами.
if (reallyRareCondition()) // handleIt() doImportantThings()
А потом вдруг Что-то Очень Важное перестало работать кроме редких случаев.
PS. Все совпадения не случайны, а действующие лица не выдуманы.
