マップ機能のリファクタリングが改善されました

Question

現在、私のメソッドをリファクタリングしています。 私はすでに長いforeachと状態を削除しましたが、今は素晴らしいマップ機能を持つために最後の仕上げで少し苦労しています。

私の方法は次のようになります。ここで

/** 
* @param \Illuminate\Support\Collection 
* @return array 
*/ 
public static function convertTimesToChartData($times) { 
    $clientTotal = '0'; 
    $arrayIndex = 0; 
    $chartData = collect($times)->filter(function($item) { 
     return !is_null($item->end_time); 
    })->map(function($item) use(&$clientTotal, &$arrayIndex){ 
     $clients[$arrayIndex]['id'] = $item->client_id; 
     $clients[$arrayIndex]['project_id'] = $item->project_id; 
     $clients[$arrayIndex]['label'] = $item->company; 
     $clients[$arrayIndex]['sec'] = $item->end_time->timestamp - $item->start_time->timestamp; 
     $clientTotal += $item->end_time->timestamp - $item->start_time->timestamp; 
     $arrayIndex++; 
     return $clients; 
    })->flatMap(function($item) { return $item; }); 


    return $chartData; 
} 

、私は2つの質問があります：

1. を配列を代入良い方法はありますか？私はその後、

    return [ 
        [$arrayIndex]['id'] => $item->client_id, 
        [$arrayIndex]['project_id'] => $item->project_id, 
        [$arrayIndex]['label'] => $item->company, 
        [$arrayIndex]['sec'] => $item->end_time->timestamp - $item->start_time->timestamp, 
       ]; 
   

しかし、のようなリターンに直接割り当てることにしようとしているとき、私は未定義のインデックスエラーが発生します。

2. 要約された配列を返す最適な方法はありますか？私は同じクライアントのためにいくつかの時間のエントリがあるので、最後に、私は要約秒を欲しいです。それは私の例では機能しますが、私はそれを行う良い方法があると思います。私はマップ関数の外でそれを参照する必要がある$ arrayIndexを定義する必要があるので特に。これが最善の方法だとは思わないでください。任意のアドバイスや助けを

   $projects = array(); 
       $projectTotal = '0'; 
       foreach($taskTimes as $time){ 
        if(!is_null($time->end_time)) 
        { 
         if (isset($projects[$time->project_id])) 
         { 
          $projects[$time->project_id]['sec'] += $time->end_time->timestamp - $time->start_time->timestamp; 
         } else 
         { 
          $projects[$time->project_id]['id'] = $time->client_id; 
          $projects[$time->project_id]['label'] = $time->company; 
          $projects[$time->project_id]['sec'] = $time->end_time->timestamp - $time->start_time->timestamp; 
         } 
         $projectTotal += $time->end_time->timestamp - $time->start_time->timestamp; 
        } 
       } 
   

   ありがとう：ここ

は、元のソースです！

Answer 1

あなたはので、あなたのオリジナルのforeachループからのようなものができなったのと同じ結果を取得するには：それはあなたのconvertTimesToChartData()方法の一例に含まれていたように私は、結果の配列にproject_idを追加しました

$projects = collect($taskTimes) 
    ->filter(function ($time) { 
     return !is_null($time->end_time); 
    }) 
    ->groupBy('project_id') 
    ->map(function ($group) { 
     $group = collect($group); 

     return [ 
      'id'   => $group->first()->client_id, 
      'project_id' => $group->first()->project_id, 
      'label'  => $group->first()->company, 
      'sec'  => $group->sum(function ($item) { 
       return $item->end_time->timestamp - $item->start_time->timestamp; 
      }), 
     ]; 
    }); 

$projectsTotal = $projects->sum('sec'); 

を

希望します。