Size: a a a

2019 September 05

KT

Konstantin Tskhovrebov in GitFox
Редко бывает прямо кривой бекенд и вписываться в его исправление - очень рисковая затея.
Чаще бывает неочевидная бизнес логика на бекенде и ее исправлять не надо, ее надо поддерживать на клиенте. Это разные вещи
источник

AK

Andrey Khitryy in GitFox
Собственно об этом речь, что поддержка кривой бизнес логики это бизнес логика
источник

ES

Eugene Shapovalov in GitFox
Andrey Khitryy
Собственно об этом речь, что поддержка кривой бизнес логики это бизнес логика
сервер мне возвращает ответ 200 и сущность с id=0, если логин не удался (аккаунт не найден).  А он мог и 400 возвращать в таком случае, зачем мне на это завязываться в Interactor?

Я не считаю, что это бизнес-логика, и трансформирую это в AccountNotFoundException.
источник

AK

Andrey Khitryy in GitFox
Я считаю что это бизнес логика и место ей в модели, т.к. по всей видимости id = 0 заложено как неизвестный/пустой пользователь.
источник

AK

Andrey Khitryy in GitFox
Еще раз на всякий случай: Я не навязываю.
источник

AK

Andrey Khitryy in GitFox
Просто иногда проблема фронт разработки в ориентировании на API, а очень часто надо рассматривать систему в целом. По каким правилам она живет и так далее.
источник

KT

Konstantin Tskhovrebov in GitFox
Eugene Shapovalov
сервер мне возвращает ответ 200 и сущность с id=0, если логин не удался (аккаунт не найден).  А он мог и 400 возвращать в таком случае, зачем мне на это завязываться в Interactor?

Я не считаю, что это бизнес-логика, и трансформирую это в AccountNotFoundException.
Хороший вопрос. Я бы это решил в интерсепторе. То есть реально "исправил" бы сервер)
источник

AK

Andrey Khitryy in GitFox
@terrakok можно и так, но это вообще потом фиг найти
источник

AK

Andrey Khitryy in GitFox
мы так фиксили только 201/204, но там косяк был ретрофита
источник

ES

Eugene Shapovalov in GitFox
Konstantin Tskhovrebov
Хороший вопрос. Я бы это решил в интерсепторе. То есть реально "исправил" бы сервер)
ну и в нем завязываться на урлы?
источник

ES

Eugene Shapovalov in GitFox
Konstantin Tskhovrebov
Хороший вопрос. Я бы это решил в интерсепторе. То есть реально "исправил" бы сервер)
Идея норм, и в след раз я наверно и буду делать так.
источник

КР

Кирилл Романенко in GitFox
Мне вот стало интересно покопаться в gitfox, и после первых 10-ти минут у меня возник вопрос: а документация к классам и методам там намеренно не пишется? Есть ридми, но действительно полезной инфы там почти нет.
источник

VB

Vitaliy Belyaev in GitFox
Там как правило по названию класса/метода можно понять, что он делает. Зачем лишние комментарии?
К тому же там есть юнит тесты, которые тоже в каком-то смысле являются документацией)
источник

KT

Konstantin Tskhovrebov in GitFox
Кирилл Романенко
Мне вот стало интересно покопаться в gitfox, и после первых 10-ти минут у меня возник вопрос: а документация к классам и методам там намеренно не пишется? Есть ридми, но действительно полезной инфы там почти нет.
А вы намеренно не создаёте пулл реквест с документацией?
источник

AK

Andrey Khitryy in GitFox
Konstantin Tskhovrebov
А вы намеренно не создаёте пулл реквест с документацией?
В 2к19 в OSS должны быть доки и желательно велком смузи
источник
2019 September 06

АЕ

Алексей Ершов in GitFox
Ребята, расскажите, пожалуйста, почему в MainFragment всё инициализируется в onActivityCreated, а не в onViewCreated?
источник

АЕ

Алексей Ершов in GitFox
Потому что AHBottomNavigationAdapter требует Activity?
источник

SV

Sviat Volkov in GitFox
Алексей Ершов
Ребята, расскажите, пожалуйста, почему в MainFragment всё инициализируется в onActivityCreated, а не в onViewCreated?
мб есть зависимости которые надо взять из активити. а в ней зависимости собираются в onCreate()
источник

KT

Konstantin Tskhovrebov in GitFox
Алексей Ершов
Ребята, расскажите, пожалуйста, почему в MainFragment всё инициализируется в onActivityCreated, а не в onViewCreated?
это древняя привычка. была связана с чем-то на апи<19 или еще чем-то,  но сейчас уже не вспомнить
источник

АЕ

Алексей Ершов in GitFox
Спасибо) Вроде у меня из onViewCreated тоже нормально работает.
источник