Ce site est optimisé pour être consulté depuis un navigateur moderne dans lequel JavaScript est activé.

How to Stop Animation By name?

Captain510

Hi.Nate.I got a problem when use your spine runtime with cocos2d-x, I need to stop animation by its name,
But I check the code of CCSkeletonAnimation and CCSkeleton:

	void setAnimation (const char* name, bool loop, int stateIndex = 0);
	void addAnimation (const char* name, bool loop, float delay = 0, int stateIndex = 0);
	void clearAnimation (int stateIndex = 0);

clearAnimation just stop all animation in queue, then I want to implement one by my own, so I check the code
inside, I knew I just need find animation by its name, and clear it from the queue, here is my code:

void AnimationState_clearAnimationByName(AnimationState* self, const char* animationName)
{
	Animation* animation = animationName ? SkeletonData_findAnimation(self->data->skeletonData, animationName) : 0;
	if(NULL == animation)
		return ;
	_Internal* internal = SUB_CAST(_Internal, self);
	_Entry* entry = internal->queue;
	if(NULL == entry)
		return ;
	//if is first one in queue, clear it and return
	if(entry->animation == animation)
	{
		internal->queue = entry->next;
		return ;
	}
	while (entry) 
	{
		_Entry* nextEntry = entry->next;
		if(nextEntry->animation == animation)
		{
			_Entry* nnextEntry = nextEntry->next;
			entry->next = nnextEntry;
			FREE(entry);
			break;
		}
		entry = nextEntry;
	}
}

But I got null ptr in line (_Entry* entry = internal->queue😉, can you give some code guide how to find the playing animation in queue?

Very appreciated.

Nate

Seems strange to need to clear from the queue. setAnimation clears the queue and adds a new animation. Sounds like whatever you passed for self is null. You can try to adapt this completely untested code:

void _AnimationState_removeQueueAnimation (AnimationState* self, Animation* animation) {
	_Internal* internal = SUB_CAST(_Internal, self);
	_Entry* entry = internal->queue;
	if (!entry) return;
	if (entry->animation == animation) {
		internal->queue = entry->next;
		FREE(entry);
		return;
	}
	while (entry->next) {
		_Entry* nextEntry = entry->next;
		if (nextEntry->animation == animation) {
			entry->next = nextEntry->next;
			FREE(nextEntry);
			return;
		}
		entry = nextEntry;
	}
}

Captain510

Thanks for the quick reply. Honest saying your code is not much different from mine 🙂

Sounds like whatever you passed for self is null.

from debug watch, your guess is right.
Internal* internal = SUB_CAST(Internal, self);
I check the variable data inside internal , the queue var is null. So I wonder maybe I just not figure out what inside the spine logic(Sorry, I'm from 3d games..in those engines..animation has a lot of combinations)

What I did before is just play a loop running animation. and when I stop moving, call stop animation. But I can not call clear all animation, because I write play/stop running animation code in player's tick function(I have to check player velocity every tick to determine whether player stop or start running), but player play fight animation instantly, which will like: play fight animation==>next tick velocity is zero, so call stop animation.then the fight animation will be cleared.

Nate

internal->queue can be null if the queue is empty.

You could use addAnimation("stop") and have logic in your game to avoid adding stop many times. Eg if velocity changes from > 0 to 0 only then call addAnimation("stop"). If fight is playing, stop will happen after. If nothing is playing stop will happen right away. If playing some animation other than fight and you want stop to happen right away, use setAnimation("stop") instead.

Captain510

Nate a écrit
internal->queue can be null if the queue is empty.

You could use addAnimation("stop") and have logic in your game to avoid adding stop many times. Eg if velocity changes from > 0 to 0 only then call addAnimation("stop"). If fight is playing, stop will happen after. If nothing is playing stop will happen right away. If playing some animation other than fight and you want stop to happen right away, use setAnimation("stop") instead.

Thanks it works, I just add a non-existed animation called "Stop", I think you can make a more clearer function to do this 🙂