ОА
Мне студия ворнинг давала, что он все и надо юзать ViewBinding
ну приехали
Size: a a a
2021 March 05
C
Miui 12?
Да, ксяоми
ОА
Ребя, всем доброго дня. Выложил на github самое тривиальное приложение - ”список дел”, мой первый опыт с mvvm, у кого есть желание провести codereview? У опытного разработчика это займет меньше минуты. Заранее всем спасибо！
https://github.com/IgS279/TODO-MVVM
https://github.com/IgS279/TODO-MVVM
InfoViewModel вообще не нужен как класс. у тебя же 1 текстовка
DH
ну приехали
мне они тоже нравились, правда немного были когда одни и те же id есть в разных хмл...
JF
Да, ксяоми
Там флаг какой-то в манифесте ставится, чтобы forceDarkMode не включать
C
Там флаг какой-то в манифесте ставится, чтобы forceDarkMode не включать
Пробовал ставить, телефон игнорирует
ОА
Ребя, всем доброго дня. Выложил на github самое тривиальное приложение - ”список дел”, мой первый опыт с mvvm, у кого есть желание провести codereview? У опытного разработчика это займет меньше минуты. Заранее всем спасибо！
https://github.com/IgS279/TODO-MVVM
https://github.com/IgS279/TODO-MVVM
зачем тебе больше 1 активити? перепиши на сингл активити
JF
По памяти пишу, возможны неточности
L2
я чет пропустил ?
C
Ксяоми вообще наплевать на многие флаги и настройки
JF
Пробовал ставить, телефон игнорирует
Какой флаг ставили?
ОА
Ребя, всем доброго дня. Выложил на github самое тривиальное приложение - ”список дел”, мой первый опыт с mvvm, у кого есть желание провести codereview? У опытного разработчика это займет меньше минуты. Заранее всем спасибо！
https://github.com/IgS279/TODO-MVVM
https://github.com/IgS279/TODO-MVVM
supportActionBar?.title = "NewTODO"
supportActionBar?.setDisplayHomeAsUpEnabled(true)
supportActionBar?.setDisplayShowHomeEnabled(true)
не хардкодь. юзай стринг. ресурсы и вообще сделай через апплай. вся красота котлина в том, чтобы не писать 3 раза имя переменной
supportActionBar?.setDisplayHomeAsUpEnabled(true)
supportActionBar?.setDisplayShowHomeEnabled(true)
не хардкодь. юзай стринг. ресурсы и вообще сделай через апплай. вся красота котлина в том, чтобы не писать 3 раза имя переменной
ОА
Ребя, всем доброго дня. Выложил на github самое тривиальное приложение - ”список дел”, мой первый опыт с mvvm, у кого есть желание провести codereview? У опытного разработчика это займет меньше минуты. Заранее всем спасибо！
https://github.com/IgS279/TODO-MVVM
https://github.com/IgS279/TODO-MVVM
_myNotes не надо так делать в андроид. мы не веб
ОА
да кароче типичный код новичка
IS
_myNotes не надо так делать в андроид. мы не веб
спасибо за ваши замечания！
ОА
спасибо за ваши замечания！
не пушь закомментированный код в гитхаб
ОА
NotesFragment тебе не кажется что многое нужно перебросить из фрагмента в вм/?
ОА
143 строчки в фрагменте и 35 в вм, где 7 из них пустота
d
ну типа если ты заново создашь контейнер с фрагментами то и те заново создадутся. не?
я вообще не пойму, он же пересоздался, значит и все фрагменты в его контейнеры должны пересоздаться,
ОА
спасибо за ваши замечания！
айси лончер пнг лежит вообще не там где надо ему лежать