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 камментов · рейтинг 11.94
15.02.2013 · 23 каммента · рейтинг 10.05
23.01.2013 · 20 камментов · рейтинг 9.31
29.08.2007 · 28 камментов · рейтинг 7.46
28.03.2008 · 23 каммента · рейтинг 6.96

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

22.07.2017  СергейПочему не работает form.submit(): Также столкнулся с неработающим .submit() при отпр...
20.07.2017  СергейСвязь: Здравствуйте. Хочу предложить Вам тройной обмен п...
17.07.2017  Илья АрхипкинСколько журналистов в России?: Мне рассказывали как журналисты освещали акцию в К...
21.05.2017  Vadim GukОбработка кликов и цели в Яндекс.Метрике, улучшенный вариант: Большое спасибо за решение задачи.
20.05.2017  ДмитрийТестовое задание для PHP-программиста (Junior): А джуны ещё нужны у вас в компании?

Статсы