Кошерно ли в Enum.map вызывать функции с сайд-эффектами? Типа вместо: users

Question

Кошерно ли в Enum.map вызывать функции с сайд-эффектами? Типа вместо: users

= Enum.map(uids, &get_user(&1))
Enum.each(uids, &send_hello(uid))
написать:
users = Enum.map(uids, fn uid ->
send_hello(uid)
get_user(uid)
)
Вроде как и почему бы и нет? Но почему-то мне второй вариант кажется каким-то попахивающим не пойми чем.
Дохтур, у меня что-то с головой?

#elixir #programming #russian

0

02.01.2021

7 ответов

17 просмотров

Lama Lover

Так ты отдели сайд-эффект от чистой функции И вызывай что-то типа uids |> Enum.map(&get_user/1) |> Enun.each(&send_hello/1) PS: Я тут немного логику поменял, соре

0

02.01.2021

Евгений Автор вопроса

Lama Lover
Так ты отдели сайд-эффект от чистой функции И выз...

Вопрос-то не об этом. Вопрос о самом стиле. В какой-нибудь императивщине вот такой код не вызывает ни малейшего неприятия: User[] users; foreach(uid; uids) { users ~= getUser(uid); sendHello(uid); }

0

02.01.2021

Anastasiya Dyachenko

Евгений
Вопрос-то не об этом. Вопрос о самом стиле. В како...

В императивном стиле это тоже выглядит не норм, у функции должна быть определенная задача. Либо получить список юзеров либо отправить всем сообщение

0

02.01.2021

Евгений Автор вопроса

Anastasiya Dyachenko
В императивном стиле это тоже выглядит не норм, у ...

Ну так это не функция, а некая часть более общей функции.

0

02.01.2021

Ihor Katkov 🇺🇦

Не нравится потому, что ты путаешь уровни абстракций. ids |> get_users |> send_notifications Enum функции уже детали реализации, которые ты должен спрятать за бизнес логикой

0

02.01.2021

Евгений Автор вопроса

Ihor Katkov 🇺🇦
Не нравится потому, что ты путаешь уровни абстракц...

Да нет, просто пример корявый. В реальном моем случае уровни абстракции одинаковые.

0

02.01.2021

V · Accepted Answer

V

Всё правильно говоришь, выглядит отвратительно. Можешь придумать способ порицания для нарушителей.

0

02.01.2021

169 похожих чатов

Кошерно ли в Enum.map вызывать функции с сайд-эффектами? Типа вместо: users

7 ответов

Похожие вопросы