Size: a a a

2019 June 05

ES

Eugene Shapovalov in GitFox
Алексей Ершов
Можете уточнить? Активити ведь уже стартануло, вроде поздно проверять.
Вообще у меня проблема в том, что AppActivity рассчитывает на то, что оно в единственном экземпляре. У него есть презентер-синглтон, который инициализирует состояние приложения, и если создаётся другой инстанс AppActivity, и к нему прикрепляется существующий презентер - возникает неправильное состояние, потому что презентер думает, что стек навигации уже есть, и ничего не делает. А стека нет, он в другом инстансе активити.
Ну эт чистый костыль, но в Application можно сохранить activity.taskId, и если он оказался другим, то закрывать это Activity и перебрасывать в другое.
источник

АЕ

Алексей Ершов... in GitFox
Eugene Shapovalov
Ну эт чистый костыль, но в Application можно сохранить activity.taskId, и если он оказался другим, то закрывать это Activity и перебрасывать в другое.
А, понял) Да, как вариант, но отдельное активити для интент фильтра - примерно того же уровня костыльности. Работать - работает, но причину, по которой на работает singleTask, я не понимаю :)
источник

KT

Konstantin Tskhovreb... in GitFox
причина в том, что запускающее приложение может в своем таске запускать. тут надо дальше играть с флагами, я не подскажу сходу.
источник

АЕ

Алексей Ершов... in GitFox
Такое ощущение, что флаги могут и не помочь, потому что запускающее приложение может их оверрайдить. Если придумаю что-нибудь более внятное, чем дополнительное активити-точка входа - поделюсь. Спасибо за помощь!
источник
2019 June 06

AK

Alexey Kalyaganov in GitFox
Привет, у меня вопрос к code style.
Вот тут в SessionInteractor есть методы
fun getUserAccounts() = sessionRepository.getUserAccounts()
fun getCurrentUserAccount() = sessionRepository.getCurrentUserAccount()
Мне кажется лучше явно указывать возвращаемый тип, иначе не совсем понятно что метод вернет с первого взгляда.
У вас в компаниях тоже так пишут?
источник

MS

Maksim Sukhotski in GitFox
Нужно ли явно указывать возвращаемый тип?
Анонимный опрос
61%
Да
39%
Нет
Проголосовало: 38
источник

MS

Maksim Sukhotski in GitFox
имхо, где есть код-ревью - нужно
но если нет, то студия умеет show property type hints
источник

ES

Eugene Shapovalov in GitFox
Alexey Kalyaganov
Привет, у меня вопрос к code style.
Вот тут в SessionInteractor есть методы
fun getUserAccounts() = sessionRepository.getUserAccounts()
fun getCurrentUserAccount() = sessionRepository.getCurrentUserAccount()
Мне кажется лучше явно указывать возвращаемый тип, иначе не совсем понятно что метод вернет с первого взгляда.
У вас в компаниях тоже так пишут?
На этой неделе будет добавлен ktlint в pipeline и он будет бить по рукам 🙂
источник

AK

Alexey Kalyaganov in GitFox
👌
источник

ES

Eugene Shapovalov in GitFox
Alexey Kalyaganov
Привет, у меня вопрос к code style.
Вот тут в SessionInteractor есть методы
fun getUserAccounts() = sessionRepository.getUserAccounts()
fun getCurrentUserAccount() = sessionRepository.getCurrentUserAccount()
Мне кажется лучше явно указывать возвращаемый тип, иначе не совсем понятно что метод вернет с первого взгляда.
У вас в компаниях тоже так пишут?
источник
2019 June 09

ES

Eugene Shapovalov in GitFox
@tlvvl

https://gitlab.com/terrakok/gitlab-client/merge_requests/207

Валик, пофикси конфликты.
источник

ES

Eugene Shapovalov in GitFox
@tlvvl
https://gitlab.com/terrakok/gitlab-client/issues/159

Ты забыл реализовать вот это поведение. Создал задачу, можешь ее закрыть тоже.
источник
2019 June 12

VL

Valentin Logvinovitc... in GitFox
@terrakok Здравствуйте. Вы закрыли мой МР(переход на авторов). Я так понимаю текущая версия остаётся (через урл) и потом реализуется диплинкой ? Или мне надо сделать новый мр с учётом 2 кейсов?
источник

KT

Konstantin Tskhovreb... in GitFox
оставьте до диплинка)
источник

VL

Valentin Logvinovitc... in GitFox
👍
источник
2019 June 20

MS

Maksim Sukhotski in GitFox
привет, что думаете по поводу использования lateinit ? стоит ли им пользоваться или стоит полностью выпилить?
https://arturbosch.github.io/detekt/potential-bugs.html#lateinitusage
источник

AT

Andrey Timofeev in GitFox
Maksim Sukhotski
привет, что думаете по поводу использования lateinit ? стоит ли им пользоваться или стоит полностью выпилить?
https://arturbosch.github.io/detekt/potential-bugs.html#lateinitusage
Как его выпилить для инжектов?
источник

MS

Maksim Sukhotski in GitFox
Andrey Timofeev
Как его выпилить для инжектов?
как отрефакторить в таком случае код, чтобы lateinit исчез? предоставляется ли это вообще возможным?)

inner class ViewHolder(view: View) :
       RecyclerView.ViewHolder(view) {

       private lateinit var category: Category

       init {
           view.setOnClickListener {
               clickListener(category)
           }
       }

       fun bind(item: Category) {
           category = item
           itemView.name.text = item.name
       }
}
источник

AT

Andrey Timofeev in GitFox
Maksim Sukhotski
как отрефакторить в таком случае код, чтобы lateinit исчез? предоставляется ли это вообще возможным?)

inner class ViewHolder(view: View) :
       RecyclerView.ViewHolder(view) {

       private lateinit var category: Category

       init {
           view.setOnClickListener {
               clickListener(category)
           }
       }

       fun bind(item: Category) {
           category = item
           itemView.name.text = item.name
       }
}
Это не тот случай, здесь не понятно зачем lateinit
источник

MS

Maksim Sukhotski in GitFox
отправил картинкой, чтоб читалось
источник