解决部分已知bug，优化代码

jdsjlzx · Jan 30, 2018 · dca2dfe · dca2dfe
1 parent a771079
commit dca2dfe
Show file tree

Hide file tree

Showing 20 changed files with 421 additions and 61 deletions.
diff --git a/LRecyclerview_library/src/main/java/com/github/jdsjlzx/recyclerview/LRecyclerView.java b/LRecyclerview_library/src/main/java/com/github/jdsjlzx/recyclerview/LRecyclerView.java
@@ -592,6 +592,25 @@ public void onScrolled(int dx, int dy) {
             mLScrollListener.onScrolled(mScrolledXDistance, mScrolledYDistance);
         }
 
+        if (mLoadMoreListener != null && mLoadMoreEnabled) {
+            int visibleItemCount = layoutManager.getChildCount();
+            int totalItemCount = layoutManager.getItemCount();
+            if (visibleItemCount > 0
+                    && lastVisibleItemPosition >= totalItemCount - 1
+                    && totalItemCount > visibleItemCount
+                    && !isNoMore
+                    && !mRefreshing) {
+
+                mFootView.setVisibility(View.VISIBLE);
+                if (!mLoadingData) {
+                    mLoadingData = true;
+                    mLoadMoreFooter.onLoading();
+                }
+
+            }
+
+        }
+
     }
 
     @Override
@@ -603,30 +622,7 @@ public void onScrollStateChanged(int state) {
             mLScrollListener.onScrollStateChanged(state);
         }
 
-        if (mLoadMoreListener != null && mLoadMoreEnabled) {
-            if (currentScrollState == RecyclerView.SCROLL_STATE_IDLE) {
-                RecyclerView.LayoutManager layoutManager = getLayoutManager();
-                int visibleItemCount = layoutManager.getChildCount();
-                int totalItemCount = layoutManager.getItemCount();
-                if (visibleItemCount > 0
-                        && lastVisibleItemPosition >= totalItemCount - 1
-                        && totalItemCount > visibleItemCount
-                        && !isNoMore
-                        && !mRefreshing) {
-
-                    mFootView.setVisibility(View.VISIBLE);
-                    if (mLoadingData) {
-                        return;
-                    } else {
-                        mLoadingData = true;
-                        mLoadMoreFooter.onLoading();
-                        mLoadMoreListener.onLoadMore();
-                    }
 
-                }
-
-            }
-        }
 
     }
 

diff --git a/...yclerview_library/src/main/java/com/github/jdsjlzx/recyclerview/LRecyclerViewAdapter.java b/...yclerview_library/src/main/java/com/github/jdsjlzx/recyclerview/LRecyclerViewAdapter.java
@@ -320,10 +320,10 @@ public void onViewDetachedFromWindow(RecyclerView.ViewHolder holder) {
         mInnerAdapter.onViewDetachedFromWindow(holder);
     }
 
-    @Override
+    /*@Override
     public void onViewRecycled(RecyclerView.ViewHolder holder) {
         mInnerAdapter.onViewRecycled(holder);
-    }
+    }*/
 
 
     public static class ViewHolder extends RecyclerView.ViewHolder {

diff --git a/LRecyclerview_library/src/main/java/com/github/jdsjlzx/recyclerview/LuRecyclerView.java b/LRecyclerview_library/src/main/java/com/github/jdsjlzx/recyclerview/LuRecyclerView.java
@@ -397,6 +397,24 @@ public void onScrolled(int dx, int dy) {
             mLScrollListener.onScrolled(mScrolledXDistance, mScrolledYDistance);
         }
 
+        if (mLoadMoreListener != null && mLoadMoreEnabled) {
+            int visibleItemCount = layoutManager.getChildCount();
+            int totalItemCount = layoutManager.getItemCount();
+            if (visibleItemCount > 0
+                    && lastVisibleItemPosition >= totalItemCount - 1
+                    && totalItemCount > visibleItemCount
+                    && !isNoMore
+                    && !mRefreshing) {
+
+                mFootView.setVisibility(View.VISIBLE);
+                if (!mLoadingData) {
+                    mLoadingData = true;
+                    mLoadMoreFooter.onLoading();
+                }
+
+            }
+
+        }
     }
 
     @Override
@@ -407,32 +425,6 @@ public void onScrollStateChanged(int state) {
         if (mLScrollListener != null) {
             mLScrollListener.onScrollStateChanged(state);
         }
-
-        if (mLoadMoreListener != null && mLoadMoreEnabled) {
-            if (currentScrollState == RecyclerView.SCROLL_STATE_IDLE) {
-                RecyclerView.LayoutManager layoutManager = getLayoutManager();
-                int visibleItemCount = layoutManager.getChildCount();
-                int totalItemCount = layoutManager.getItemCount();
-                if (visibleItemCount > 0
-                        && lastVisibleItemPosition >= totalItemCount - 1
-                        && totalItemCount > visibleItemCount
-                        && !isNoMore
-                        && !mRefreshing) {
-
-                    mFootView.setVisibility(View.VISIBLE);
-                    if (mLoadingData) {
-                        return;
-                    } else {
-                        mLoadingData = true;
-                        mLoadMoreFooter.onLoading();
-                        mLoadMoreListener.onLoadMore();
-                    }
-
-                }
-
-            }
-        }
-
     }
 
     /**

diff --git a/LRecyclerview_library/src/main/java/com/github/jdsjlzx/view/LoadingFooter.java b/LRecyclerview_library/src/main/java/com/github/jdsjlzx/view/LoadingFooter.java
@@ -35,22 +35,22 @@ public class LoadingFooter extends RelativeLayout implements ILoadMoreFooter {
 
     public LoadingFooter(Context context) {
         super(context);
-        init(context);
+        init();
     }
 
     public LoadingFooter(Context context, AttributeSet attrs) {
         super(context, attrs);
-        init(context);
+        init();
     }
 
     public LoadingFooter(Context context, AttributeSet attrs, int defStyleAttr) {
         super(context, attrs, defStyleAttr);
-        init(context);
+        init();
     }
 
-    public void init(Context context) {
+    public void init() {
 
-        inflate(context, R.layout.layout_recyclerview_footer, this);
+        inflate(getContext(), R.layout.layout_recyclerview_footer, this);
         setOnClickListener(null);
 
         onReset();//初始为隐藏状态

diff --git a/app/app.iml b/app/app.iml
@@ -99,6 +99,7 @@
       <excludeFolder url="file://$MODULE_DIR$/build/intermediates/shaders" />
       <excludeFolder url="file://$MODULE_DIR$/build/intermediates/splits-support" />
       <excludeFolder url="file://$MODULE_DIR$/build/intermediates/symbols" />
+      <excludeFolder url="file://$MODULE_DIR$/build/intermediates/tmp" />
       <excludeFolder url="file://$MODULE_DIR$/build/intermediates/transforms" />
       <excludeFolder url="file://$MODULE_DIR$/build/outputs" />
       <excludeFolder url="file://$MODULE_DIR$/build/tmp" />

diff --git a/app/build.gradle b/app/build.gradle
@@ -23,10 +23,10 @@ dependencies {
     implementation fileTree(include: ['*.jar'], dir: 'libs')
     implementation 'com.android.support:cardview-v7:27.0.2'
     implementation 'com.android.support.constraint:constraint-layout:1.0.2'
-    compile 'com.github.bumptech.glide:glide:4.3.1'
-    annotationProcessor 'com.github.bumptech.glide:compiler:4.1.1'
+    implementation 'com.github.bumptech.glide:glide:4.3.1'
+    annotationProcessor 'com.github.bumptech.glide:compiler:4.3.1'
     implementation project(path: ':LRecyclerview_library')
     implementation 'com.nineoldandroids:library:2.4.0'
-    compile 'com.github.jdsjlzx:banner:1.0.0'
+    implementation 'com.github.jdsjlzx:banner:1.0.0'
     implementation 'com.android.support:design:27.0.2'
 }
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
@@ -52,7 +52,8 @@
         <activity android:name=".ui.WebViewActivity" />
         <activity android:name=".ui.LinearLayoutDelActivity" />
         <activity android:name=".ui.CoordinatorLayoutActivity" />
-        <activity android:name=".ui.AlipayHomeActivity"></activity>
+        <activity android:name=".ui.AlipayHomeActivity" />
+        <activity android:name=".ui.AlphaChangeActivity" />
     </application>
 
 </manifest>
diff --git a/app/src/main/java/com/lzx/demo/adapter/DataAdapter.java b/app/src/main/java/com/lzx/demo/adapter/DataAdapter.java
@@ -1,6 +1,9 @@
 package com.lzx.demo.adapter;
 
 import android.content.Context;
+import android.text.Editable;
+import android.text.TextWatcher;
+import android.widget.EditText;
 import android.widget.TextView;
 
 import com.lzx.demo.R;
@@ -28,6 +31,32 @@ public void onBindItemHolder(SuperViewHolder holder, int position) {
         ItemModel item = mDataList.get(position);
 
         TextView titleText = holder.getView(R.id.info_text);
+        EditText titleEditor = holder.getView(R.id.info_edtor);
         titleText.setText(item.title);
+        titleEditor.addTextChangedListener(textWatcher);
     }
+
+    @Override
+    public void onViewRecycled(SuperViewHolder holder) {
+        super.onViewRecycled(holder);
+        EditText titleEditor = holder.getView(R.id.info_edtor);
+        titleEditor.removeTextChangedListener(textWatcher);
+    }
+
+    TextWatcher textWatcher = new TextWatcher() {
+        @Override
+        public void beforeTextChanged(CharSequence s, int start, int count, int after) {
+
+        }
+
+        @Override
+        public void onTextChanged(CharSequence s, int start, int before, int count) {
+
+        }
+
+        @Override
+        public void afterTextChanged(Editable s) {
+
+        }
+    };
 }
diff --git a/app/src/main/java/com/lzx/demo/ui/AlipayHomeActivity.java b/app/src/main/java/com/lzx/demo/ui/AlipayHomeActivity.java
@@ -7,6 +7,7 @@
 import android.support.v7.app.AppCompatActivity;
 import android.os.Bundle;
 import android.support.v7.widget.LinearLayoutManager;
+import android.support.v7.widget.RecyclerView;
 import android.util.Log;
 import android.view.View;
 
@@ -37,6 +38,7 @@ public class AlipayHomeActivity extends AppCompatActivity implements AppBarLayou
     private LRecyclerView mRecyclerView = null;
     private LRecyclerViewAdapter mLRecyclerViewAdapter = null;
     private DataAdapter mDataAdapter = null;
+
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
@@ -101,6 +103,7 @@ public void run() {
 
     }
 
+    //效果不是很完美
     @Override
     public void onOffsetChanged(AppBarLayout appBarLayout, int verticalOffset) {
         Log.d("lzx", "verticalOffset=" + verticalOffset);