Czy powinienem używać Ecto.Repo w kontrolerze lub modelu dla Elixir Phoenix?

Question

jakiegoś zapytania w Kontrolera Phoenix, są tam dwa plany dla mnie

Plan 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 

Plan 2:

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

wolę plan 2. Ponieważ w większości sytuacje, mogę umieścić całą logikę na temat pobierania danych w Modelu.

Ale znajdę oficjalny podręcznik użytkowania Plan 1 (docs/model) i Phoenix domyślny kod alias Repo w kontrolerze zamiast Modelu (web/web.ex)

Który z nich jest lepszy? I dlaczego?

Answer 1

Powinieneś utrzymywać swoje wywołania Repo wewnątrz kontrolera. Jeśli twoja logika jest skomplikowana, powinieneś rozważyć przeniesienie logiki do własnego modułu serwisowego.

Powinieneś traktować funkcje swojego modelu jako czyste (bez efektów ubocznych), aby działały tylko na danych. Tak na przykład można mieć:

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

Ale nie powinien mieć:

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

To dlatego, że zapytania są wyłącznie dane, wywołanie Repo.all ma skutki uboczne (schodzili do bazy danych), tak należy do kontrolera.