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