PHP 比較演算子について

Question

###前提・実現したいこと
初期の色を「黒」
プラスだったら「青」
マイナス「赤」
という感じに色がつくようにしたいです。

###該当のソースコード

<?php if($total_price_str >= 0){ $color = 'blue'; }else{ $color = 'red'; } ?> <?php printf ('<span style="color:' . $color . ';">%d円</span><br />', ceil($total_price_str)); ?>

###試したこと
上記のコードではうまくいきますが、比較演算子によって０が青のままに表示されてしまうので０を黒にするにはどんな感じに比較演算子を使ったらいいでしょうか？ご教授お願いします。

Answer 1

PHP7以降ですが

$colors = [
   -1 => 'red',
    0 => 'black',
    1 => 'blue',
];
$color = $colors[$total_price_str <=> 0];

と書くと非常にスマートです。なんなら一時変数なしでも。

$color = [
   -1 => 'red',
    0 => 'black',
    1 => 'blue',
][$total_price_str <=> 0];

Answer 2

前提：
$total_price_strには必ず数値が入っている。

☆1つ1つ条件を整理する
・初期の色は黒
・プラスであるというのはすなわち 0より大きい
・マイナスであるというのはすなわち 0より小さい

それをそのままプログラムで書く。

// 初期の色 black
$color = 'black'; 
// 0より大きい場合 blue
if($total_price_str > 0){ 
$color = 'blue'; 
// 0より小さい場合 red
}else 
if($total_price_str < 0){ 
$color = 'red'; 
}
// この時点で上記に当てはまった色文字列が格納されている

Comments

[marusu]

ご回答ありがとうございました。
他の人と同じ回答でした。とても勉強になりました。
ありがとうございました。

Answer 3

幾らでもやり方はありますが、今のやり方を活かすならこうですかね。
※未検証

<?php 
$color = 'black'; 

if($total_price_str > 0){ 
$color = 'blue'; 
}elseif($total_price_str < 0){ 
$color = 'red'; 
} 
?>

<?php printf ('<span style="color:' . $color . ';">%d円</span><br />', ceil($total_price_str)); ?>

Comments

[marusu]

ご回答ありがとうございました。
なるほどです。最初の段階に黒の値を変数に格納すればいいのですね。
ありがとうございました。勉強になりました。

[m.ts10806]

ただ、htmlやbodyなどの全体の要素でCSSで色を指定していない限りは黒がデフォルト色となりますので、その場合はblackの指定がいらなくなりますし、styleも不要になると思います。
ちょっとした課題ですが今度は「黒になる場合はstyle指定なし」というのにもチャレンジしてみてください。
あと、金額の表示のようですが、money_format()という関数もあるのでそちらの方が処理がすっきりしそうですね。
http://php.net/manual/ja/function.money-format.php

Answer 4

if($total_price_str == 0){ 
    $color = 'black'; 
} else if($total_price_str > 0){ 
    $color = 'blue'; 
}else{ 
    $color = 'red'; 
} 

修正版

<?PHP
$total_price = ceil($total_price_str);
if($total_price == 0){ 
    $color = 'black'; 
} else if($total_price > 0){ 
    $color = 'blue'; 
}else{ 
    $color = 'red'; 
} 
?>

<?php printf ('<span style="color:' . $color . ';">%d円</span><br />', $total_price_str); ?>

Comments

[marusu]

ご回答ありがとうございます。
試したことに書き忘れてましたが、私の方で同じコードを試して見たのですが黒のままでした。

[ttyp03]

もしかしてceilを使っているということは小数値ですか？
という前提で修正版を追記しておきました。
これでいかがでしょうか。

[marusu]

度々コードミスがありました。
ご教授くれたコードでうまくいくことができました。
最初の段階で==を使えばよかったのですね。
単純なことなのに思いつきませんでした。
ありがとうございました。

[ttyp03]

もしかして==ではなく、=にしてたってオチですか…。

[marusu]

過去に遡って自分のコードを確認したところその通りでした。
比較演算子に３時間も悩んでたのでとてもすっきりしました。
ありがとうございました。