Czy niestandardowa metoda wyszukiwania w modelu Laravel powinna być statyczna?

Question

W następującym Laravel 5 Model powinien być metoda findByIdAndCourseOrFail statyczne?

class Section extends Model { 

    //should this method be static? 
    public function findByIdAndCourseOrFail($id, $courseId) 
    { 

     $result = $this->where('id', $id)->where('course_id', $courseId)->first(); 

     if (!is_null($result)) 
     { 
      return $result; 
     } 

     throw (new ModelNotFoundException())->setModel(Section::class); 

    } 


    } 

Z kontrolera:

class SectionsController extends Controller { 

protected $sections; 

public function __construct(Section $section) 
{ 

    $this->sections = $section; 

} 

public function foo($id, $courseId) //illustration only 
{ 
    $section = $this->sections->findOrFail($id); 

    $section = $this->sections->findByIdAndCourseOrFail($id, $courseId); 
    //would need to be non-static 

    $section = Section::findByIdAndCourseOrFail($id, $courseId); 
    //weird when compared with find above 
} 

Z jednej strony, nie jesteśmy działając na przykład sekcja [uwaga]. Z drugiej strony, w kontrolerze z autododatkowym wtryskiem przez Laravel's service container działalibyśmy na instancji: $sections = $this->sections-> findByIdAndCourseOrFail(7,3);, a mój IDE (PhpStorm) skrzeczałby, jeśli Static.

[Uwaga]: ten komentarz może być nieporozumieniem w działaniu modeli Laravel. Dla mnie, oczekiwałbym, że będzie to metoda klasy, a więc statyczna, w przeciwieństwie do instancji, która powróciłaby metoda find.

Answer 1

Nie jestem pewien, czy zamierzam użyć takiego rozwiązania w postaci local scopes. Ale to działa na mnie na laravel 5.2:

public function scopeFindByIdAndCourseOrFail($query, $id, $courseId) 
{ 
    $result = $query->where('id', $id)->where('course_id', $courseId)->first(); 

    if (!is_null($result)) 
    { 
     return $result; 
    } 

    throw (new ModelNotFoundException())->setModel(Section::class); 
} 

W regulatorze można go używać w obie strony:

$section = Section::findByIdAndCourseOrFail($id, $courseId); 

Albo

$model = new Section(); 
$section = $model->findByIdAndCourseOrFail($id, $courseId); 

Answer 2

class Section extends Model { 


public static function findByIdAndCourseOrFail($id, $courseId) 
{ 

    $result = self::where('id', $id)->where('course_id', $courseId)->first(); 

    if (!is_null($result)) 
    { 
     return $result; 
    } 

    throw (new ModelNotFoundException())->setModel(Section::class); 

} 


} 

class Section extends Model { 


public static function findByIdAndCourseOrFail($id, $courseId) 
{ 

    $result = self::where('id', $id)->where('course_id', $courseId)->first(); 

    if (!is_null($result)) 
    { 
     return $result; 
    } 

    throw (new ModelNotFoundException())->setModel(Section::class); 

} 


} 

Answer 3

Osobiście zrobiłbym to statyczną metodą, nie jestem pewien, czy istnieje "poprawna" odpowiedź, jak można zrobić. Sposób, w jaki je oddzielałem, jest taki, że jeśli robię coś dla instancji modelu, to robię normalną publiczną funkcję. Jeśli robię coś do kolekcji, używam statycznego. Na przykład:

$person = new Person(); 
$person->setAdmin(true); 
$person->save(); 

// OR 

$admins = Person::getAdmins(); 

w pierwszym przykładzie mamy konkretną instancję Person i jesteśmy manipulowania, cały kod byłoby po prostu manipulowanie tej konkretnej instancji. W drugim przykładzie działamy na całą kolekcję Person i chcemy, aby zwrócono kolekcję obiektów.

W twoim przypadku trzeba by zainicjować instancję Section tak, aby móc korzystać z non-statycznej metody publiczne, tak:

$section = new Section(); 
$foundSection = $section->findByIdAndCourseOrFail(7,3); 

Więc $section staje się tymczasową zmienną, która nie jest używana . Z drugiej strony, jeśli zrobisz to statycznie, możesz to nazwać, nie musząc tego robić.

$section = Section::findByIdAndCourseOrFail(7,3); 

Mam nadzieję, że ma to sens.