programmaik / FilmPopuler

use API from The Movie DB
0 stars 0 forks source link

Feedback #1

Open crescenthikari opened 7 years ago

crescenthikari commented 7 years ago
programmaik commented 7 years ago

1.Untuk menempatkan class - class yang kita buat yang saling berhubungan dan menghindari kemungkinan terjadinya konflik penamaan class yang sama dengan class yang sudah disediakan java, dengan package struktur program akan jadi lebih rapi dan mudah dipahami. Di dalam package features.main ada beberapa package dan kelas yang digunakan di kelas lain. ada kesalahan saat membuat package harusnya cuma package features lalu didalamnya ada beberapa package, tanpa ada package main terlebih dahulu

2.Kelas TopRatedMovieListService untuk implementasi broadcast and service dalam mengambil data.

3.Karna kelas AddOrDeleteFavoriteMovieContract tidak akan digunakan untuk menginstansiasi jadi sebuah objek di kelas lain dan kelas ini hanya digunakan untuk mengakses atribute yang bersifat static

4.Mengguanaka try catch serta Exceptionuntuk menangkap Error jika terjadi Error.

5.Dengan loader proses mengambil data dapat berjalan di background, selagi UI tetap dapat menampilkan informasi yang relevan dan secara otomatis akan menampilkan data terbaru jika ada perubahan, menggunakan loader mampu juga mengatasi force close saat aplikasi dijalankan

6.Agar bisa menerapkan loader untuk mengambil data top rate movie, saya letakkan fungsi call atau kode tersebut di loader callback