PHPを利用し、複数の項目が特定の値の場合に一覧表示で背景色を変更したい

###前提・実現したいこと

PHPにてテーブル一覧を作成しており、ループで回して一覧表示後に特定の値を持つデータを複数抽出し、テーブル内のそのセルだけ背景色を変えたいと思うのですが、上手くいきません。

###発生している問題・エラーメッセージ

背景色が最初の名前しか背景色が変更されていない状態です。

###対象のコード

test.html

<!DOCTYPE html>
<html>
<head>
省略
</head>
<body>
<table>
<tr>
 <th>名前</th><th>誕生日</th><th>年齢</th>
</tr>
<tr>
 <#LIST#>//PHPにて置き換えしています
</tr>
</body>
</html>

test.php

<?php
//接続は省略
$query='select * from check_info';

$i = $lines = "";

if ($result = mysqli_query($link, $query)) {

     $i=0;
     while ($row = mysqli_fetch_row($result)) {
       $name[$i] = $row[0];
       $birthday[$i] = $row[1];
       $age[$i] = $row[2];


     if($name[$i] == "未確定"){
        $lines .= "<tr><td style='background-color:red;'>".$name[$i]."</td><td>".$birthday[$i]."</td><td>".$age[$i]."</td></tr>\n";
//未確定ならセルの背景色を赤にする     

} elseif ($name[$i] == "未確定" && $age[$i] == "30"){
$lines .= "<tr><td style='background-color:red;'>".$name[$i]."</td><td>".$birthday[$i]."</td><td style='background-color:red;'>".$age[$i]."</td></tr>\n";
//名前が未確定、年齢が30なら二つのセルの背景色を赤にする     

} else {
        $lines .= "<tr><td>".$name[$i]."</td><td>".$birthday[$i]."</td><td>".$age[$i]."</td></tr>\n";
//そうでなければ普通に表示
     }
     $i++;
    }
     mysqli_free_result($result);
}
 mysqli_close($link);


 $fp=fopen('./test.html','r');

 while(!feof($fp)) {
    $html_line=fgets($fp);
    $lines=str_replace("<#LIST#>",$lines,$html_line2);

    echo $lines;
 }

 fclose($fp);
 exit();

?>

###試したこと
年齢もelseifで指定して、複数のカラムを選択して抽出しようとしたら、最初の名前のみ背景色が赤くなりますが、例えば年齢のテーブル背景色が赤くならないです。
elseifの指定や、構文の考え方が間違っているのでしょうか？

###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報
MAC OS

行動規範の内容に同意します

回答1件

ベストアンサー

最初のifですでにtrueのためだと思います。
以下のように変更してみてはどうでしょう。

PHP
1if($name[$i] == "未確定" && $age[$i] == "30"){
2        $lines .= "<tr><td style='background-color:red;'>".$name[$i]."</td><td>".$birthday[$i]."</td><td>".$age[$i]."</td></tr>\n";
3    
4
5} elseif ($name[$i] == "未確定"){
6$lines .= "<tr><td style='background-color:red;'>".$name[$i]."</td><td>".$birthday[$i]."</td><td style='background-color:red;'>".$age[$i]."</td></tr>\n";
7
8} else {
9
10(略）
11

投稿2017/06/07 20:44

madone99

総合スコア1855

narutomo

2017/06/08 00:28

なるほど、構文をからきちんと勉強します。上記のように修正したら反映されました。ありがとうございます！ですが、今回ので一つ思ったことがありまして、今は2つのデータがこの値ならとしてますが、もしこれがもっと多くの数を扱うとしても、それだけelseif構文を書いていけば対応できると思いました。ただ私自身、参考書でプログラミング的にelseif構文は長くなりすぎないようにとの情報も見たことがありまして、if構文の長さついて、どのように考えればよろしいのでしょうか？具体的でなくて申し訳ありません。

madone99

2017/06/08 00:49

今回のように意図しない挙動になったり、コードの見通しも悪くなるのでまとめられる条件は極力シンプルにすることで、後々修正もしやすいと思います。今回だとこんな風にも書けるのではないかと思います。（もっときれいに書けそうですが。） if( ($name[$i] == "未確定" && $age[$i] == "30") || ( $name[$i] == "未確定")) { 以下、略

narutomo

2017/06/08 01:04

ありがとうございます。まとめて見やすくできるように頑張って書いて見ます。

行動規範の内容に同意します