w
ща пофикшу
Size: a a a
2021 March 03
AD
а еще я всегда думал о репозиториях как о штуке, которая исключительно доменная, т.е. UsersRepository нужен, если мы хотим вытащить для дашборда юзеров, а если мы хотим например юзеров с их балансами, которые лежат в другой таблице, то это будет например UsersBalancesRepo, которое строит запрос и возвращает другую структуру, хоть и в базе оно обращается к 100500 таблицам
У меня такое было отношение: есть юзеры для одного домена (например ордеров), есть для другого (биллинг) вот тут два репозитория с разной логикой но для юзеров
w
готово
AD
а, я понял
Ага
AD
т.е. to_a надо в конце метода for_dashboard
Ага
DG
`
attr_reader :repo` еще можно под private убратьw
Я бы модель через конструктор прокидывал бы и вместо миксинов наследование бы сделал. А в целом круто, мне нравится 👍
зачем только через наследование? чем это лучше?
w
`
attr_reader :repo` еще можно под private убратьа с какой целью?
DG
а с какой целью оно будет торчать наружу? )
w
всегда воспринимаю private как условность, такой ритуал для того чтоб людям вокруг было красивее
DG
так код и пишется для людей в первую очередь
особенно, если язык сам по себе красивый, такой как Ruby, и распологает к тому, чтобы делать людям красиво, то почему нет?
особенно, если язык сам по себе красивый, такой как Ruby, и распологает к тому, чтобы делать людям красиво, то почему нет?
DG
а вот тут
https://gist.github.com/IvanShamatov/e5db7f936ecf80bb7ba2ffdaee7d7712#file-users_repo-rb-L17
IMO плохая практика вот так что-то перекрывать
https://gist.github.com/IvanShamatov/e5db7f936ecf80bb7ba2ffdaee7d7712#file-users_repo-rb-L17
repo - это getter, или уже переменная внутри метода с таким же именем, которая просто getter перекрыла собой?IMO плохая практика вот так что-то перекрывать
AD
зачем только через наследование? чем это лучше?
Да просто вкусовщина
w
ок, поправил, сделав ридер приватным
AP
Зачем писать геттер, если переменная за пределами класса (даже в наследниках) нигде не используется?
AP
repo = filtered(dataset: repo, filter: filter) - вот тут непонятно, уйдёт в filtered репо из геттера, или nil из только что объявленной локальной переменной. Читатель не обязан знать такие тонкости языкаAP
На мой взгляд, приватные геттеры нужны только для классов-наследников. А в самом классе надо использовать переменную с собачкой. Для наглядности) Но это скорее вкусовщина
DG
ок, поправил, сделав ридер приватным
💖
AD
На мой взгляд, приватные геттеры нужны только для классов-наследников. А в самом классе надо использовать переменную с собачкой. Для наглядности) Но это скорее вкусовщина
Есть с этим проблема, гетер имутабельный как интерфейс, а инстанс переменная - нет
AP
Есть с этим проблема, гетер имутабельный как интерфейс, а инстанс переменная - нет
Согласен. Но этот вопрос в любом случае на совести разработчика. Написал вместо attr_reader attr_accessor - и вот тебе мутабельность) Защита от сильного дурака, не более того)