0

Na interface do PHPStorm é exibido uma mensagem de aviso informando que estou chamando o método orderBy de forma estática, porém, este não é estático. Minha chamada:

Artigo::orderBy('edicao', 'desc')->paginate(25);

Isso estaria "errado"? Ou teria que fazer:

(new Artigo)->orderBy('edicao', 'desc')->paginate(25);

Obs: estou utilizando Laravel 5.6

5
  • Para acessar um método ou campo de objeto, você deveria usar o operador seta -> Eu só vi o operador de resolução de escopo :: sendo usado para determinar métodos e Campos de classe/estáticos em php Commented 21/03/2018 às 1:13
  • Isso é chamado Facade dá uma busca na documentação do Laravel
    – novic
    Commented 21/03/2018 às 1:25
  • 2
    laravel.com/docs/5.6/facades de uma lida e ignore mensagem do PHPStorm não tem problema usar a primeira forma
    – novic
    Commented 21/03/2018 às 1:28
  • Dei uma olhada em grandes projetos, é normal/comum essa abordagem pelo que pude observar. Commented 21/03/2018 às 1:58
  • Não tem problema, a primeira forma está correta sim.
    – arllondias
    Commented 22/03/2018 às 15:48

1 Resposta 1

0

A forma como você está fazendo não é errada, mas acho que a melhor forma de fazer a chamada dos models seria chamar eles no __constructor ex.:

private $artigoModel

public function __constructor(Artigo $artigoModel){
    $this->artigoModel = $artigoModel
}

Assim você consegue fazer a chamada por $this->artigoModel em todo o código e não precisa ficar instanciando várias vezes caso precise dele em mais de uma função.

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .