А можно немного критики относительно стиля определения метода/функции? я иногда

Question

А можно немного критики относительно стиля определения метода/функции? я иногда

путаю аргументы, и потом мучаюсь с дебагом.

решил это мапом побороть:

def get_session(%{user_id: user_id, id: id}) when is_nil(id) or is_nil(user_id), do: nil
def get_session(p = %{user_id: user_id, id: id}), do: get_session_query(p) |> Repo.one()

defp get_session_query(%{user_id: user_id, id: id}), do: from s in Session, where: s.user_id == ^user_id and s.id == ^id

#elixir #programming #russian

0

16.06.2023

16 ответов

29 просмотров

Darkwing Duck

def get_session(%{id: nil}), do: nil def get_session(%{user_id: nil}), do: nil def get_session(p), do: get_session_query(p) |> Repo.one() и не уверен что-то я ,что сработает ^ т.к. оно в паттерн-матчинге только работает

0

16.06.2023

Fried Zucchini

Darkwing Duck
def get_session(%{id: nil}), do: nil def get_sessi...

from это макрос из Ecto.Query, там ^ имеет другое значение

0

16.06.2023

Vihtor Автор вопроса

Darkwing Duck
def get_session(%{id: nil}), do: nil def get_sessi...

а when не предпочтительнее? так все пишут? 🙂

0

16.06.2023

Darkwing Duck

Vihtor
а when не предпочтительнее? так все пишут? 🙂

ну красивее же, "так все пишут" - не замечал) when юзаю когда паттерн-матч не прикрутить

0

16.06.2023

Vihtor Автор вопроса

Darkwing Duck
ну красивее же, "так все пишут" - не замечал) when...

красивее — да. у меня просто DRY OCD.

0

16.06.2023

Lama Lover

Vihtor
красивее — да. у меня просто DRY OCD.

OCD?

0

16.06.2023

Vihtor Автор вопроса

Lama Lover
Длинно, зачем , do: когда можно нормальный do end

можно еще короче? def get_session(%{user_id: user_id, id: id}) when is_nil(id) or is_nil(user_id), do: nil def get_session(%{user_id: user_id, id: id}) do query = from(s in Session, where: s.user_id == ^user_id and s.id == ^id) Repo.one(query) end

0

16.06.2023

Lama Lover

Vihtor
можно еще короче? def get_session(%{user_id: ...

Лайфхак: вместо двух is_nil можно писать nil in [id, user_id]

0

16.06.2023

Vihtor Автор вопроса

Lama Lover
OCD?

https://en.wikipedia.org/wiki/Obsessive%E2%80%93compulsive_disorder

0

16.06.2023