クリーンな方法は、

Question

私はしばしばこのように構造が必要になります。

$data = []; 
foreach($input as $entry) { 
    if(!isset($data[$entry->type])) { 
     $data[$entry->type] = 0; 
    } 
    $data[$entry->type] += $entry-value; 
} 

私は、これは信じられないほど醜いなく読むことは非常に簡単だと思います。どのようにこれを解決し、より流暢な/読みやすい/より速く入力する任意のアイデアですか？

私は通常、これらの種類の「問題」についてはあまり気にしませんが、これらの構造はかなり必要であり、それらを書いた後はいつも少し恥じます。この場合、より好適なのは、

Answer 1

これはnull合体演算子と7.0以降PHPに美しく行うことができますが存在しない場合

$data = []; 
foreach($input as $entry) { 
    $data[$entry->type] = $entry-value + ($data[$entry->type] ?? 0); 
} 

$data[$entry->type] ?? 0は$data[$entry->type]または0の値を返します。詳細はthe RFC

Answer 2

emptyである。

$data = []; 
foreach($input as $entry) { 
    if(empty($data[$entry->type])) { 
     $data[$entry->type] = 0; 
    }else{ 
     $data[$entry->type] += $entry->value; 
    } 
} 

三項演算子を使用することもできます。それはこのタイプが満たされた最初の時間であれば、私はあなたが開始値として0を意味するものではありませんよねものの

$data[$entry->type] = isset($data[$entry->type])?$data[$entry->type] + $entry->value:0; 

が、$entry->value：

$data[$entry->type] = empty($data[$entry->type]) ? 0 : $data[$entry->type] + $entry->value; 

Answer 3

非常にクリーンソリューションは、三項演算子を使用しています：あなたは、コードBのスタイルの詳細については懸念している場合は、a）はPHPのコーディング標準にhttp://www.php-fig.org/を確認することができますいずれか

$data[$entry->type] = isset($data[$entry->type])?$data[$entry->type] + $entry->value:$entry->value; 

Answer 4

は）あなた場合のif/else文のための三項演算子を使用しますより効率性を重視しています。

$data = []; 
foreach($input as $entry) 
{ 
    $data[$entry->type] = empty($data[$entry->type]) ? 0 : $data[$entry->type] + $entry->value; 
}