Skip to content

Commit 42618e0

Browse files
committed
fix bugs
1 parent 0833e56 commit 42618e0

File tree

9 files changed

+49
-5
lines changed

9 files changed

+49
-5
lines changed

app/src/main/java/com/code4a/jlibrarydemo/home/frag/home/two/girls/GirlsFragment.java

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -147,10 +147,17 @@ public void showNormal() {
147147
}
148148
}
149149

150+
@Override
151+
public void onPause() {
152+
super.onPause();
153+
mPresenter.cancel();
154+
}
155+
150156
@Override
151157
public void onDestroyView() {
152158
super.onDestroyView();
153-
mPresenter.cancel();
159+
mPresenter.releaseRes();
160+
mPresenter = null;
154161
// unbinder.unbind();
155162
}
156163
}

app/src/main/java/com/code4a/jlibrarydemo/home/frag/home/two/girls/GirlsPresenter.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,4 +17,6 @@ protected GirlsPresenter(GirlsView view) {
1717
public abstract void getGirls(int page, int size, boolean isRefresh);
1818

1919
public abstract void cancel();
20+
21+
public abstract void releaseRes();
2022
}

app/src/main/java/com/code4a/jlibrarydemo/home/frag/home/two/girls/GirlsPresenterImpl.java

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ protected GirlsPresenterImpl(GirlsView view) {
1919
}
2020

2121
@Override
22-
public void start(){
22+
public void start() {
2323
getGirls(1, 20, true);
2424
}
2525

@@ -28,17 +28,17 @@ public void getGirls(int page, int count, final boolean isRefresh) {
2828
gankCommonInteractor.getRes(Constants.FULI, page, count, new GankCommonInteractor.LoadSplashResListener() {
2929
@Override
3030
public void onResLoaded(GirlsBean girlsBean) {
31-
if(isRefresh){
31+
if (isRefresh) {
3232
getView().refresh(girlsBean.getResults());
33-
}else{
33+
} else {
3434
getView().load(girlsBean.getResults());
3535
}
3636
getView().showNormal();
3737
}
3838

3939
@Override
4040
public void onDataNotAvailable() {
41-
if(isRefresh) {
41+
if (isRefresh) {
4242
getView().showError();
4343
}
4444
}
@@ -48,6 +48,11 @@ public void onDataNotAvailable() {
4848
@Override
4949
public void cancel() {
5050
gankCommonInteractor.cancel();
51+
}
52+
53+
@Override
54+
public void releaseRes() {
5155
release();
56+
gankCommonInteractor = null;
5257
}
5358
}

app/src/main/java/com/code4a/jlibrarydemo/home/frag/hot/HotFragment.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,4 +102,12 @@ private void loadDefaultImg() {
102102
.into(imageViews.get(i));
103103
}
104104
}
105+
106+
@Override
107+
public void onDestroyView() {
108+
super.onDestroyView();
109+
mHotPresenter.releaseRes();
110+
mHotPresenter = null;
111+
}
112+
105113
}

app/src/main/java/com/code4a/jlibrarydemo/home/frag/hot/HotPresenter.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,4 +15,6 @@ protected HotPresenter(HotView view) {
1515
public abstract void start(int page, int size);
1616

1717
public abstract void cancel();
18+
19+
public abstract void releaseRes();
1820
}

app/src/main/java/com/code4a/jlibrarydemo/home/frag/hot/HotPresenterImpl.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,11 @@ public void onDataNotAvailable() {
3939
@Override
4040
public void cancel() {
4141
gankCommonInteractor.cancel();
42+
}
43+
44+
@Override
45+
public void releaseRes() {
4246
release();
47+
gankCommonInteractor = null;
4348
}
4449
}

app/src/main/java/com/code4a/jlibrarydemo/splash/SplashFragment.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -108,4 +108,11 @@ public void onPause() {
108108
super.onPause();
109109
mPresenter.cancel();
110110
}
111+
112+
@Override
113+
public void onDestroyView() {
114+
super.onDestroyView();
115+
mPresenter.releaseRes();
116+
mPresenter = null;
117+
}
111118
}

app/src/main/java/com/code4a/jlibrarydemo/splash/SplashPresenter.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,4 +15,6 @@ protected SplashPresenter(SplashView view) {
1515
public abstract void start();
1616

1717
public abstract void cancel();
18+
19+
public abstract void releaseRes();
1820
}

app/src/main/java/com/code4a/jlibrarydemo/splash/SplashPresenterImpl.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,12 @@ public void onDataNotAvailable() {
3838
@Override
3939
public void cancel() {
4040
gankCommonInteractor.cancel();
41+
}
42+
43+
@Override
44+
public void releaseRes() {
4145
release();
46+
gankCommonInteractor = null;
4247
}
48+
4349
}

0 commit comments

Comments
 (0)