Skip to content

Commit ee5b507

Browse files
committed
Refactor CircleMenuView.onClick method
1 parent 10bdc5e commit ee5b507

File tree

1 file changed

+17
-12
lines changed

1 file changed

+17
-12
lines changed

circle-menu/src/main/java/com/ramotion/circlemenu/CircleMenuView.java

Lines changed: 17 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -211,23 +211,28 @@ public void onClick(View view) {
211211
animation.addListener(new AnimatorListenerAdapter() {
212212
@Override
213213
public void onAnimationStart(Animator animation) {
214-
if (mListener != null) {
215-
if (mClosedState) {
216-
mListener.onMenuOpenAnimationStart(CircleMenuView.this);
217-
} else {
218-
mListener.onMenuCloseAnimationStart(CircleMenuView.this);
219-
}
214+
if (mListener == null) {
215+
return;
216+
}
217+
218+
if (mClosedState) {
219+
mListener.onMenuOpenAnimationStart(CircleMenuView.this);
220+
} else {
221+
mListener.onMenuCloseAnimationStart(CircleMenuView.this);
220222
}
221223
}
222224
@Override
223225
public void onAnimationEnd(Animator animation) {
224226
animation.removeListener(this);
225-
if (mListener != null) {
226-
if (mClosedState) {
227-
mListener.onMenuOpenAnimationEnd(CircleMenuView.this);
228-
} else {
229-
mListener.onMenuCloseAnimationEnd(CircleMenuView.this);
230-
}
227+
228+
if (mListener == null) {
229+
return;
230+
}
231+
232+
if (mClosedState) {
233+
mListener.onMenuOpenAnimationEnd(CircleMenuView.this);
234+
} else {
235+
mListener.onMenuCloseAnimationEnd(CircleMenuView.this);
231236
}
232237
}
233238
});

0 commit comments

Comments
 (0)