Телеграмм чат группы gitfox страница 165

Size: a a a

GitFox

296 membersпожаловаться на группу

2019 June 27

Eugene Shapovalov in GitFox

Пройдитесь по веб-приложению, по LabCoat, и может нужно добавить очень важную логику в приложение.

2019 June 28

https://gitlab.com/terrakok/gitlab-client/blob/develop/app/src/main/java/ru/terrakok/gitlabclient/model/system/flow/FlowRouter.kt

FlowRouter - это новый роутер, который может управлять глобальным роутером.
Т.е. получается на новый flow создается свой роутер со своим navHolder, которые роутят все внутри flow. И если нужно, то передают команду в глобальный роутер.

GitLab

app/src/main/java/ru/terrakok/gitlabclient/model/system/flow/FlowRouter.kt · develop · Konstantin Tskhovrebov / GitFox

GitFox is your application to manage GitLab projects. We created the app with "Clean Architecture" concepts and intuitive user interface.

Не очень нравится FlowRouter#finishFlow - почему один из дочерних фрагментов флоу должен знать, что он не просто выходит, а именно завершает флоу.

источник

00:53пожаловаться

Konstantin Tskhovrebov in GitFox

Только в тех моментах, когда ему это надо знать: экран с такой логикой, что кнопка назад ведёт на предыдущий экран, а крест в туллбаре - выходит из флоу

источник

02:55пожаловаться

Konstantin Tskhovrebov in GitFox

Расскажите, как это сделаете вы

источник

02:55пожаловаться

Aleksey D. in GitFox

Konstantin Tskhovrebov

Расскажите, как это сделаете вы

Я бы взял этакие координатор с методами onDataOneCollected и onDataTwoCollected и внутри решал бы - обращаться к своему роутеру или глобальному.

источник

09:17пожаловаться

2019 June 29

Sergey8827 in GitFox

вижу андроид клиент построен не совсем так, как в описаниях клин архитекче.

источник

12:33пожаловаться

Eugene Shapovalov in GitFox

Sergey8827

вижу андроид клиент построен не совсем так, как в описаниях клин архитекче.

А что в реализации приложения противоречит Clean Architecture?

источник

12:35пожаловаться

Sergey8827 in GitFox

Eugene Shapovalov

А что в реализации приложения противоречит Clean Architecture?

не увидел, чтобы слой данных лежал отдельным андроид модулем и импортился через градл

источник

12:37пожаловаться

Sergey8827 in GitFox

так чтобы дата слой абсолютно не имел доступа к презентешен, и ничего о нем не знал

источник

12:39пожаловаться

Sergey8827 in GitFox

хотя я вижу есть интеракторы

источник

12:42пожаловаться

Aleksey D. in GitFox

Sergey8827

не увидел, чтобы слой данных лежал отдельным андроид модулем и импортился через градл

Это ведь условности, а не правила

источник

12:44пожаловаться

Sergey8827 in GitFox

Aleksey D.

Это ведь условности, а не правила

я скорее плохо теще изучил проект. И глянул поверхностно. Как на твой взгляд, сам проект отлично подходит как образец по клин архитекче?

источник

13:20пожаловаться

Sergey8827 in GitFox

или есть репозитории по удачней например https://github.com/rakshit444/news-sample-app

GitHub

rakshit444/news-sample-app

A sample news app which demonstrates clean architecture and best practices for developing android app - rakshit444/news-sample-app

источник

13:20пожаловаться

Sergey8827 in GitFox

прост тут все по андроид модулям, например

источник

13:21пожаловаться

Alexey Pushkarev in GitFox

Sergey8827

или есть репозитории по удачней например https://github.com/rakshit444/news-sample-app

GitHub

rakshit444/news-sample-app

A sample news app which demonstrates clean architecture and best practices for developing android app - rakshit444/news-sample-app

По слоям давно уже не делают. Потому что с увеличением кодовой базы у тебя модули будут сильно распухать. Надо либо по фичам разбивать, либо гибридное слой/фича.

источник

13:23пожаловаться

Konstantin Tskhovrebov in GitFox

Модули и клин архитектура вообще не связаны

источник

13:32пожаловаться

Nikita Yatskivskiy in GitFox

Alexey Pushkarev

Поддерживаю.
Бьёшь по фичам, а внутри фичи уже по клину можешь.

источник

13:33пожаловаться

Nikita Yatskivskiy in GitFox