Хорошо ли использовать выражения if — else в PHP и не только?

Question

[romazhan]

У меня депрессия.
Выражения if - else - неотъемлемые конструкции языков программирования, но в интернете есть статьи осуждающие ветвление кода. Как быть?

Comments

[Владислав Лысков]

про какое именно ветвление речь?

[romazhan]

Владислав Лысков, if { if {} else {} } else {}

[Владислав Лысков]

polak228, можно пример поконкретнее?

[Ипатьев]

задавая такие дурацкие вопросы, надо сразу давать ссылку на "статьи"
без ссылки или хотя бы осмысленного пересказа своими словами вопрос под удаление

[sl0]

Как быть?

Использовать их с умом.

Answer 1

[Slava Rozhnev]

Если речь идет о большой вложенности ветвлений, то да такого лучше избегать.
Например конструкцию вида:

<?php
$a=17;

if ($a < 0 ) {
	if ($a % 2 === 1) {
		echo "$a is negative odd";
	} else {
		echo "$a is negative even";
	}	
} else {
	if ($a % 2 === 1) {
		echo "$a is positive odd";
	} else {
		echo "$a is positive even";
	}
}

можно упростить используя функции:

function positive_or_negative($a) {
	return ($a < 0 ) ? 'negative' : 'positive';
}

function odd_or_even($a) {
	return ($a % 2 === 1 ) ? 'odd' : 'even';
}

echo "$a is " 
		. positive_or_negative($a)
		. " "
		. odd_or_even($a);

Test PHP code online

Answer 2

[Vladislav Orlov]

Скорее тут дело в том, что бы не фигачить if-else на пустом месте, а попробовать для начала понять логику и сделать проще и читабельнее, ведь потом вам с этим кодом работать.

Пример когда один и тот же код можно написать по разному:
if-else
without if-else
Согласитесь, без if-else смотрится лучше. Иногда нужно просто включить голову

Comments

[Vladislav Orlov]

Или например часто бывает типо такого

$balance = 500;
$transactionAmount = 300;
$newBalance = $balance  - $transactionAmount;

if ($newBalance > 0) {
 //some logic with transfer between bills
} else {
 //ohh dude, you write wrong amount, please change your amount
}

-----------------
Вместо того что бы написать просто

if ($newBalance < 0) {
 //ohh dude, you write wrong amount, please change your amount
 return false;
}

//some logic with transfer between bills

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

[Adamos]

Vladislav Orlov, пример так себе. Return, рассыпанные по коду функции, ее читаемость не улучшают.

if($conditionOne) {
  if($conditionTwo = calcC2()) {
    if($conditionThree = calcC3()) {
      return doSomething();
    }
  }
}
return false;

Else в этом коде тоже вполне возможны, просто опущены для простоты.