Как правильно оформить условие?

Question

[amazinginternetsites]

Есть код

<?php foreach($formula as $key=>$value) { ?>
	<?php
		$data = $item;
                        if ($data['dopvopremgost'] == 2){ 
					
					$data['lr1P'] = ($data['lr1l']+$data['lr1w'])*2; //Периметр
					$data['lr1SF'] = $data['lr1l']*$data['lr1w']; //Площадь пола
					$data['lr1SWALL'] = $data['lr1P']*$data['lr1h']; // Площадь стен

			                                                    }
					
                        elseif ($data['dopvopsuvsid'] == 2 || 3 || 4 || 5){ 				
					
					
					$data['sp1P'] = ($data['sp1l']+$data['sp1w'])*2; //Периметр
					$data['sp1SF'] = $data['sp1l']*$data['sp1w']; //Площадь пола
					$data['sp1SWALL'] = $data['sp1P']*$data['sp1h']; // Площадь стен
					
                                                                                             } 
                         elseif ($data['dopvopsuvsid'] == 3 || 4 || 5){ 
					
					$data['sp2P'] = ($data['sp2l']+$data['sp2w'])*2; //Периметр
					$data['sp2SF'] = $data['sp2l']*$data['sp2w']; //Площадь пола
					$data['sp2SWALL'] = $data['sp2P']*$data['sp2h']; // Площадь стен

                                                                                             } 

					
					eval('$formula_res = '.$value.';');
					
				?>
                     <?php } ?>

Проблема в том, что почему-то срабатывает одно условие.
Например если все условия удовлетворены выполняется только первое.

Не могу разобраться. Помогите в контексте данного кода

Заранее спасибо

Answer 1

[Евгений]

А чем у вас elseif вообще отличаются? Тем, что во втором нет 2?

Comments

[amazinginternetsites]

Дело в том, что первый блок кода должен работать при $data['dopvopremgost'] == 2
при этом если $data['dopvopsuvsid'] == 2 || 3 || 4 || 5 то второй блок тоже работает и т.д.

[Евгений]

amazinginternetsites: А что вам вообще надо? какую логику работы вы ждете?

Выражение elseif выполнится, если предшествующее выражение if и предшествующие выражения elseif эквивалентны FALSE, а текущий elseif равен TRUE.

[Евгений]

Не знаю как у вас там должно, но может сделаете второе и третье условия иначе.

к примеру if $data['dopvopsuvsid'] !=2 {
}
if($data['dopvopsuvsid'] ==2{
}

[amazinginternetsites]

Евгений: мне нужно, чтобы проверки были независимыми.

[amazinginternetsites]

$data['dopvopsuvsid'] == 2 || 3 || 4 || 5 можно заменить на $data['dopvopsuvsid'] != 1 - спасибо

Но есть 2 условия
$data['dopvopsuvsid'] == 3 || 4 || 5
$data['dopvopsuvsid'] == 4 || 5

Как быть с ними?

[Евгений]

тогда пишите везде if и полную проверку, как посоветовал Алексей.
if ($data['dopvopsuvsid'] == 2 || $data['dopvopsuvsid'] ==3|| $data['dopvopsuvsid'] ==4 || $data['dopvopsuvsid'] ==5){
}

[amazinginternetsites]

Евгений: if ... {....} if ... {....} if ... {....} Такую конструкцию можно использовать?

[Евгений]

А вообще, я не могу понять, чем вам elseif отличаются, какую логику вы в них хотите вложить.

[Евгений]

amazinginternetsites: Если ваши условия независимы то только так и надо, использовать if. Если, если, если. А если они зависимы тогда, если и иначе если, иначе. (if, elseif, else)

[Евгений]

amazinginternetsites: @amazinginternetsites
$data['dopvopsuvsid'] == 2 || 3 || 4 || 5 можно заменить на $data['dopvopsuvsid'] != 1 - спасибо

Но есть 2 условия
$data['dopvopsuvsid'] == 3 || 4 || 5
$data['dopvopsuvsid'] == 4 || 5

Как быть с ними?

Я бы попробовал так

if($data['dopvopsuvsid'] != 1){}
if($data['dopvopsuvsid'] != 1 || $data['dopvopsuvsid'] != 2){}
if($data['dopvopsuvsid'] == 4 || $data['dopvopsuvsid'] == 5){}

[amazinginternetsites]

Есть выпадающий список который передает значения $data['dopvopsuvsid']
Если он передает значения от 2 до 5, выполняется код
$data['lr1P'] = ($data['lr1l']+$data['lr1w'])*2;
$data['lr1SF'] = $data['lr1l']*$data['lr1w'];
$data['lr1SWALL'] = $data['lr1P']*$data['lr1h'];

при этом если он передает значения от 3 до 5, выполняется еще один код
$data['lr2P'] = ($data['lr2l']+$data['lr2w'])*2;
...
...
при этом если он передает значения 4 или 5, выполняется еще один код
$data['lr3P'] = ($data['lr3l']+$data['lr3w'])*2;
...
...
и если он передает значения 5, выполняется еще один код
$data['lr4P'] = ($data['lr4l']+$data['lr4w'])*2;
...
...

[amazinginternetsites]

я понимаю. говнокод еще тот. но надо временно сделать решение.

[Евгений]

amazinginternetsites: ну тогда я бы пробовал так
if($data['dopvopsuvsid'] != 1){
$data['lr1P'] = ($data['lr1l']+$data['lr1w'])*2;
$data['lr1SF'] = $data['lr1l']*$data['lr1w'];
$data['lr1SWALL'] = $data['lr1P']*$data['lr1h'];
}
if($data['dopvopsuvsid'] != 1 || $data['dopvopsuvsid'] != 2){
$data['lr2P'] = ($data['lr2l']+$data['lr2w'])*2;
...
...
}
if($data['dopvopsuvsid'] == 4 || $data['dopvopsuvsid'] == 5){
$data['lr3P'] = ($data['lr3l']+$data['lr3w'])*2;
...
...
}
if($data['dopvopsuvsid'] == 5){
$data['lr4P'] = ($data['lr4l']+$data['lr4w'])*2;
}

[Евгений]

А вообще, если у вас там численные значения, то можно просто писать
>1
>2
>3
>4

[amazinginternetsites]

Евгений: вот оно! то что нужно. сейчас опробую

[amazinginternetsites]

Евгений: помогло. спасибо

Answer 2

[Алексей Уколов]

Во-первых, если вам нужно, чтобы проверки были независимыми, сделайте их независимыми. Сейчас ваш код читается как "если 1 то, иначе если 2 то, иначе если 3 то". Иными словами, уберите из логики "иначе".

Во-вторых, это хрень полная: $data['dopvopsuvsid'] == 3 || 4 || 5. Это условие всегда будет срабатывать, потому что 4 и 5 всегда будут приводиться к true. Либо пишите полное сравнение каждый раз, либо используйте in_array.

Comments

[amazinginternetsites]

Насчет 4 и 5.
У меня логика такая. ($data['dopvopsuvsid'] == 3 || 4 || 5) т.е. должно срабатывать и при 3, и при 4, и при 5. Есть такие, которые срабатывают если == 2 или ==4.

Независимыми это так <?php if ($data['dopvopremgost'] == 2){ ?> ... <?php } ?>

[Алексей Уколов]

Я понимаю, какая у вас логика, но она записана некорректно. Это условие будет выполняться всегда, потому что 4 и 5 ни с чем не сравниваются и просто приводятся к true.

[amazinginternetsites]

Что же делать? нужно быстрое временное решение. Не подскажете?

[Алексей Уколов]

Я в ответе уже всё написал.

Answer 3

[Wol_fi]

Проблема в том, что почему-то срабатывает одно условие.
Например если все условия удовлетворены выполняется только первое.

Простите, а что вы ожидаете от elseif (ИНАЧЕ ЕСЛИ)?
Ознакомьтесь
php.net/manual/ru/control-structures.if.php
php.net/manual/ru/control-structures.else.php
php.net/manual/ru/control-structures.elseif.php

Comments

[amazinginternetsites]

А что посоветуете использовать?

[Wol_fi]

amazinginternetsites: переделать с учётом логических конструкций и с правильным синтаксисом, ссылки на мануалы выше давал. Ну а если вам срочно надо, и вникать нет времени - обратитесь на фриланс биржи.