Реализация условия

[Parallelogram]

Дело такое, необходимо выполнить условие: если $res0 < $zap0 или $res1 > $zap1, то выполняется проверка на флаг. Если $flag = 1, то отправляем письмо ( с темой $subject0 / $subject1 соответственно) и изменяем флаг на 0. Во всех остальных случаях условие не должно выполняться.
Как это сделал я:

PHP код:

if($res0 < $zap0 ) 
    { 
    if ($flag == 1) 
        { 
        mail($to, $subject0, $message, $ot);
        mysql_query('UPDATE `tablica` SET flag = 0 WHERE id = 1');
        } 
    else {echo "flag_0 error";} 
    } 
else {echo "No_0";} 
 
if($res1 > $zap1 ) 
    { 
    if ($flag == 1) 
        { 
        mail($to, $subject1, $message, $ot);
        mysql_query('UPDATE `tablica` SET flag = 0 WHERE id = 1');
        } 
    else {echo "flag_1 error";} 
    } 
else {echo "No_1";} 


Как мне посоветовали:

PHP код:

if (  (($res0 < $zap0) || ($res1 > $zap)) && ($flag == 1)   ) 
{
        mail($to, $subject0, $message, $ot);
        mysql_query('UPDATE `tablica` SET flag = 0 WHERE id = 1');
} 
else {
        echo "error";
      } 


но необходимо, чтобы $subject0 и $subject1 соответствовали каждый своему условию.

[pompiduskus]

Второй вариант более правильнее. Он короток и все сразу понятно что к чему.

И еще на счет второго варианта, можно записать с меньшим количеством вот этих штук "( )" Забыл как они называются

PHP код:

if (($res0 < $zap0 || $res1 > $zap) && $flag == 1){

} 


По человечески это звучало бы там.

"ЕСЛИ" (( "тут парвда" "ИЛИ" "тут парвда" ) "И" "тут парвда"){

}

Но тогда не выходит с отправить нужное письмо. Поэтому можно еще вот так изменить твой вариант.

PHP код:


//сразу проверям флаг так как он один на двоих

if($flag == 1){
    if($res0 < $zap0){

         // отправляем пиьмо "0"
         mail($to, $subject0, $message, $ot);
         mysql_query('UPDATE `tablica` SET flag = 0 WHERE id = 1');

    }else if($res1 > $zap1){

         // отправляем пиьмо "1"
         mail($to, $subject1, $message, $ot);        
         mysql_query('UPDATE `tablica` SET flag = 0 WHERE id = 1');
    }else { echo " False;" }

} 


[ADSoft]

Всегда следует отказываться от одинаковых кусков кода.

Лучше сюда добавить условие для темы

if (($res0 < $zap0 || $res1 > $zap) && $flag == 1){
If ($res0 < $zap0) $subject0='tema1' else $subject0='tema2'
}

[pompiduskus]

Вооо это вообще шикарно. А я награмоздил