Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Cannot remove ViewPropertyAnimator's listener

When I run this code inside my custom view, onAnimationStart and onAnimationEnd are kept being called repeatedly. Isn't that weird? As an Android programmer, I expected them to be called only once respectively.

    final ViewPropertyAnimator animator = animate().setDuration(1000).alpha(0.0f);
    animator.setListener(new AnimatorListenerAdapter() {
        @Override
        public void onAnimationStart(Animator animation) {
            Utils.log("----------------start");
        }

        @Override
        public void onAnimationEnd(Animator animation) {
            Utils.log("--------- end");
        }
    }).start();

But then I tried to fix the problem by removing the listener when onAnimationEnd gets called by ViewPropertyAnimator's setListener(null) but it never worked despite what's written in the docs:

public ViewPropertyAnimator setListener (Animator.AnimatorListener listener)

Added in API level 12
Sets a listener for events in the underlying Animators that run the property animations.

Parameters
listener    The listener to be called with AnimatorListener events. A value of null removes any existing listener.
Returns
This object, allowing calls to methods in this class to be chained.

Has anyone else run into this weird problem? maybe it's an Android's bug?

like image 957
Leo Avatar asked Apr 02 '15 07:04

Leo


1 Answers

I just ran into this issue but without the custom view.

In my case, I had two animations on the same view. A show and hide.

So it was

showView(){
  myView.animate().translationY(myView.getHeight()).setListener(new ...{
    ...
    onAnimationEnd(Animation animation){
     hideView();
    }
    ...}).start();
}
hideView(){
  myView.animate().translationY(0).start();
}

When hideView() finished, it would call itself again. This is because the old listener was still set. The key to fixing it ended up being to set the listener to null in the second animation. e.g.

hideView(){
  myView.animate().translationY(0).setListener(null).start();
}
like image 79
Jon F Hancock Avatar answered Nov 08 '22 23:11

Jon F Hancock