Цитата |
---|
Антон Долганин написал: Вот это неправильно: ЦитатаИзбегать лишних фигурных скобок если в условии/блоке одно выражение. Это ОЧЕНЬ мешает отладке. К примеру, код. И ты идешь с die, ища место ошибки. Доходишь до такого места:Кодif($strUserName) echo 'Привет ' . $strUserName;Как мне тут die вставить? Правильно, дописывать за программиста скобки. Поэтому они всегда должны быть. И отсюда следствие, что лучше писать не Кодif () { }а Кодif () { } |
Я обращал и на это внимание, но как-то редко получается такие условия исправлять, чаще всего пишешь там что-то короткое, типа if($name) echo $name, более сложные условия типа:
Код |
---|
if ($this->form->fields['users_id']->value)
$orderuser = Ordersuserinfo::GetRow(array('user_id' => $this->form->fields['users_id']->value, 'order_id' => $this->itemId));
else
$orderuser = $db->SelectRow("SEL ECT * FR OM `users_no_reg` WHERE `id` = '" . ($this->form->fields['users_no_reg_id']->value) . "';");
|
стараешься сразу заключать в скобки, т.к. 100% придется проверять, что вернул запрос в $orderuser, это уже как-то сознательно идет автоматом.
Кстати, форматтер PHPStorm умеет добавлять скобки ко всем сокращенным условиям автоматом, если так настроен, при желании достаточно нажать Ctrl + Alt + L
Еще с годами формируется навык чтения кода, открываешь, просматриваешь и все понимаешь, что тут происходит, мне очень удобно читать, когда скобки слева,
Если писать вот так, то я тут спотыкаюсь так, что аж падаю, не могу читать, смотрю как на грабли какие-то.
Еще бывает вот такая подстава))
А вот так уже лучше:
Но в многих шаблонах и зарубежных скриптах, например в OpenCart, везде так:
Код |
---|
<?php if ($reward) { ?>
<li><a href="<?php echo $reward; ?>"><?php echo $text_reward; ?></a></li>
<?php } ?> |
Если форматировать код, как мне нравится в php со скобками слева, тогда получится каша:
Код |
---|
<?php if ($reward)
{ ?>
<li><a href="<?php echo $reward; ?>"><?php echo $text_reward; ?></a></li>
<?php } ?> |
Одно удобство порождает другое неудобство, поэтому шаблоны мне нравится оформлять так:
Код |
---|
<? if ($reward): ?>
<li><a href="<?= $reward; ?>"><?php echo $text_reward; ?></a></li>
<? endif; ?>
|
В общем, еще надо будет подумать
Цитата |
---|
Антон Долганин написал: А вообще я не встречал еще ни одного проекта (ни своего, ни чужого), который выстрелил бы на правильном коде. Не потому что правильный код и проект не выстреливает, а вот садишься, все правильно пишешь, код лоснится прямо, а проект умирает. Постоянно! Если что-то было написано на коленке, то он живет и развивается, говно-код накапливается, и вот ты уже думаешь "как его рефакторить". Как-будто сглаз ООП-ный какой. |
Да, есть такое проклятье.
Еще бывает делаешь сайт с душой, как для себя, его либо не ценят, либо вообще забрасывают к чертям, делаешь как на коленке, так и говорят: сайт делаете, как на коленке)))