Dubrowsky
Хроники одного дупла
Блогово  →  WebDev  → 

Блин, дети, никогда ТАК не делайте!

18 Июня 2007 года

if (!isset($something)) {
   // Нештатная ситуация
   return;
} else {
   // Работаем в штатном режиме
}

Блин, если используете выход из середины метода (что само по себе спорное решение), не надо хоть такую путаницу вносить! Использование else здесь лишено всякого смысла: если блок if содержит return, все, что следует за ним как бы автоматом попадает в else (т.е. этот код выполняется, если проверка if не сработала). Можно было бы сократить один уровень вложенности. Но это не самое страшное. Такая конструкция начисто сбивает с толку. Чтоб разобраться, почему не выполняется код после завершения блока else {}, может уйти немало времени. Из такой грязи и складывается неэффективность. Чтоб найти такую "ошибку", нужно вникать в суть кода, причем отбросив свои соображения насчет логики конструкции (т.к. на самом деле логика вовсе отсутствует). Надо каким-то местом почуять, что бойцы, писавшие код, могли сотворить ТАКОЕ, и это ТАКОЕ целенаправленно искать. Короче я зол.

Камменты

MiRaj20.06.2007, 11:06#
Где такое откопал? :)
Николай Дубровскийон самый20.06.2007, 11:57#
Кусок одного портальчега на аутсорсе писался, там и откопал. Там еще много интересного :)
MiRaj20.06.2007, 12:07#
Ужос. Школьники атакуют..
krolikr25.06.2007, 21:59#
здравствуйте,
извините за оффтоп, какой у вас плагин для использования openid в комментариях?
Николай Дубровскийон самый26.06.2007, 14:08#
Плагин вот этот: http://the-notebook.org/12/01/2006/openid-comments-for-wordpress/
Dmitr18.07.2007, 11:30#
В программинге знаю только минимум основ, но вначале подумал что запись относится не к программингу, а к ситуации повседневного характера...

Написать коммент: памятка постеру

 

Крутые посты wtf??? →

15.02.2013 · 24 каммента · рейтинг 9.73
23.01.2013 · 20 камментов · рейтинг 8.83
06.03.2008 · 29 камментов · рейтинг 7.59
30.01.2013 · 13 камментов · рейтинг 7.13
28.03.2008 · 23 каммента · рейтинг 6.78

Последне камменты

16.02.2018  CrazyMiheyКак изменить оболочку (shell) по умолчанию для пользователя Linux: Ну и сколько Страждущих ломанулись подбирать Парол...
28.01.2018  Привет Ера Связь: 04acw1pf8fwgmqhpfp22t6398o4zszhe3x9lj7luwizkodzt7...
26.01.2018  JasonСвязной.Банк закрыл возможность пополнения WebMoney: Hello Здравствуйте. Сервис продвижения в социальны...
24.01.2018  Annaтам же: Hi Здравствуйте. Сервис продвижения в социальных с...
14.12.2017  БраузерНовый Яндекс.Браузер - браузер "Yandex" (отзыв / обзор): Я предпочитаю использовать яндекс браузер

Статсы