Wie man mit sich wiederholenden Parametern umgeht?

Question

Wie viele wiederkehrende Parameter in einem create() zu behandeln. Ich mag es, ordentlich und lesbar zu sein.

Zum Beispiel haben wir in der Steuerung addList() -Methode mit vielen Parametern.

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, 
       ); 
} 

In der List Klasse haben wir ein Verfahren schaffen, die auch viele sich wiederholende viele Parameter. Und in der $this->api->create() und $this->listRepository->create() die auch irgendwie wiederholt wird. Gibt es eine Möglichkeit, dies zu bereinigen oder umzuformen?

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

Passing-Request-Objekt ist gut und nicht einzeln die jeden Parameter zu senden.

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

In List-Klasse, da Request-Objekt sowieso Benutzersitzung hat, übergeben einfach das Anfrage-Objekt zu listRepository Klasse.

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; 
    } 
}