- ”список дел”, мой первый опыт с mvvm, у кого есть желание провести codereview? У опытного разработчика это займет меньше минуты. Заранее всем спасибо!
https://github.com/IgS279/TODO-MVVM
меньше минуты? серьезно?
юзай котлинэкстеншн и не пиши каждый раз findviewbyid
Ну я в вас не сомневаюсь)
Он не дипрекейтед разве?
Похоже ты уже использовал свой лимит на ревью, пока писал этот месседж )
я чет пропустил ?
А кто сказал что деприкейтед?
Мне студия ворнинг давала, что он все и надо юзать ViewBinding
private lateinit var infoViewModel: InfoViewModel незачем тебе это. вне одного метода не юзается
ну приехали
InfoViewModel вообще не нужен как класс. у тебя же 1 текстовка
мне они тоже нравились, правда немного были когда одни и те же id есть в разных хмл...
зачем тебе больше 1 активити? перепиши на сингл активити
supportActionBar?.title = "NewTODO" supportActionBar?.setDisplayHomeAsUpEnabled(true) supportActionBar?.setDisplayShowHomeEnabled(true) не хардкодь. юзай стринг. ресурсы и вообще сделай через апплай. вся красота котлина в том, чтобы не писать 3 раза имя переменной
_myNotes не надо так делать в андроид. мы не веб
спасибо за ваши замечания!
не пушь закомментированный код в гитхаб
айси лончер пнг лежит вообще не там где надо ему лежать
mobile_navigation.xml так. стоп. а это на кой ?
видимо это graph Navigation Library
да понятно. но там вроде и не нужно было (как я сначала подумал)
Обсуждают сегодня