LINUX.ORG.RU
ФорумTalks

Бзик — проверять условие на TRUE

 , ,


0

1

ЛОР, а у меня одного такой бзик, проверять что условие всегда TRUE? Вот терпеть ненавижу ставить if NOT ..., if (!something)... if (!file_exists()) { header(404); exit; }

Не люблю, не знаю почему. Аж передергивает от такого. Обязательно проверяю чтобы условие было «верным»: if (file_exists()) { } else { header(404); }

Я один такой?

А теперь про Новый год.

Сидят мужчины за столом, почевать собираются. От ароматов еды все уже язык проглотили и молча ждут...

Приходит дурнораскрашенная женщина и ядовитым запахом духов отбивает всем аппетит.

Тру стори.

Почему бабы такие... бабы?

Ни грамма спиртного. Весь вечер потягивал апельсиновый сок, а потом пошёл спать. Проснулся с лёгкостью в шесть утра, и сразу за работу, код писать...

Я считаю, праздник удался! =)

★★★★★

я-то думал, что моя жизнь - говно.

Deleted
()

вроде вопрос про программирование, но ты умудрился ввернуть сюда девушек, свою работу охранником и прочие сопли. как ты это делаешь?

Black_Roland ★★★★
()

рекомендую еще любую строчку заключать в конструкцию, для успокоения if (TRUE) { ... }

anonymous_sapiens ★★★★★
()
Последнее исправление: anonymous_sapiens (всего исправлений: 1)

Проверять надо то условие, которое чаще всего имеет место быть. Потому что если функцию вызывать в цикле, а она в первую очередь проверяет условие, которое раз в пятилетку срабатывает, условно говоря — нафик надо такое?

Virtuos86 ★★★★★
()

Ты не один, я тоже стараюсь так делать. Это упрощает чтение кода.

Deleted
()

Ты просто не любишь говорить «нет». Это психологический эффект у многих людей

four_str_sam
()

Забавно) У самого похожая хрень с условиями (проверка всего и вся), ну а насчет тян...

DeadEye ★★★★★
()

Сильный независимый Спуфинг

buddhist ★★★★★
()
Ответ на: комментарий от sudo

А, действительно, я невнимательный, показалось, что это он собирался почивать опосле застолья.

Chaser_Andrey ★★★★★
()
Ответ на: комментарий от i-rinat

Если если после после, значит после перед если.
Если после после значит, значит значит перед после.
Если перед перед после, значит после после перед.
Если значит после если, значит если перед значит.

alex_the_v ★★★
()

Приходит дурнораскрашенная женщина и ядовитым запахом духов отбивает всем аппетит

Иначе она отбила бы аппетит запахом пота.

crowbar
()

Лучше размещать первой короткую ветвь. Или даже вообще выходить из подпрограммы, чтобы не было большой вложенности.

Например

// Короткая ветвь впереди
if (!file_exists()) {
  // Сообщение об ошибке
} else {
  // Какая-то деятельность с файлом. Больший блок кода
  ...
  ...
  ...
}

// Условие прекращает обработку
if (!file_exists()) {
  return false;
}

// Какая-то деятельность с файлом. Большой блок кода
...
...
...

Такой код проще читать.

UPD: http://www.gamedev.ru/projects/forum/?id=160897

ziemin ★★
()
Последнее исправление: ziemin (всего исправлений: 1)
Ответ на: комментарий от KRoN73

operator ==

Да не, тут как кому нравится... if(a) и if (a == true) - на вкус и цвет :)

А как у переменной типа bool может быть значение не true и не false? Вот что меня удивило. Последний else это и проверяет, собственно.

veco
()
Ответ на: комментарий от unt1tled

Можно привести синтаксис в соответствие - синтаксис вообще г..но, после третьего языка на него не обращаешь внимание. А вот с трехзначной логикой, где есть TRUE, FALSE и NULL, уже просто так не спрячешь.

no-dashi ★★★★★
()
Ответ на: комментарий от Eddy_Em

играли в настолку Twilight Imperium всю ночь, до 7 утра. Если бы не пришедшие посреди ночи гости, забыли бы о спиртном совсем :)

stevejobs ★★★★☆
()

Сидят мужчины за столом, почевать собираются.

почевать

Что-что делать собираются?

ptarh ★★★★★
()

1с вас всех побери! Такой праздник испортили

der_looser ★★
()

Сравни:

if(file_exists($fname)) {
    $content = file_get_contents($fname);
    if($content) {
        // что-то делаем с контентом
        ...
        if($content_test_result) {
            // еще что-то делаем
            ...
            // еще 100500 уровней вложенности
                        ...
                        return true;
        } else {
            // пнх1
        }
    } else {
        // пнх2
    }
} else {
    // пнх3
}
И:
if(!file_exists($fname)) {
    // пнх1
}
$content = file_get_contents($fname);
if(empty($content)) {
    // пнх2
}
// что-то делаем с контентом
...
if(!$content_test_result) {
    // пнх3
}
...
return true;

drull ★☆☆☆
()

Не люблю if not, люблю unless.

iVS ★★★★★
()

if (file_exists()) { } else { header(404); }

Ппц... Не преумножайте сущности без надобности, пожалейте тех, кто случайно увидит этот код.

iVS ★★★★★
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.