Devo usare Ecto.Repo in Controller o Model per Elixir Phoenix?

Question

Per alcuni query in controller di Phoenix, Ci sono due piani per me

Piano 1:

defmodule Demo.UserController do 
    # ... 
    def index do 
    # This is just for example 
    # The point is Repo in used here 
    Repo.all(User) 
    end 
end 

Piano 2:

defmodule Demo.User do 
    # ... 
    def all do 
    # Put all Repo API and building query logic in Model 
    Repo.all(__MODULE__) 
    end 
end 

preferisco il Piano 2. Poiché nella maggior parte situazioni, posso mettere tutta la logica sul recupero dei dati nel modello.

Ma trovo utilizzo guida ufficiale Plan 1 (docs/model) e di default Phoenix codice alias Repo nel controller al posto del modello (web/web.ex)

Qual è la migliore? E perché?

Answer 1

È necessario mantenere le chiamate di Repo all'interno del controller. Se la tua logica è complicata, dovresti considerare di spostare la logica nel proprio modulo di servizio.

Si dovrebbe trattare il modello come puro (privo di effetti collaterali) in modo che possano agire solo sui dati. Così, per esempio si potrebbe avere:

def alphabetical(query) 
    order_by(query, [u], u.name) 
end 

Ma non si dovrebbe avere:

def alphabetical(query) 
    order_by(query, [u], u.name) 
    |> Repo.all 
end 

Questo perché le query sono puramente i dati, la chiamata a Repo.all ha effetti collaterali (andando fuori al database) in modo appartiene al tuo controller.