DR
cond
isDead? -> deadAmount()
isSeparated? -> separatedAmount()
isRetired? -> retiredAmount()
true -> normalPayAmount()
end
isDead? -> deadAmount()
isSeparated? -> separatedAmount()
isRetired? -> retiredAmount()
true -> normalPayAmount()
end
Size: a a a
2018 December 04
DR
Пожалуйста и никаких return-ов в Elixir-е.
DR
И exception-ов
Е
вкусовщина (с) Евгений
да, я с этим согласен, я против фанатизма.
ŹR
def getPayAmount(true, _, _), do: deadAmount()
...
как вариант
DR
я канешна понимаю, что здесь найдется куча народу утверждающая что первое понятнее второго, непрерывный контрол флоу и прочее 😂
Первое не есть хорошо, но для этого есть switch в императивных языках и cond в Elixir-е, и поэтому надо сравнивать не вермишель из if-ов, а то как можно прекрасно и без return-ов написать тоже самое.
ŹR
Но это вкусовщина
DR
def getPayAmount(true, _, _), do: deadAmount()
...
как вариант
cond - уже сам по себе прекрасно справляется.ŹR
Все функции с несколькими перегрузками можно заменить на кейс внутри и наоборот
ŹR
Так что по сути это вкусовщина
ŹR
Ну и кейс <-> конд
ŹR
Надо только иногда следить чтобы какая-нибудь функция несколько раз не выполнялась
DR
cond - может быть в данном случае лучше, чем мэтчинг в функции и case-а, если что-то из этого нужно считать. (тогда можно посчитать первое и оно true, то не посчитается второе, в отличие от case-а, где вначале все посчитается.)DR
Но суть в том, что рефакторинг именно на кучу ретурнов тут совсем, ну никак не нужен.
ŹR
Dmitry Russ (Aleksandrov)
cond - может быть в данном случае лучше, чем мэтчинг в функции и case-а, если что-то из этого нужно считать. (тогда можно посчитать первое и оно true, то не посчитается второе, в отличие от case-а, где вначале все посчитается.)Ну я об этом и говорю.
ŹR
return не нужен точно, с этим согласен
ŹR
Кстати, интересно,
case({foo(), bar()}) do
{true, _} -> :ok
end
Оптимизируется чтобы не считать bar, или фиг вам?
case({foo(), bar()}) do
{true, _} -> :ok
end
Оптимизируется чтобы не считать bar, или фиг вам?
AK
Кстати, интересно,
case({foo(), bar()}) do
{true, _} -> :ok
end
Оптимизируется чтобы не считать bar, или фиг вам?
case({foo(), bar()}) do
{true, _} -> :ok
end
Оптимизируется чтобы не считать bar, или фиг вам?
Сильно сомневаюсь. Легко проверить ;)
DR
Кстати, интересно,
case({foo(), bar()}) do
{true, _} -> :ok
end
Оптимизируется чтобы не считать bar, или фиг вам?
case({foo(), bar()}) do
{true, _} -> :ok
end
Оптимизируется чтобы не считать bar, или фиг вам?
Оно не имеет права это по семантике оптимизировать
DR
Если ты хочешь, чтобы в case-е bar() выполнился бы в любом случае, то его невыполнение - это баг в компиляторе.