Incompatible Type For Argument 1 of 'CGRectContainsPoint'

cocos2d for iPhone » Programming » Programming - Everything else

[closed]

Incompatible Type For Argument 1 of 'CGRectContainsPoint'

(89 posts) (15 voices)

Started 1 year ago by Joethemonkey101
Latest reply from Steve Oldmeadow

Tags:

No tags yet.

12 3 Next »

Joethemonkey101
On probation
I'm having a lot of trouble with this. I was following a tutorial on this website a fellow member showed me, but it was outdated, so I think that's where my errors are coming from. Here's the code I have
```
-(void) spriteCheck:(UITouch *)touchLocation {
	CGPoint location = [touchLocation locationInView:[touchLocation view]];
	location = [[CCDirector sharedDirector] convertToGL:location];
	if (CGRectContainsPoint(pauseButtonRect, location)) {
	[self pauseButton];
	}
}
```
Any help would be extremely appreciated. I've researched this everywhere I could think of but couldn't find the answer.
Posted 1 year ago #
robodude666
Member

What is pauseButtonRect? How is it declared, and how is its value set?

Posted 1 year ago #

Joethemonkey101
On probation

Sorry I forgot about that.
This is in the .h

pauseButtonRect = CGRectMake((pauseButton.position.x-(pauseButton.contentSize.width)/2), (pauseButton.position.y-(pauseButton.contentSize.height)/2), (pauseButton.contentSize.width), (pauseButton.contentSize.height));

Posted 1 year ago #

araker
Moderator

@joe, some friendly advice.. please take one step back and go learn the basics of programming, object oriented programming, objective-c and debugging. From what I've seen, several times people have suggested good resources to you to learn the basics.

Without a basic understanding every error will seem like a mystery and you wouldn't even know where to begin with when searching for answers. Relying on this forum for every problem you encounter, isn't the solution.

Posted 1 year ago #
Codemattic
Moderator

if pauseButtonRect is defined as a CGRect, then I dont see the error.

"This is in the .h"

it shouldnt be. Put declarations in .h and code in .m

Posted 1 year ago #
Joethemonkey101
On probation
@Codemattic
Oh I see my mistake now. Thanks. I thought when it said " put this in the header " it meant header file (.h). Thanks for the reply!
Although now it's saying 'incompatible types in assignment'.
I put that code in the init method.
I'm also getting the original error (Incompatible type for argument 1 of 'CGRectContainsPoint') for this code.
```
-(void) spriteCheck:(UITouch *)touchLocation {
	CGPoint location = [touchLocation locationInView:[touchLocation view]];
	location = [[CCDirector sharedDirector] convertToGL:location];
	if (CGRectContainsPoint(pauseButtonRect, location)) {           <---- That line has the error
	[self pauseButton];
	}
}
```
Posted 1 year ago #
Codemattic
Moderator

@Joethemonkey101, if you want to, make a small project that just causes the error you are having and upload it somewhere and Im sure we can figure it out. Otherwise its hard to guess whats wrong. The snippet you posted looks ok to me.

Posted 1 year ago #

petrusali
Member

If I may suggest. Do like this.

-(void) spriteCheck:(UITouch *)touchLocation {
	CGPoint location = [touchLocation locationInView:[touchLocation view]];
	location = [[CCDirector sharedDirector] convertToGL:location];
        CGRect pauseButtonRect = CGRectMake((pauseButton.position.x-(pauseButton.contentSize.width)/2), (pauseButton.position.y-(pauseButton.contentSize.height)/2), (pauseButton.contentSize.width), (pauseButton.contentSize.height));
	if (CGRectContainsPoint(pauseButtonRect, location)) {
	[self pauseButton];
	}
}

Posted 1 year ago #

Joethemonkey101
On probation

Thanks I'll try that too.

Posted 1 year ago #
Joethemonkey101
On probation

I'm still getting these errors after moving that code to spriteCheck.
'incompatible types in assignment'
'incompatible type for argument 1 of 'CGRectContainsPoint'

Posted 1 year ago #

Joethemonkey101
On probation

Here's the entire file (.M)

#import "GamePlay.h"
#import "HelloWorldScene.h"

@implementation GamePlay

@synthesize walkAction = walkAction;
@synthesize mole = mole;

-(id) init {

	if( (self=[super init]) )
	{
		self.isTouchEnabled = YES;
		//background sprite
		background = [CCSprite spriteWithFile:@"Background.jpg"];
		background.position = ccp(240, 160);
		[self addChild:background z:0];
		//level 1 label
		one = [CCLabel labelWithString:@"Level 1" fontName:@"Marker Felt" fontSize:48];
		one.color = ccc3(0, 0, 255);
		id action = [CCMoveTo actionWithDuration:2 position:ccp(240, 285)];
		id action2 = [CCMoveTo actionWithDuration:2 position:ccp(550, 0)];
		[one runAction:[CCSequence actions: action, action2, nil]];
		[self addChild:one];
		//pauseButton sprite
		pauseButton = [CCSprite spriteWithFile:@"paused"];
		pauseButton.position = ccp(460, 15);
		[self addChild:pauseButton];
		//mole
		mole = [CCSprite spriteWithSpriteSheet:moleSheet rect:CGRectMake(0, 0, 0, 0)];
		mole.position = ccp(100, 100);
		[self addChild:moleSheet z:5];
		//bunny
		bunny = [CCSprite spriteWithSpriteSheet:bunnySheet rect:CGRectMake(0, 0, 0, 0)];
		bunny.position = ccp(100, 100);
		[self addChild:bunnySheet z:5];

	}
	return self;
}

-(void) one: (id) sender {

}	

-(void) background: (id) sender {

}
-(BOOL) ccTouchBegan:(UITouch *)touch withEvent:(UIEvent *)event
{
	touches = touch;
	[self spriteCheck:touches];
	return YES;
}

-(void) spriteCheck:(UITouch *)touchLocation {
	CGPoint location = [touchLocation locationInView:[touchLocation view]];
	location = [[CCDirector sharedDirector] convertToGL:location];
	pauseButtonRect = CGRectMake((pauseButton.position.x-(pauseButton.contentSize.width)/2), (pauseButton.position.y-(pauseButton.contentSize.height)/2), (pauseButton.contentSize.width), (pauseButton.contentSize.height));
	if (CGRectContainsPoint(pauseButtonRect, location)) {
	[self pauseButton];
	}
}

if (pauseButtonStatus == 0); {
	pauseButtonStatus = 1;
	[[SimpleAudioEngine sharedEngine] pauseBackgroundMusic];
	[self stopGameLoopTimer];
	[self stopCollisionTimer];
	[[CCDirector sharedDirector] pause];
	CGSize s = [[CCDirector sharedDirector] winSize];
	pauseLayer = [CCColorLayer layerWithColor: ccc4(0, 0, 0, 0) width:300 height:150];
	pauseLayer.position = ccp(s.width/2, s.height/2);
	pauseLayer.relativeAnchorPoint = YES;
	[self addChild: pauseLayer z:8];
	resume = [CCMenuItemFont itemFromString:@"Resume" target:self selector:@selector(resume:)];
	mainMenu = [CCMenuItemFont itemFromString:@"Main Menu" target:self selector:@selector(mainMenu:)];
	menu = [Menu menuWithItems:resume, mainMenu, nil];
	[menu alignItemsVertically];
	[self addChild:menu z:10];
}
-(void) resume: (id) sender {
	[[CCDirector sharedDirector] replaceScene:[CCFlipAngularTransition transitionWithDuration:1.2f scene:[GamePlay node]]];

}
-(void) mainMenu: (id) sender {
	[[CCDirector sharedDirector] replaceScene:[CCFlipAngularTransition transitionWithDuration:1.2f scene:[HelloWorld node]]];

}
-(void)resumeGame: (id) sender {
	[self removeChild:menu cleanup:YES];
	[self removeChild:pauseLayer cleanup:YES];
	//play background music
	[[CCDirector sharedDirector] resume];
	pauseButtonStatus = 0;
}

@end

And yes I know there are other errors within here.

Posted 1 year ago #

Joethemonkey101
On probation

Anyone?

Posted 1 year ago #
Codemattic
Moderator

why is the code if (pauseButtonStatus == 0); { not inside a method? Show the .h please

Posted 1 year ago #

Joethemonkey101
On probation

#import "cocos2d.h"

@interface GamePlay : CCLayer {

	CCSprite *mole;
	CCSprite *pauseButton;
	CCAction *moveAction;
	CCMenu *menu;
	CCSprite *bunny;
	CCSprite *background;
	CCSpriteSheet *moleSheet;
	CCSpriteSheet *bunnySheet;
	CCLabel *one;
	CCColorLayer *pauseLayer;
	CGRect *pauseButtonRect;
}

@property (nonatomic, retain) CCSprite *mole;
@property (nonatomic, retain) CCAction *walkAction;

@end

@interface pauseLayer : CCColorLayer {
	CCMenuItemFont *resume;
	CCMenuItemFont *mainMenu;
}

@end

Posted 1 year ago #

Joethemonkey101
On probation

Errors

Posted 1 year ago #
liquidfire
Member

Here is one of my favorite quotes from Lao Tzu :

Give a man a fish and you feed him for a day. Teach him how to fish and you feed him for a lifetime.

The reason I am saying this is because it pertains to @Joethemonkey101. No offense, but if we help you with every error you get, you will never learn how to program, never complete an app, and never be successful. If you learn how to program through books, tutorials, and classes, you will be able to create error free code, as well as complete apps, and could possibly be very successful. I am a self-taught programmer, and it really isn't that difficult, it just takes dedication. I would recommend actually delving into resources and learning it otherwise apps will take years to complete, and will require the help of others.

------

I will, however be contradictory and help you.

1) In your .h - you have CGRect *pauseButtonRect;. A CGRect is a struct, and doesn't need a pointer. Take away the asterisk (*).

2) Look at what @Codemattic said. There is an if statement outside of a method. Furthermore, inside that if statement, unless you still have compatibility with 0.8. or have some class called Menu that I don't know about, the line menu = [Menu menuWithItems:resume, mainMenu, nil]; will cause errors, as Menu has been changed to CCMenu.

3) The compiler will return erroneous errors due to my other 2 points. Fix the errors, and your code should compile correctly, unless there are other errors I missed.

------
[pep-talk]
EDIT: Just a little followup - I am young, and went through the stage of 100's of errors (My record was about 8,000 errors :P), but not knowing what to do about them. Whether you are like me; young, not very much money to buy books, unable to really dedicate much time, or an adult just starting, programming is very difficult and has a steep learning curve, but the payoff is great. Give it time, and you will be fine.
[/pep-talk]

Posted 1 year ago #
Joethemonkey101
On probation

@liquidfire
I know I should take the time to learn cocos2d itself, but where can I learn it? There are iPhone developing books, but none pertaining to cocos2d. Thanks for the help though. I'll keep looking for sites about cocos2d.
Where can I learn about using UITouches with cocos2d? Because I'm not sure how to declare 'touches'. Also, I'm not quite sure which method I would need for that if statement.

Posted 1 year ago #
liquidfire
Member

I'm not quite sure which method I would need for that if statement.

Well it depends - when do you want it to be called?

but none pertaining to cocos2d.

Cocos2D isn't a language - it's a framework. Everything about Objective-C , such as putting if statements inside a method, pertains to Cocos2D, and vice-versa.

Where can I learn about using ... with cocos2d?

Google around, but the best place for many things is the documentation - in your case UITouches, here is a good link from the docs. Look into the section called "Responding to touch". Your first first game.

Here is the link to the documentation in full.

Posted 1 year ago #
Joethemonkey101
On probation

Ok thanks again @liquidfire

Posted 1 year ago #
liquidfire
Member

np! Good luck! :)

Posted 1 year ago #
Joethemonkey101
On probation

Ok so from what I've read, standard touch gets all touches, and targeted touch gets 1 touch at a time. The page of the programming guide you gave me uses targeted touch. My app will use multitouch, so my question is, can you use both standard and targeted touch?
And what do you mean by what do you want to be called?

Posted 1 year ago #
liquidfire
Member

About the touches - Yes, but honestly I am not too great at touch detection, so I can't help you all that well.

What I mean is when do you want the if statement to be checked? When do you want it to see if the value is equal to zero, and execute the code if it is?

Posted 1 year ago #
Joethemonkey101
On probation

That if statement is pausing everything going on and adding a new layer over the original layer, a pause layer.

Posted 1 year ago #
Joethemonkey101
On probation
Just noticed I have 1 more error in a void method. It's saying 'expected expression before pauseLayer'. Here's the method.
```
-(void)resumeGame: (id) sender {
	[self removeChild:menu cleanup:YES];
	[self removeChild:pauseLayer cleanup:YES];
	[[CCDirector sharedDirector] resume];
}
```
Posted 1 year ago #

liquidfire
Member

Why is:

@interface pauseLayer : CCColorLayer {
	CCMenuItemFont *resume;
	CCMenuItemFont *mainMenu;
}

@end

in your .h but nothing in your .m?

Posted 1 year ago #

Joethemonkey101
On probation

This is in my if statement

pauseLayer = [CCColorLayer layerWithColor: ccc4(0, 0, 0, 0) width:300 height:150];
	pauseLayer.position = ccp(s.width/2, s.height/2);
	pauseLayer.relativeAnchorPoint = YES;
	[self addChild: pauseLayer z:8];
	resume = [CCMenuItemFont itemFromString:@"Resume" target:self selector:@selector(resume:)];
	mainMenu = [CCMenuItemFont itemFromString:@"Main Menu" target:self selector:@selector(mainMenu:)];
	menu = [CCMenu menuWithItems:resume, mainMenu, nil];
	[menu alignItemsVertically];
	[self addChild:menu z:10];

Posted 1 year ago #

liquidfire
Member

But you have the interface, but never "@implementation" it.

Posted 1 year ago #

Joethemonkey101
On probation

Oh ok I forgot about that. Here's my entire .M, it contains some other stuff I was working on too. It still has 3 errors though.

#import "GamePlay.h"
#import "HelloWorldScene.h"
#import "CCTouchDispatcher.h"

@implementation GamePlay

@synthesize walkAction = walkAction;
@synthesize mole = mole;

-(id) init {

	if( (self=[super init]) )
	{
		//play background music
		self.isTouchEnabled = YES;
		//background sprite
		background = [CCSprite spriteWithFile:@"Background.jpg"];
		background.position = ccp(240, 160);
		[self addChild:background z:0];
		//level 1 label
		one = [CCLabel labelWithString:@"Level 1" fontName:@"Marker Felt" fontSize:48];
		one.color = ccc3(0, 0, 255);
		id action = [CCMoveTo actionWithDuration:2 position:ccp(240, 285)];
		id action2 = [CCMoveTo actionWithDuration:2 position:ccp(550, 0)];
		[one runAction:[CCSequence actions: action, action2, nil]];
		[self addChild:one];
		//pauseButton sprite
		pauseButton = [CCSprite spriteWithFile:@"paused"];
		pauseButton.position = ccp(460, 15);
		[self addChild:pauseButton];
		//bunny
		bunny = [CCSprite spriteWithSpriteSheet:bunnySheet rect:CGRectMake(0, 0, 0, 0)];
		bunny.position = ccp(100, 100);
		[self addChild:bunnySheet z:5];
		//targeted touch
		[[CCTouchDispatcher sharedDispatcher] addTargetedDelegate:self priority:0 swallowsTouches:YES];
	}
	return self;
}

-(void) one: (id) sender {

}	

-(void) background: (id) sender {

}
-(void) addMole: (id) sender {
	mole = [CCSprite spriteWithSpriteSheet:moleSheet rect:CGRectMake(0, 0, 0, 0)];
	[self addChild:moleSheet z:5];

	CGSize winSize = [[CCDirector sharedDirector] winSize];

}
-(BOOL) ccTouchBegan:(UITouch *)touch withEvent:(UIEvent *)event {
	return YES;
}

-(void) spriteCheck:(UITouch *)touchLocation {
	CGPoint location = [touchLocation locationInView:[touchLocation view]];
	location = [[CCDirector sharedDirector] convertToGL:location];

	pauseButtonRect = CGRectMake((pauseButton.position.x-(pauseButton.contentSize.width)/2), (pauseButton.position.y-(pauseButton.contentSize.height)/2), (pauseButton.contentSize.width), (pauseButton.contentSize.height));
	if (CGRectContainsPoint(pauseButtonRect, location)) {
	[self pauseButton];
	}
}
@end

@implementation pauseLayer

#import "GamePlay.h"

if (pauseButtonStatus == 0); {
	pauseButtonStatus = 1;
	[[SimpleAudioEngine sharedEngine] pauseBackgroundMusic];
	[self stopGameLoopTimer];
	[self stopCollisionTimer];
	[[CCDirector sharedDirector] pause];
	CGSize s = [[CCDirector sharedDirector] winSize];
	pauseLayer = [CCColorLayer layerWithColor: ccc4(0, 0, 0, 0) width:300 height:150];
	pauseLayer.position = ccp(s.width/2, s.height/2);
	pauseLayer.relativeAnchorPoint = YES;
	[self addChild: pauseLayer z:8];
	resume = [CCMenuItemFont itemFromString:@"Resume" target:self selector:@selector(resume:)];
	mainMenu = [CCMenuItemFont itemFromString:@"Main Menu" target:self selector:@selector(mainMenu:)];
	menu = [CCMenu menuWithItems:resume, mainMenu, nil];
	[menu alignItemsVertically];
	[self addChild:menu z:10];
}
-(void) resume: (id) sender {
	[[CCDirector sharedDirector] replaceScene:[CCFlipAngularTransition transitionWithDuration:1.2f scene:[GamePlay node]]];

}
-(void) mainMenu: (id) sender {
	[[CCDirector sharedDirector] replaceScene:[CCFlipAngularTransition transitionWithDuration:1.2f scene:[HelloWorld node]]];

}
-(void)resumeGame: (id) sender {
	[self removeChild:menu cleanup:YES];
	[self removeChild:pauseLayer cleanup:YES];
	//play background music
	[[CCDirector sharedDirector] resume];
}

@end

Posted 1 year ago #

liquidfire
Member

You still have the if statement where it doesn't belong. Put it inside your method. I'm not sure what you want to do with it, but maybe put it in an init method?

Posted 1 year ago #
Joethemonkey101
On probation

I tried that before, it caused 11 errors...
Sorry, it's hard to explain. That statement is telling everything to be paused, and then adds another layer on top of that.

Posted 1 year ago #

RSS feed for this topic

12 3 Next »

Topic Closed

This topic has been closed to new replies.