Adding loader to MVP example

myotive · Dec 27, 2016 · def5394 · def5394
1 parent 5925058
commit def5394
Show file tree

Hide file tree

Showing 5 changed files with 34 additions and 1 deletion.
diff --git a/mvpexample/src/main/java/com/example/myotive/mvpexample/SpeakerActivity.java b/mvpexample/src/main/java/com/example/myotive/mvpexample/SpeakerActivity.java
@@ -2,6 +2,8 @@
 
 import android.support.v7.app.AppCompatActivity;
 import android.os.Bundle;
+import android.view.View;
+import android.widget.ProgressBar;
 
 import com.example.myotive.codemash_common.BaseApplication;
 import com.example.myotive.codemash_common.network.CodeMashAPI;
@@ -22,6 +24,7 @@ protected void onCreate(Bundle savedInstanceState) {
         overridePendingTransition(R.anim.slide_in_right, R.anim.slide_out_left);
 
         speakerListView = (SpeakerListView)findViewById(R.id.speakerView);
+
         codeMashAPI = BaseApplication.getApplication(this).getApplicationComponent().CodeMashAPI();
 
         speakerListPresenter = new SpeakerListPresenter(codeMashAPI, speakerListView);

diff --git a/mvpexample/src/main/java/com/example/myotive/mvpexample/speakerlist/SpeakerListContract.java b/mvpexample/src/main/java/com/example/myotive/mvpexample/speakerlist/SpeakerListContract.java
@@ -13,6 +13,8 @@
 public interface SpeakerListContract {
     interface View extends BaseView<Presenter>{
         void updateSpeakerList(List<Speaker> speakers);
+        void showLoading();
+        void hideLoading();
     }
 
     interface Presenter extends BasePresenter{

diff --git a/...xample/src/main/java/com/example/myotive/mvpexample/speakerlist/SpeakerListPresenter.java b/...xample/src/main/java/com/example/myotive/mvpexample/speakerlist/SpeakerListPresenter.java
@@ -1,6 +1,8 @@
 package com.example.myotive.mvpexample.speakerlist;
 
 import android.util.Log;
+import android.view.View;
+import android.widget.ProgressBar;
 
 import com.example.myotive.codemash_common.network.CodeMashAPI;
 import com.example.myotive.codemash_common.network.models.Speaker;
@@ -28,7 +30,6 @@ public SpeakerListPresenter(CodeMashAPI codeMashAPI, SpeakerListContract.View vi
         this.codeMashAPI = codeMashAPI;
         this.view = view;
 
-
         this.view.setPresenter(this);
     }
 
@@ -46,6 +47,8 @@ public void stop() {
 
     @Override
     public void getSpeakerList() {
+
+        view.showLoading();
         speakerCall = codeMashAPI.GetSpeakers();
 
         speakerCall.enqueue(new Callback<List<Speaker>>() {
@@ -54,11 +57,13 @@ public void onResponse(Call<List<Speaker>> call, Response<List<Speaker>> respons
                 if(response.isSuccessful()){
                     view.updateSpeakerList(response.body());
                 }
+                view.hideLoading();
             }
 
             @Override
             public void onFailure(Call<List<Speaker>> call, Throwable t) {
                 Log.e(TAG, "Error calling CodeMash API", t);
+                view.hideLoading();
             }
         });
     }

diff --git a/mvpexample/src/main/java/com/example/myotive/mvpexample/speakerlist/SpeakerListView.java b/mvpexample/src/main/java/com/example/myotive/mvpexample/speakerlist/SpeakerListView.java
@@ -7,6 +7,7 @@
 import android.util.AttributeSet;
 import android.view.View;
 import android.widget.FrameLayout;
+import android.widget.ProgressBar;
 
 import com.example.myotive.codemash_common.network.models.Speaker;
 import com.example.myotive.codemash_common.ui.SpeakerAdapter;
@@ -23,6 +24,7 @@
 public class SpeakerListView extends FrameLayout implements SpeakerListContract.View {
 
 
+    private ProgressBar loader;
     private RecyclerView speakerRecyclerView;
     private SpeakerAdapter speakerAdapter;
 
@@ -58,6 +60,8 @@ public SpeakerListView(Context context, AttributeSet attrs, int defStyleAttr) {
     private void init() {
         inflate(getContext(), R.layout.view_speaker_list, this);
 
+        loader = (ProgressBar)findViewById(R.id.loading);
+
         speakerAdapter = new SpeakerAdapter(getContext(), Collections.<Speaker>emptyList(), speakerClickListener);
         speakerRecyclerView = (RecyclerView) this.findViewById(R.id.rv_speakers);
         speakerRecyclerView.setLayoutManager(new LinearLayoutManager(getContext()));
@@ -69,6 +73,18 @@ public void updateSpeakerList(List<Speaker> speakers) {
         speakerAdapter.swap(speakers);
     }
 
+    @Override
+    public void showLoading() {
+        loader.setVisibility(VISIBLE);
+        speakerRecyclerView.setVisibility(GONE);
+    }
+
+    @Override
+    public void hideLoading() {
+        loader.setVisibility(GONE);
+        speakerRecyclerView.setVisibility(VISIBLE);
+    }
+
     @Override
     public void setPresenter(SpeakerListContract.Presenter presenter) {
         this.presenter = presenter;

diff --git a/mvpexample/src/main/res/layout/view_speaker_list.xml b/mvpexample/src/main/res/layout/view_speaker_list.xml
@@ -6,6 +6,13 @@
     tools:context="com.example.myotive.mvpexample.speakerlist.SpeakerListView"
     tools:parentTag="FrameLayout">
 
+    <ProgressBar
+        android:id="@+id/loading"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_gravity="center"
+        />
+
     <android.support.v7.widget.RecyclerView
         android:id="@+id/rv_speakers"
         android:layout_width="match_parent"