New Touch system issues « cocos2d for iPhone

cocos2d for iPhone » Programming » cocos2d (graphics engine)

New Touch system issues

(14 posts) (6 voices)

Started 8 months ago by dnsolo
Latest reply from remover

Tags:

dnsolo
Member

Hi all,

I'm having problems detecting sprite touches using the new TouchDispatcher system.

The code below doesn't work and no touch is detected, am I doing something wrong here?

[self schedule:@selector(tick:)];

- (void)tick:(ccTime)delta
{
[self touchHandler];
}

- (BOOL)touchHandler {
if(mySprite.state == kMenuOptionStateGrabbed) {
NSLog(@"Sprite has been Touched");
}
return kEventHandled;
}

In my ccTouchBegan method code I have...

- (BOOL)ccTouchBegan:(UITouch *)touch withEvent:(UIEvent *)event
{
state = kMenuOptionStateGrabbed;
return kEventHandled;
}

Thanks,

Dan

Posted 8 months ago #
emalamisura
Member

You need to register the sprite for a targeted touch:

[[TouchDispatcher sharedDispatcher] addTargetedDelegate:self priority:0 swallowsTouches:YES];

Make sure you subscribe to the Touch Delegate protocol as well...

Posted 8 months ago #
dnsolo
Member

Hi,

Yes, I'm doing both those things and it's still not detecting any touches.

Any other ideas?

Thanks,

Dan

Posted 8 months ago #
emalamisura
Member

The other issues I ran into was that I had multiple layers and one of my upper layers was return kEventHandled which prevent further delegation, or one of my upper layers was swallowingTouches...

In addition make sure you are using the proper method names for the TargetedTouchDelegate, they do not end with 'es' so it would be ccTouchBegan versus the untargeted touch delegate (forget exact name), which end with 'es' such as ccTouchesBegan...

Also make sure in your ApplicationDelegate (entry point) that you are enabling touches for the entire application...

That's all I can think of atm...Hope one of those helps!

Posted 8 months ago #
jd
Member

You still have enable touches on each layer too, right?

Posted 8 months ago #
dnsolo
Member

Hi,

I've done everything suggested but still no joy.

I've based my own code on the Pong demo supplied in v0.8 and so there cannot be too much wrong with it.

To recap, I've done all of the following:

// Added in my AppDelegate class
[[TouchDispatcher sharedDispatcher] setDispatchEvents: YES];

// Implemented the correct method in my main scene and layer class
- (BOOL)ccTouchBegan:(UITouch *)touch withEvent:(UIEvent *)event
{
if(mySprite1.state == kMenuOptionStateGrabbed) {
NSLog(@"SPRITE TOUCHED");
return kEventHandled;
}
return kEventIgnored;
}

// In my class that sub classes TextureNode (like the Pong demo does) the relevant handling methods are:

- (BOOL)ccTouchBegan:(UITouch *)touch withEvent:(UIEvent *)event
{
state = kMenuSpriteStateGrabbed;
return kEventIgnored;
}

- (void)ccTouchEnded:(UITouch *)touch withEvent:(UIEvent *)event
{
state = kMenuSpriteStateUngrabbed;
}

Additional notes...

I've debugged the app and the 'onEnter' method is definitely being called before each sprite is added to the layer.
My implementation for this method is:

- (void)onEnter
{
[[TouchDispatcher sharedDispatcher] addTargetedDelegate:self priority:0 swallowsTouches:NO];
[super onEnter];
}

Note, it doesn't matter if 'swallowsTouches' is set to YES or NO in this method, it still doesn't work.

Dan

Posted 8 months ago #
emalamisura
Member

don't know what to tell you, everything mentioned should solve the issue so now its up to you to find it...

Posted 8 months ago #
dnsolo
Member

Hi all,

I've literally tried everything advised but cannot get the new Touch system to work.

I have decided to use the 'old' touch system as follows:

- (BOOL)ccTouchesBegan:(NSSet *)touches withEvent:(UIEvent *)event {
UITouch * touch = [touches anyObject];
CGPoint location = [[Director sharedDirector] convertCoordinate: [touch locationInView:touch.view]];
CGRect spriteRect = CGRectMake(mySprite1.position.x, mySprite1.position.y,
mySprite1.contentSize.width, mySprite1.contentSize.height);

if(CGRectContainsPoint(mySprite1, location)) {
// do stuff
return kEventHandled;
}
return kEventIgnored;
}

This code works fine and will enable me to finish my game. It's not ideal though and I really wanted to use
the new Touch system which I know would make things much easier to manage.

I think what is needed here is another version of the Pong demo, one that uses 'AtlasSprite' for the 'Paddle' instead of a 'TextureNode' ... and one where several sprites are placed into an NSArray and where each sprite can receive touch events using
the new TouchDispatcher system.

cheers,

dan

Posted 8 months ago #
riq
Key Master
The easiest way to use the new touch system is to create a subclass of Layer (like in the old touch system) and add this method:
```
-(void) registerWithTouchDispatcher
     {
        [[TouchDispatcher sharedDispatcher] addTargetedDelegate:self priority:INT_MIN+1 swallowsTouches:YES];
     }
```
Now, instead of receiving multiple touches, you will receive just 1 touch.

See cocos2d/menu.m to see how it handle the touches by overriding registerWithTouchDispatcher
Posted 8 months ago #
dnsolo
Member

Hi Riq,

I've tried this approach and can now successfully receive touches, thanks.

But how can I detect when a certain sprite is touched using the new touch system and avoid
the messy way I used to do this as shown in my previous post?

- (BOOL)ccTouchBegan:(UITouch *)touch withEvent:(UIEvent *)event
{
// How can I detect which sprite was touched?
return YES;
}

Thanks,

Dan

Posted 8 months ago #
riq
Key Master

@dnsolo: see cocos2d/menu.m to see how it detects if a touch is inside a menuitem.

also, tests/TouchDemo uses a similar approach.

Posted 8 months ago #

shivaz
Member

@dnsolo: If you grab the demo from this post: http://www.cocos2d-iphone.org/forum/topic/364

You can change it in the following ways - so that it uses the new touch system.
In the MotionLayer.m - delete all the touch methods.
In the MySprite.h change the interface to the following:

typedef enum tagSpriteState {
	kSpriteStateGrabbed,
	kSpriteStateUngrabbed
} SpriteState;

@interface MySprite : AtlasSprite <TargetedTouchDelegate>
{
	bool ready;
	BOOL canTrack; //tell us if a Sprite in the Array can be tracked.
	float spriteSize;
	CGPoint touchStartPosition;
	@private SpriteState state;
}

Then in MySprite.m do the following:
Replace rect method with:

- (CGRect) rect {// rectangle that contains the sides coords of each sprite.
	float s = self.spriteSize;
	return CGRectMake(-s / 2, -s / 2, s, s);
}

Then add the following methods:

- (void)onEnter
{
	[[TouchDispatcher sharedDispatcher] addTargetedDelegate:self priority:0 swallowsTouches:YES];
	[super onEnter];
}

- (void)onExit
{
	[[TouchDispatcher sharedDispatcher] removeDelegate:self];
	[super onExit];
}	

- (BOOL)containsTouchLocation:(UITouch *)touch
{
	return CGRectContainsPoint(self.rect, [self convertTouchToNodeSpaceAR:touch]);
}

- (BOOL)ccTouchBegan:(UITouch *)touch withEvent:(UIEvent *)event
{
	if (state != kSpriteStateUngrabbed) return NO;
	if ( ![self containsTouchLocation:touch] ) return NO;
	state = kSpriteStateGrabbed;
	return YES;
}

- (void)ccTouchMoved:(UITouch *)touch withEvent:(UIEvent *)event
{
	NSAssert(state == kSpriteStateGrabbed, @"Unexpected state!");	

	CGPoint touchPoint = [touch locationInView:[touch view]];
	touchPoint = [[Director sharedDirector] convertCoordinate:touchPoint];

	self.position = CGPointMake(touchPoint.x, self.position.y);
}

- (void)ccTouchEnded:(UITouch *)touch withEvent:(UIEvent *)event
{
	NSAssert(state == kSpriteStateGrabbed, @"Unexpected state!");	

	state = kSpriteStateUngrabbed;
}

That should do it. All basically straight from the TouchDemo. Only difference is that I use a uniform size for the sprite - spriteSize - which is set when the sprite is created. You could be less general in your rect function.

Justin

Posted 8 months ago #

dnsolo
Member

Hi Justin,

That's very helpful, many thanks.

I will implement this code and post back my results.

cheers,

dan

Posted 8 months ago #
remover
Member

could somebody please tell me why this bit of code identifies the correct rectangle (the origin is a minus number!?)

- (CGRect) rect {// rectangle that contains the sides coords of each sprite.
float s = self.spriteSize;
return CGRectMake(-s / 2, -s / 2, s, s);
}

surely we want the origin to be (0,0)??

Posted 1 month ago #

RSS feed for this topic

Reply

You must log in to post.