Как правильно продумывать логику условных операторов ветвления «if»?

Question

[Евгений Федоров]

Ребят я никак не могу научиться с первого и даже с 3 раза рисовать у себя в голове схему сложных условных операторов. Я знаю как себя должен вести код но почему то у меня всегда затык на логике с операторами которых несколько, в группах и с реверсными значением условного выражения "!". Я замечаю что на этих этапах особо сильно торможу =) Может кто подскажет метод, благодаря которому все станет проще, метод который позволит легко визуализировать поведение, и избавит от тормозов на этом этапе разработки? =)
Банальный простой пример из php

if(!$this->data['status']) {
    //code
}

выдает ошибку в случае когда индекса"status" нету "Undefined index:". Переделываю в

if (isset($this->data['status']) && !$this->data['status']) {
    //code
}

и получаю не корректную логику. Пришлось лезть за блокнотиком... уже решил но потратил на это лишнее время.
И так со мной всегда =)
Может кто то уже сам создал свой простой шаблон и готов им поделиться =)

Comments

[AlikDex]

$this->data['status']
замените это говно на вменяемое
$this->status

[Евгений Федоров]

AlikDex: Это не мой код =) Увы тут все в индусских руках =)

[AlikDex]

Евгений Федоров: в лучших традициях говнокода тогда будет подавлять варнинги собакой @ :D

if (! @$this->data['status']) {
    //code
}

[Евгений Федоров]

AlikDex: Спасибо, не знал =)

Answer 1

[Night]

Я немного фигею от местных ответов :)
Один советует нахрен сменить язык, второй говорит про экзепшены, которые не существуют в пыхе, третий предлагает перейти на TDD... no comments.

И только лишь один, вменяемый fshp, сказал, в чем у человек проблема - array_key_exits (applaus)

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

Comments

[Евгений Федоров]

Ага, присоеденяюсь =) Только iseet работает быстрее. stackoverflow.com/a/3210982/2309309

[Night]

Евгений Федоров: да, но и что? он не может заменить array_key_exists, isset некорректно обрабатывает ключи в массиве с null

[Евгений Федоров]

Night: В моем случае если ключ есть и он null то это false. В конце концов я исправил на

(isset(!$this->data['status']) || !$this->data['status'])

[Максим Мосейчук]

Night: корректно он обрабатывает. Но по другому.

[Евгений Федоров]

fshp: Я к тому что логика не страдает от isset в данном случае. Если я ничего не путаю.

[Максим Мосейчук]

Евгений Федоров: "Переделываю в ... и получаю не корректную логику.". Это ваши слова.

[Евгений Федоров]

fshp: Так это и за того что мне сложно рисовать ветвление в голове =) А не из оператора =) Я же пост создал не для того что бы хаяли гавно код а что бы помогли с визуализацией кода =)

Answer 2

[Максим Мосейчук]

А причём тут if? Вы изначально используете не те функции, что требуется. Соответственно, результат не правильный.

1) Используйте array_key_exists вместо isset. isset по другому работает с null
2) В PHP есть гарантия того, что левый операнд оператора && выполнится первым всегда? Если нет, то опять же вы можете словить ошибку "Undefined index".

Comments

[Евгений Федоров]

да, сначала левый, и iseet работает быстрее. stackoverflow.com/a/3210982/2309309

Answer 3

[Curly Brace]

тут дело не в if а в продумывании наперед.

обычно программисты продумывают все возможные и невезможные варианты входных данных, а вы просто слишком легкомысленно относитесь к логике и if тут не при чем, так можно и null reference exception (хз есть такое в php или нет) получить).

правда есть обратная сторона, когда особо дотошные, код загромождают кучей проверок на случай сценариев, которые могут произойти только если сотона аццкая поработит этот мир)))

Comments

[Neonoviiwolf]

т.е. user будет пользоваться этой программой