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??? →

27.06.2012 · 37 камментов · рейтинг 12.31
15.02.2013 · 22 каммента · рейтинг 10.19
30.01.2013 · 13 камментов · рейтинг 7.79
29.08.2007 · 28 камментов · рейтинг 7.57
20.03.2014 · 7 камментов · рейтинг 6.68

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

18.04.2017  ДубрОбработка кликов и цели в Яндекс.Метрике, улучшенный вариант: Вячеслав, блин, я там скобку потерял =) Обновил, ...
17.04.2017  Вячеславтам же: вот http://www.totalat.ru/service/remont-akpp-bmw/
17.04.2017  Вячеславтам же: Вашу версию вставил. Даже alert проверочный вывод...
17.04.2017  Вячеславтам же: То что вы там запостили это вместо последнего куск...
17.04.2017  Дубртам же: Привет! 1) код лучше запостить куда-нибудь на ht...

Статсы