implement advice from submission 1

Author: riyandifirman

app/src/main/java/com/riyandifirman/githubuser/adapter/SectionsPagerAdapter.kt

@@ -19,9 +19,7 @@ class SectionsPagerAdapter(private val mContext: Context, fManager: FragmentMana
     private val TAB_TITLES = intArrayOf(R.string.tab_a, R.string.tab_b)
 
     // fungsi untuk mengembalikan jumlah tab
-    override fun getCount(): Int {
-        return 2
-    }
+    override fun getCount(): Int = TAB_TITLES.size
 
     // fungsi untuk mengembalikan fragment yang akan ditampilkan pada tab
     override fun getItem(position: Int): Fragment {

app/src/main/java/com/riyandifirman/githubuser/ui/detail/DetailUserActivity.kt

@@ -1,5 +1,6 @@
 package com.riyandifirman.githubuser.ui.detail
 
+import android.nfc.NfcAdapter.EXTRA_ID
 import androidx.appcompat.app.AppCompatActivity
 import android.os.Bundle
 import android.view.View
@@ -20,13 +21,6 @@ class DetailUserActivity : AppCompatActivity() {
     private lateinit var binding: ActivityDetailUserBinding
     private lateinit var viewModel: DetailUserViewModel
 
-    // companion object digunakan untuk membuat variabel yang dapat diakses dari mana saja
-    companion object {
-        const val EXTRA_USERNAME = "extra_username"
-        const val EXTRA_ID = "extra_id"
-        const val EXTRA_AVATAR = "extra_avatar"
-    }
-
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
         binding = ActivityDetailUserBinding.inflate(layoutInflater)
@@ -106,13 +100,12 @@ class DetailUserActivity : AppCompatActivity() {
     }
 
     // fungsi untuk menampilkan progress bar
-    private fun showLoading(isLoading: Boolean) {
-        if (isLoading) {
-            // jika isLoading true maka progress bar akan muncul
-            binding.progressBar.visibility = View.VISIBLE
-        } else {
-            // jika isLoading false maka progress bar akan hilang
-            binding.progressBar.visibility = View.GONE
-        }
+    private fun showLoading(state: Boolean) { binding.progressBar.visibility = if (state) View.VISIBLE else View.GONE }
+
+    // companion object digunakan untuk membuat variabel yang dapat diakses dari mana saja
+    companion object {
+        const val EXTRA_USERNAME = "extra_username"
+        const val EXTRA_ID = "extra_id"
+        const val EXTRA_AVATAR = "extra_avatar"
     }
 }

app/src/main/java/com/riyandifirman/githubuser/ui/main/MainActivity.kt

@@ -80,15 +80,7 @@ class MainActivity : AppCompatActivity() {
     }
 
     // fungsi untuk menampilkan progress bar
-    private fun showLoading(isLoading: Boolean) {
-        if (isLoading) {
-            // jika isLoading true maka progress bar akan muncul
-            binding.progressBar.visibility = View.VISIBLE
-        } else {
-            // jika isLoading false maka progress bar akan hilang
-            binding.progressBar.visibility = View.GONE
-        }
-    }
+    private fun showLoading(state: Boolean) { binding.progressBar.visibility = if (state) View.VISIBLE else View.GONE }
 
     // fungsi untuk menampilkan option menu search
     override fun onCreateOptionsMenu(menu: Menu): Boolean {

app/src/main/java/com/riyandifirman/githubuser/viewmodel/DetailUserViewModel.kt

@@ -2,6 +2,7 @@ package com.riyandifirman.githubuser.viewmodel
 
 import android.app.Application
 import android.util.Log
+import android.widget.Toast
 import androidx.lifecycle.AndroidViewModel
 import androidx.lifecycle.LiveData
 import androidx.lifecycle.MutableLiveData
@@ -48,7 +49,8 @@ class DetailUserViewModel(application: Application) : AndroidViewModel(applicati
 
             // jika gagal
             override fun onFailure(call: Call<DetailUserResponse>, t: Throwable) {
-                Log.d("ERROR", t.message.toString())
+                // menampilkan pesan error dengan toast
+                Toast.makeText(getApplication(), t.message, Toast.LENGTH_SHORT).show()
             }
         })
     }

app/src/main/java/com/riyandifirman/githubuser/viewmodel/MainViewModel.kt

@@ -14,10 +14,7 @@ import retrofit2.Response
 class MainViewModel(private val preferences: SettingsPreferences) : ViewModel() {
 
     // companion object digunakan untuk membuat variabel yang dapat diakses tanpa harus membuat objek
-    companion object {
-        private const val ERROR = "Failure"
-        private const val USERNAME = "riyandi"
-    }
+
 
     // fungsi yang akan dijalankan pertama kali ketika kelas ini diinisiasi
     init {
@@ -81,4 +78,9 @@ class MainViewModel(private val preferences: SettingsPreferences) : ViewModel()
     class Factory(private val preferences: SettingsPreferences) : ViewModelProvider.NewInstanceFactory() {
         override fun <T : ViewModel> create(modelClass: Class<T>): T = MainViewModel(preferences) as T
     }
+
+    companion object {
+        private const val ERROR = "Failure"
+        private const val USERNAME = "riyandi"
+    }
 }