PHP・htmlのコーディングについて

Question

プログラマー歴一年の新米です。

現在、別の方が書いたコードをリファクタリングする作業をしているのですが、
if文がたくさん出てくる箇所がありどう記述するのが
美しいのか悩んでおります

<div class="a <?php if($a === true):?> b <?php endif: ?>">test</div>
    <div class="c <?php if($b === true):?> d <?php endif: ?>">test</div>
        <?php if($a === true):?>
            <?php echo 'aaaaaaa'; ?>
        <?php endif; ?>

元のコードになります。

if文の値がclassファイルから持ってくるので一文がすごく長くなってしまうのと
classの中にif文があるのが気持ち悪いので下記のように書いてみたところこれもifがたくさんできてしまい見づらくなってしまうのでなにかいい書き方があればご教授お願いいたします

<?php if($a === true): ?>
    <div class="a b">test</div>
<?php else: ?>
    <div class="a">test</div>
<?php endif; ?>

<?php if($b === true): ?>
    <div class="c d">test</div>
<?php else: ?>
    <div class="c">test</div>
<?php endif; ?>

Comments

[dddd_gond]

条件式のところに代入してますけど、これはタイプミスですかね。

[kei344]

PHPで処理することが決まっているなら、質問タグに「PHP」を追加してください。

[n.t]

申し訳ありませんでした。修正いたしました。

[dddd_gond]

コロンとセミコロンの対応があってないみたいですけど。それと、else()とかはエラーになるんじゃないですか？

Answer 1

true/falseの2パターンぐらいだったら、配列で直に書いてもそんなに圧迫感ない気がします。

<?php

$a = true;
$b = false;

?>

<div class="<?= ['a', 'a b'][$a] ?>">test</div>
<div class="<?= ['c', 'c d'][$b] ?>">test</div>

<div class="a b">test</div>
<div class="c">test</div>

PHP: 配列 - Manual
http://php.net/manual/ja/language.types.array.php

bool も integer にキャストされます。つまり、 キーに true を指定すると実際には 1 に格納され、 同様にキーを false とすると実際には 0 となります。

PHP 5.4.x から PHP 5.5.x への移行
http://php.net/manual/ja/migration55.new-features.php

Array と string をデリファレンスして、 個々の要素や文字に直接アクセスできるようになりました。

Comments

[退会済みユーザー]

横からすみません。これってよく使われる手法なんでしょうか？
少し長めのコードの中にあったら、「ん？」ってなりそうな気がして、ちょっと使用される頻度が気になりました。
よく使われる手法なら、頭にねじ込んでおこうかなぁと＾＾；

[nnssn]

見かけたのも使ったのも2,3度といった感じです。あまり使われてはいないと思います。質問文のコードを見て、分岐させるのはいや、コントローラでクラス名用意するのもいやと考えてたらふいに思い出しました。

[退会済みユーザー]

ありがとうございます。
勉強になりました。

Answer 2

php側で吐き出す文字列を絞り込み、三項演算子ですっきりさせれば、

<div class="<?php echo $a?'a b':'a'; ?>">test</div>
<dif class="<?php echo $b?'c d':'c'; ?>">test</div>

のような形になります。

Comments

[n.t]

ありがとうございます。すっきり書くことができました