Где я допустил ошибку?

Question

[amazinginternetsites]

Не ругайтесь, только изучаю...
Есть такой код, но запуская, выскакивает ошибка: Parse error: syntax error, unexpected '<'

<?php foreach($formula as $key=>$value) { ?>
	<?php
		$data = $item;
                    <?php if ($data['dopvopremgost'] == 2){ ?>
                                        $data['lr1P'] = ($data['lr1l']+$data['lr1w'])*2; 
					$data['lr1SF'] = $data['lr1l']*$data['lr1w']; 
					$data['lr1SWALL'] = $data['lr1P']*$data['lr1h'];
         <?php } ?>

          <?php if ($data['dopvopremgost'] == 3){ ?>
                                        $data['rr1P'] = ($data['rr1l']+$data['rr1w'])*2; 
					$data['rr1SF'] = $data['rr1l']*$data['rr1w']; 
					$data['rr1SWALL'] = $data['rr1P']*$data['rr1h'];
         <?php } ?>

         <?php if ($data['dopvopremgost'] == 4){ ?>
                                        $data['or1P'] = ($data['or1l']+$data['or1w'])*2; 
					$data['or1SF'] = $data['or1l']*$data['or1w']; 
					$data['or1SWALL'] = $data['or1P']*$data['or1h'];
         <?php } ?>
eval('$formula_res = '.$value.';');
					
				?>

<?php } ?>

Answer 1

[Котик Антон]

Очеловечу ваш код:

<?php 

foreach ($formula as $key => $value) {
    $data = $item;
    $prefixGroups = array(
        'dopyyyy' => array(2 => 'yy'),
        'dopvopremgost' => array(2 => 'lr', 3 => 'rr', 4 => 'or'),
        'dopdopvopsuvsid' => array(2 => 'zz'),
    );
    foreach (array_keys($prefixGroups) as $key => $prefixes) {
        if (array_key_exists($data[$key], $prefixes)) {
            $prefix = $prefixes[$data[$key]];
            $data[$prefix . '1P'] = ($data[$prefix . '1l'] + $data[$prefix . '1w']) * 2; 
            $data[$prefix . '1SF'] = $data[$prefix . '1l'] * $data[$prefix . '1w']; 
            $data[$prefix . '1SWALL'] = $data[$prefix . '1P'] * $data[$prefix . '1h'];
        }
    }
    eval('$formula_res = ' . $value . ';');     
}

Comments

[Alex Safonov]

Рефакторинг - блеск!
Но
eval('$formula_res = ' . $value . ';');
серьезно?

[amazinginternetsites]

очень красивый код. спасибо. Ради понимания конструкции.
Что если есть еще условия
<?php if ($data['dopyyyy'] == 2){ ?>
$data['yy1P'] = ($data['yy1l']+$data['yy1w'])*2;
$data['yy1SF'] = $data['yy1l']*$data['yy1w'];
$data['yy1SWALL'] = $data['yy1P']*$data['yy1h'];
<?php } ?>

<?php if ($data['dopdopvopsuvsid'] == 2){ ?>
$data['zz1P'] = ($data['zz1l']+$data['zz1w'])*2;
$data['zz1SF'] = $data['zz1l']*$data['zz1w'];
$data['zz1SWALL'] = $data['or1P']*$data['zz1h'];
<?php } ?>
Как в таком случае выглядит код?

[Котик Антон]

amazinginternetsites: Дополнил. В последнем условии в конце $data['zz1P'] должно быть вместо $data['or1P']?

[amazinginternetsites]

27cm: да-да, верно, опечатка. Спасибо вам, буду разбираться очень интересное решение

[amazinginternetsites]

27cm: я верно понимаю, что если совпадут несколько условий, то несколько результов выведется?
Т.е., например, если $data['dopdopvopsuvsid'] == 2
выведется
$data['zz1P'] = ($data['zz1l']+$data['zz1w'])*2;
$data['zz1SF'] = $data['zz1l']*$data['zz1w'];
$data['zz1SWALL'] = $data['or1P']*$data['zz1h'];

и если $data['dopvopremgost'] == 4
выведется и это
$data['or1P'] = ($data['or1l']+$data['or1w'])*2;
$data['or1SF'] = $data['or1l']*$data['or1w'];
$data['or1SWALL'] = $data['or1P']*$data['or1h'];

[amazinginternetsites]

27cm: Кстати, а что если действительно, как в поледнем условии, останется $data['or1P']
Как понимаю вместо префикса просто указать "or"
$data[$prefix . '1SWALL'] = $data[or . '1P'] * $data[$prefix . '1h'];

[Котик Антон]

amazinginternetsites: Да, если совпадут несколько условий, да, несколько результатов будет. Потому что там в цикле все вариаеты перебираются.

А вот если префикс где-то отличаться будет, то тут уже такого красивого решения не получится. Нужно будет или вернуться к тому виду, что был у вас, или что-то еще придумывать.

[amazinginternetsites]

27cm: мое решение составит не менее 15к строк )))

Answer 2

[Alexander Litvinenko]

<?php
$data = $item;
<?php

Comments

[amazinginternetsites]

т.е. вместо <?php должен быть просто if {} elseif {} elseif {} elseif {} elseif {} elseif {} elseif {} eval('$formula_res = '.$value.';'); так?

[Alexander Litvinenko]

1. Пхп открывает один раз, его ненадо открывать в кажой строчке
а вместо if {} elseif {} else.... лучше switch....case

[amazinginternetsites]

не покажете на моем примере?

[amazinginternetsites]

я пока изучаю, и все варианты очень интересны. Ваша помощь бесценна

[Alexander Litvinenko]

amazinginternetsites: не покажу, для развития ваших навыков, полезнее если вы сами найдете.

[Alexander Litvinenko]

amazinginternetsites: это очень базовые знания.

Answer 3

[tigra]

а в какой конкретно строчке ошибку показывает?
и используй лучше конструкцию не

<?php foreach($formula as $key=>$value) { ?>
.....
<?php } ?>

А вот так

<?php foreach($formula as $key=>$value) : ?>
.....
<?php endforeach; ?>

С конструкцией if так же. Со скобками гораздо проще запутаться, ну и менее читаемо.

Comments

[amazinginternetsites]

Ругается на строчку <?php if ($data['dopvopremgost'] == 2){ ?>