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.92
28.04.2008 · 44 каммента · рейтинг 9.51
23.01.2013 · 20 камментов · рейтинг 9
29.08.2007 · 28 камментов · рейтинг 7.34
19.01.2008 · 20 камментов · рейтинг 6.32

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

18.11.2017  JohnКак закрыть окно Skype / свернуть нафиг в трэй: Спасибо, добрый человек! Неимоверно раздражало.
10.11.2017  NikПочему не работает form.submit(): Автор, спасибо за статью 5-летней давности. Подари...
27.10.2017  ДенисОбработка кликов и цели в Яндекс.Метрике, улучшенный вариант: Блин. А для не в зуб ногой - можно код целиком выл...
17.10.2017  ДубрDenwer - настройка алиасов (ServerAlias): Ахуенно не это, ахуенно то что кто-то пользуется Д...
29.09.2017  Maxтам же: Спасибо, чувак! Это ахуенно, оно заработало!!!

Статсы