cocos2d for iPhone

A fast, easy to use, free, and community supported 2D game engine

1. 

   7thsmith
   Member

   Hey Guys,

   Hope this is not a dumb problem that I can't seem to figure out. Here is the code I am working with:
   

   @implementation SinglePlayArea
   - (id)init
   {
   	if ((self = [super init])) {
   
   		playerDeck = [[PlayerDeck alloc] init];
   		[playerDeck setupDeck];
   		[playerDeck shuffleDeck];
   		CCMenuItem *deckB = [CCMenuItemImage itemFromNormalImage:@"Icon.png" selectedImage:@"Icon.png" target:self selector:@selector(deckNewCards:)];
   		CCMenu *deckButton = [CCMenu menuWithItems:deckB, nil];
   		deckButton.position = ccp(350, 50);
   		[self addChild: deckButton];
   	}
   	return self;
   }
   
   - (void)deckNewCards:(id)sender
   {
   	for (int i = 0; i < 4; i++)
   	{
   		Cards *card = [playerDeck.shuffledlist objectAtIndex:i];
   		CCTexture2D *cardTexture = [[CCTextureCache sharedTextureCache] addImage:[NSString stringWithFormat:@"%@", card.imagePath]];
   		card = [Cards cardWithTexture:cardTexture];
   		card.position = ccp( 30 + i * (i + 50), 160);
   		[self addChild:card z:(i - i * 2)];
   	}
   }

   The playerDeck is an array of 52 card objects that have 3 variables: Type, vaule and imagePath. The problem happens at
   “CCTexture2D *cardTexture = [[CCTextureCache sharedTextureCache] addImage:[NSString stringWithFormat:@"%@", card.imagePath]];”
   In the deckNewCards method. The debugger is saying that the variable for imagePath is out of scope. If I place the code from deckNewCards into my init method, the code works fine. Thanks for any help you can give or can point me in the right direction.

   Posted 1 year ago #
2. 

   serenecrue
   Member

   Not sure about the programming languages you are using,is that Visual c++?

   pavement reflectors

   Posted 1 year ago #
3. 

   7thsmith
   Member

   I am using Objective-c.

   Posted 1 year ago #
4. 

   chrisco
   Member

   I could be wrong, but based on where the link took me, I don't think @serenecrue is an actual person. Or at least, not one that really cares what programming language you're using...

   What prints in the console if you add this after Cards *card = [playerDeck.shuffledlist objectAtIndex:i];:

   NSLog(@"card.imagePath = %@", card.imagePath);

   Or does it still tell you it's out of scope?

   Chris

   Posted 1 year ago #
5. 

   Reonarudo
   Member

   @chrisco

   Nop I don't it is a person. The new spamming algoritms can pick up on contexts and give something like that above.

   Posted 1 year ago #
6. 

   7thsmith
   Member

   Thanks for the help chrisco. The console prints -
   

   card.imagePath = {(
       <UIWindow: 0xf18e00; frame = (0 0; 320 480); layer = <CALayer: 0xf1a640>>
   )}

   Posted 1 year ago #
7. 

   chrisco
   Member

   I'm guessing that's not what you want it say, right? At what point in your code do you assign the imagePath to your card object? Maybe post the code for your Cards class?

   Posted 1 year ago #
8. 

   7thsmith
   Member

   I just want to give credit to the right person, I found the code for the card and deck classes in a tutorial online.

   Here is my cards.h
   

   #import <Foundation/Foundation.h>
   #import "cocos2d.h"
   
   typedef enum tagCardState {
   	kCardStateGrabbed,
   	kCardStateUngrabbed
   } CardState;
   
   @interface Cards : CCSprite <CCTargetedTouchDelegate> {
   	int rank;
   	NSString *suit;
   	NSString *imagePath;
   	CardState cardState;
   }
   
   @property(assign) int rank;
   @property(assign) NSString *suit;
   @property(assign) NSString *imagePath;
   @property(nonatomic, readonly) CGRect rect;
   
   + (id)cardWithTexture:(CCTexture2D *)texture;
   
   @end

   But the information is assigned in the deck class. Here is the code where the variables are assigned

   for (i = 1; i < 14; i++)
   	{
   		Cards *c;
   		c = [[Cards alloc]init];
   		c.rank = i;
   		c.suit = @"Wind";
   		c.imagePath = [NSString stringWithFormat:@"%@_%i.png", c.suit, c.rank];
   		//c.cardState = kCardStateUngrabbed;
   		[cardlist addObject:c];
   	}

   Posted 1 year ago #
9. 

   chrisco
   Member

   Try changing this line:

   @property(assign) NSString *imagePath;

   to this:

   @property(retain) NSString *imagePath;

   Right now, you're assigning an autoreleased string, when you should be retaining an autoreleased string.

   Also, unrelated to your current problem, you may have a memory leak when adding your cards to the cardlist array - an array will retain your card already, so you should release the card (c) after adding it to the array or call autorelease when you initialize it, like so:

   for (i = 1; i < 14; i++)
   	{
   		Cards *c;
   		c = [[[Cards alloc]init] autorelease];
   		c.rank = i;
   		c.suit = @"Wind";
   		c.imagePath = [NSString stringWithFormat:@"%@_%i.png", c.suit, c.rank];
   		//c.cardState = kCardStateUngrabbed;
   		[cardlist addObject:c];
   	}

   Hope that helps.
   Chris

   Posted 1 year ago #
10. 

    7thsmith
    Member

    OMG thanks Chris, that totally worked. You have no idea how long that problem has been bothering me.

    Yeah I know about the memory leak, they came with the code. Just wanted to get the code working before I go through and starting cleaning it up. Thanks again for the help.

    Posted 1 year ago #
11. 

    chrisco
    Member

    Glad it worked :)

    Posted 1 year ago #

RSS feed for this topic

Reply

You must log in to post.