繰り返しパラメータを扱うにはどうすればいいですか？

Question

create()で多くの繰り返しパラメータを処理する方法。私はきれいに見えるし、読むことが好きです。

たとえば、コントローラには、多くのパラメータを持つaddList（）メソッドがあります。我々はこれも多くの反復的な多くのパラメータのメソッドを作成する必要がありListクラスで

public function addList(CreateListRequest $request) 
{ 
    $created = $this->list->create(
        $request->user(),    
        $request->name, 
        $request->subject, 
        $request->description, 
        $request->location, 
        $request->do_email, 
        $request->provider, 
        $request->something1, 
        $request->something2, 
       ); 
} 

。そして、$this->api->create()と$this->listRepository->create()にもちょっと繰り返されています。これを清掃する方法やリファクタリングする方法はありますか？

class List 
{ 
    public function create($user, $name, $subject, $description, $location, $doEmail, $provider, $something, $something2) 
    { 
     $list = $this->api->create($name, $subject, $description, $location); 

     if ($list->status == "success") { 
      // Add to database 
      $row = $this->listRepository->create($user->id, $name, $subject, $description, $location, $doEmail, $provider, $something, $something2); 
      return $row; 
     } 

     return false; 
    } 
} 

Answer 1

のパラメータを個別に送信するのではなく、要求オブジェクトを渡すのが適切です。要求オブジェクトは、とにかくユーザセッションを有しているので

public function addList(CreateListRequest $request) { 
    $created = $this->list->create($request); 
} 

Listクラスにおいて、 は単にlistRepositoryクラスに要求オブジェクトを渡します。

class List { 

    public function create($request) { 

     $list = $this->api->create($request->only(['name', 'subject', 'description', 'location'])); 

     if ($list->status == "success") { 
      $row = $this->listRepository->create($request); 
      return true; 
     } 

     return false; 
    } 
}