2012-02-16 28 views
0

私は、従業員がオフィスにいるかどうかを示す必要があります。以下のコードは動作しますが、従業員がオフィスにいるかどうかを示すために1つの画像しか表示しません。現時点では、例えば、従業員が自分の出席者にam_inをスタンプし、次にam_outを彼のステータスにスタンプすると、2つの画像が表示されます。状態を表示する(PHP)PHP MySQL

if ($row_io['am_in'] == !NULL || $row_io['pm_in'] == !NULL) { 
    echo '<img src="../images/in.png">'; 
    } 
    else ($row_io['am_out'] == !NULL || $row_io['pm_out'] == !NULL) { 
     echo '<img src="../images/out.png">'; 
     } 
+0

http://codepad.viper-7.com/KMdYdH – hakre

答えて

1

あなたは出力から決定ロジックを分離しなければならないので、デバッグが簡単です:

$isEmployeeIn = isEmployeeIn($row_io); 
printf('<img src="../images/%s.png>', $isEmployeeIn ? 'in' : 'out'); 

あなたはその後、isEmployeeInにロジックのために必要なものを入れて、分離出力が世話をするあなただけの表示一度に1つの画像。ところで

$row_io['am_in'] == !NULL || $row_io['pm_in'] == !NULL 

は同等です:

$row_io['am_in'] || $row_io['pm_in'] 

あなたは多くのミスということはできませんので、最小限にコードを減らすようにしてください。

+0

+1、可能であればロジックとテンプレート用に別々のファイルがあります。 – halfer

関連する問題