task/hw8 todo_list #8
Conversation
|
|
||
| // Kotlin components | ||
| implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version" | ||
| api "org.jetbrains.kotlinx:kotlinx-coroutines-core:$rootProject.coroutines" |
| val allNotes: Flow<List<Note>> = noteDao.getListNotes() | ||
|
|
||
| @Suppress("RedundantSuspendModifier") | ||
| @WorkerThread |
There was a problem hiding this comment.
А смысл?, у тебя дао само его уже на ио запустит, нет смысла указывать тред на верхние уровни
|
|
||
| class NotesApplication : Application() { | ||
|
|
||
| val applicationScope = CoroutineScope(SupervisorJob()) |
There was a problem hiding this comment.
не оч понятно зачем это. Наверное, чтоб при первом запуске что-то было в бд, но выглядит как-то мега костыльно. Смысл скоупа в том что он зависит от жц элементов, а ты создаешь applicationScope на все приложение. Тогда мог бы и глобалскоуп использовать. Вообщем очень сомнительное решение, которое в принципе рушит идею самих скоупов
| import com.example.androidpractice2020.interfacecell.CellClickListener | ||
| import com.example.androidpractice2020.notes.Note | ||
|
|
||
| class NoteListAdapter(private val cellClickListener: CellClickListener) : androidx.recyclerview.widget.ListAdapter<Note, NoteListAdapter.NoteViewHolder>( |
There was a problem hiding this comment.
Зачем создавать интерфейс, если можно использовать лямбду. В этом был бы еще смысл, если б ты листенер инжектил в адаптер, а так намного же проще, быстрее, читабельнее использовать лямбду
| NoteListFragment.OnSelectedButtonListener, | ||
| CellClickListener { | ||
|
|
||
| private val manager = supportFragmentManager |
There was a problem hiding this comment.
Не оч понятен смысл этого
No description provided.