M
за второй вариант
+
Size: a a a
2020 October 10
V
Кстати, в рубишном геме sequel автор расположил функции и методы ... в алфавитном порядке. И знаете, читать такой код оказалось очень удобно. Сперва непривычно, да, но когда функций больше сотни - довольно быстро начинаешь замечать экономию времени.
YS
ну код не всегда в IDE читается, ревью на гитхабе например
YS
и по алфавиту быстрее ориентируешься
YS
да есть поиск
YS
но все-равно
AD
V
Кстати, в рубишном геме sequel автор расположил функции и методы ... в алфавитном порядке. И знаете, читать такой код оказалось очень удобно. Сперва непривычно, да, но когда функций больше сотни - довольно быстро начинаешь замечать экономию времени.
мне кажется когда функций больше сотни стоит их разделять на несколько модулей)
V
мне кажется когда функций больше сотни стоит их разделять на несколько модулей)
зачем?
AD
да все за тем же, для удобства поиска и чтения
LL
V
зачем?
Потому что это
code smellV
да все за тем же, для удобства поиска и чтения
учитывай что это гем, его чаще юзают, чем читают
AD
ну код не всегда в IDE читается, ревью на гитхабе например
если читать без ide, второй вариант удобнее из-за того что не нужно искать где определены используемые приватные функции
AD
да и код ревьюить удобно, не нужно прыгать по всему коду, достаточно просто листать вниз и весь код понятен
V
Матц прикольно ввёл ключевое слово
private в язык :) Интересно, Жозе обдумывал его заимствование и отказался или просто забыл?V
Потому что это
code smellКак ты можешь, не видя код, утверждать, что он smell?
V
Наверное отказался. Я бы тоже отказался. Сейчас вот смотрю километровый листинг руби, непонятно - случайный def - публичный или приватный.
F
не уверен, что правильно понял вопрос, но по всей видимости вы ищите
https://hexdocs.pm/ecto/Ecto.Query.html#where/3
wherehttps://hexdocs.pm/ecto/Ecto.Query.html#where/3
query =
from(c in City, where: c.country == "Sweden")
from(c in City, where: c.title == 2)
Repo.all(query)
это ведь чушь?))
from(c in City, where: c.country == "Sweden")
from(c in City, where: c.title == 2)
Repo.all(query)
это ведь чушь?))
AD
Franco
query =
from(c in City, where: c.country == "Sweden")
from(c in City, where: c.title == 2)
Repo.all(query)
это ведь чушь?))
from(c in City, where: c.country == "Sweden")
from(c in City, where: c.title == 2)
Repo.all(query)
это ведь чушь?))
я бы все-таки рекомендовала ознакомиться сначала с sql, если еще не
AB
Потому что это
code smellПонятие code smell придумали бизнес-ориентированные наци, опять же.
AB
Вообще, единственный критерий качества кода - ревью, никаким правилам сложнее "единый стиль именования переменных" оно не поддается. И даже в этом тупом примере уже могут возникать нестыковки