PHP - if else文に変数を代入する

Question

私はこれに対する確かな答えを見つけることができませんでした。 if/else文に変数を代入することができるので、一部のHTMLに文全体を含める必要はありません。

たとえば、これは正しいのでしょうか？そうでない場合は、正しい方法はありますか？

$agency = if ($event == "Tornado Watch" || $event == "Severe Thunderstorm Watch") { 
      echo "NWS Storm Prediction Center"; 
     } elseif ($event == "Hurricane Watch" || $event == "Tropical Storm Watch") { 
      echo "NWS National Hurricane Center"; 
     } else { 
      echo $wfo; 
     } 

Answer 1

私は何がやりたいことはいくつかのロジックに基づいて$機関に値を代入して、$機関の値をエコーであると思います。

<?php 
$agency = $wfo; 
if ($event == "Tornado Watch" || $event == "Severe Thunderstorm Watch") 
{ 
    $agency = "NWS Storm Prediction Center"; 
} 
elseif ($event == "Hurricane Watch" || $event == "Tropical Storm Watch") 
{ 
    $agency = "NWS National Hurricane Center"; 
} 

echo $agency; 

[編集]あなたはそれがより保守制御構造に吹き出さあなたの文字列比較のすべてを持つスキップや機関にあなたのイベントをマップする連想配列を作成するかもしれません。あなたがこれを行うことができますいくつかの方法がありますが、ここでは単純なものです：

<?php 
$eventAgencyMap = [ 
    'Tornado Watch'    => 'NWS Storm Prediction Center', 
    'Severe Thunderstorm Watch' => 'NWS Storm Prediction Center', 
    'Hurricane Watch'   => 'NWS National Hurricane Center', 
    'Tropical Storm Watch'  => 'NWS National Hurricane Center' 
]; 

$agency = (array_key_exists($event, $eventAgencyMap)) ? $eventAgencyMap[$event] : $wfo; 

Answer 2

私はとしてIMOそれはもう少しクリーンかつ少ないコードであるロブのソリューションを使用していました。それで、私はこの解決法も捨てたいと思っていました。誰かが私が考えていたswitch文について言及しました。だから私はロブの答えを見て、これは私のためにうまくいった前にそれを試した。それはロブが選択された解決策でなければならないにもかかわらず、代わりの方法です。

$agency = ''; 

      switch($event) 
{ 

       case 'Tornado Watch': 
        $agency = 'NWS Storm Prediction Center'; 
           break; 
       case 'Severe Thunderstorm Watch': 
        $agency = 'NWS Storm Prediction Center'; 
           break; 
       case 'Hurricane Watch': 
        $agency = 'NWS National Hurricane Center'; 
           break;    
       case 'Tropical Storm Watch': 
        $agency = 'NWS National Hurricane Center'; 
           break; 
       case 'Flash Flood Watch': 
        $agency = $wfo; 
           break; 

}