invalid receiver type 'UIImageView *[50]'

[NeoNate]

I need some help understanding the following error and how to fix it. I have set up the following to handle a series of images:

UIImageView *Object[50];

I then try to execute the following:

UIImageView *imageView = [Object objectAtIndex:0];
imageView.layer.zPosition = 1000;

Which gives me a warning about invalid receiver type 'UIImageView *[50]', the application crashes when this line is executed.

What does this mean?

Also do I need to release the memory from imageView at the end?

Many thanks, Nathan

[FlyingDiver]

You declared the type of Object as a C array, but then you're trying to access it as an NSArray item. Pick one or the other. Then you're going to have to actually alloc/init the UIImageView objects you're trying to store in the array.

joe

[NeoNate]

Joe, thanks for the reply.

I'm really struggling with this. I don't suppose you could give me some pointers, I'm new to both C and Objective C. Here is what I've been trying to do since my last post:

NSMutableArray *Object[50];

@property (nonatomic, retain) NSMutableArray *Object;

@dynamic Object;

Object = [[NSMutableArray alloc] init];

for(int i = 0; i < 50; i++)
{
	UIImageView *myImageView = UIImageView * myImageView = [self newPieceViewWithImageNamed:@"" atPostion:CGPointMake(0,0)];[self addSubview:Object[i]];
	[Object addObject:myImageView];
	[myImageView release];
}

UIImageView *imageView = [Object objectAtIndex:0];
imageView.layer.zPosition = 1000;

I still get the same error so imagine I am still mixing the two languages.

[meowmix23F]

Originally Posted by NeoNate

Joe, thanks for the reply.

I'm really struggling with this. I don't suppose you could give me some pointers, I'm new to both C and Objective C. Here is what I've been trying to do since my last post:

NSMutableArray *Object[50];

@property (nonatomic, retain) NSMutableArray *Object;

@dynamic Object;

Object = [[NSMutableArray alloc] init];

for(int i = 0; i < 50; i++)
{
	UIImageView *myImageView = UIImageView * myImageView = [self newPieceViewWithImageNamed:@"" atPostion:CGPointMake(0,0)];[self addSubview:Object[i]];
	[Object addObject:myImageView];
	[myImageView release];
}

UIImageView *imageView = [Object objectAtIndex:0];
imageView.layer.zPosition = 1000;

I still get the same error so imagine I am still mixing the two languages.

Ok, lets take a look at your code.

You declared *50* NSArrays. Just say

NSMutableArray *object;

NOT

NSMutableArray *object[50];
that means that

object[1] is an NSArray
object[2] is an NSArray
etc.

object itself is not an NSArray, rather a C array with pointers to NSArrays, so when you call alloc init on it it fails because A C ARRAY IS NOT AN OBJECT!

[FlyingDiver]

Try something like this. I just edited your code, it has not been compiled or tested.

NSMutableArray *viewArray;
@property (nonatomic, retain) NSMutableArray * viewArray;

@synthesize viewArray;

viewArray = [[NSMutableArray alloc] init];

for(int i = 0; i < 50; i++)
{
	UIImageView *myImageView = [self newPieceViewWithImageNamed:@"" atPostion:CGPointMake(0,0)];
	[viewArray addObject:myImageView];
        [self addSubview: myImageView];
	[myImageView release];
}

UIImageView *imageView = [viewArray objectAtIndex:0];
imageView.layer.zPosition = 1000;

I renamed "Object". Terrible name for a variable.

Create the array. Add all the image views to it. Then use the first one in the array.

You need to check what "newPieceViewWithImageNamed" returns. If it's an auto-released object, you don't need the "[myImageView release];" line, it'll cause a release exception.

joe

[NeoNate]

Joe and Meowmix23f, thanks that worked brilliantly!

My variables are named better, I was using Object for demonstration, perhaps myArray may have been a better choice

[NeoNate]

Thanks for Joe and Meomix23f for their help.

For anyone who may refer to this thread in the future I have supplied the full solution below:

/* HEADER */
NSMutableArray *myArray;

@property (nonatomic, retain) NSMutableArray * myArray;


/* MAIN */
@dynamic myArray;

for(int i = 0; i < 50; i++)
{
	UIImageView * myImageView = [[UIImageView alloc] init];
	[myArray addObject:myImageView];
	[myImageView release];
}


UIImageView *imageView = [myArray objectAtIndex:0];				
imageView = [self newPieceViewWithImageNamed:@"myImage.png" atPostion:CGPointMake(200,400)];	[self addSubview:imageView];	
imageView.transform = CGAffineTransformMakeRotation(M_PI / 2.0);
imageView.layer.zPosition = 100;
				
[imageView release];

[Georg]

Originally Posted by NeoNate

Thanks for Joe and Meomix23f for their help.

For anyone who may refer to this thread in the future I have supplied the full solution below:

/* HEADER */
NSMutableArray *myArray;

@property (nonatomic, retain) NSMutableArray * myArray;


/* MAIN */
@dynamic myArray;

for(int i = 0; i < 50; i++)
{
	UIImageView * myImageView = [[UIImageView alloc] init];
	[myArray addObject:myImageView];
	[myImageView release];
}


UIImageView *imageView = [myArray objectAtIndex:0];				
imageView = [self newPieceViewWithImageNamed:@"myImage.png" atPostion:CGPointMake(200,400)];	[self addSubview:imageView];	
imageView.transform = CGAffineTransformMakeRotation(M_PI / 2.0);
imageView.layer.zPosition = 100;
				
[imageView release];

I am using C array like this:

.h:
UIImageView *imgStone[50];



and in .m with my code:
for (int i=0; i<50; i++)
{
imgStone[i] = [[UIImageView alloc] initWithImage:StoneFileName];
imgStone[i] = CGRectMake(x, y);
[self.view addSubview:imgStone[i]];
[imgStone[i] release];
}


but it is strange, since I've released the imgStone array, I still can move these imgStone by CGPointMake:
imgStone[i].center = CGPointMake(new_x, new_y, width, height);


Just curious if this is correct way? because my app runs quite smooth, but I found out that there is memory leak. If this is not a proper way, then how should I do? I have created another version which is using:

.h:
NSMutableArray *imgStoneArray;
UIImageView *imgStone;

.m:
for(int i=0;i<50;i++)
{
imgStone = [[UIImageView alloc] initWithImage:StoneFileName];
imgStone.frame = CGRectMake(x, y, width, height);
[imgFishArray addObject:imgFStone];
[self.view addSubview:imgStone];
[imgFish release];
}
then move Stones:
for(int i=0;i<50;i++)
{
UIImageView *loadedImage = [[UIImageView alloc] init];
loadedImage = [imgStoneArray objectAtIndex:i];
loadedImage.center = CGPointMake(new_x, new_y);
[loadedImage release];
}

the imgStone moves by a timer (0.2 second)

but the latter one crashed, I don't know what was wrong. Can anyone help on this? which way is better?

[nobre84]

Originally Posted by Georg

I am using C array like this:
and in .m with my code:
for (int i=0; i<50; i++)
{
imgStone[i] = [[UIImageView alloc] initWithImage:StoneFileName];
imgStone[i] = CGRectMake(x, y);
[self.view addSubview:imgStone[i]];
[imgStone[i] release];
}

...


m:
then move Stones:
for(int i=0;i<50;i++)
{
UIImageView *loadedImage = [[UIImageView alloc] init];
loadedImage = [imgStoneArray objectAtIndex:i];
loadedImage.center = CGPointMake(new_x, new_y);
[loadedImage release];
}

the imgStone moves by a timer (0.2 second)

but the latter one crashed, I don't know what was wrong. Can anyone help on this? which way is better?

Your first snippet looks ok... You may be leaking when disposing of your array or when moving the images.

In the second snippet you just load the images, you don't need to alloc them again. Your loadedImage release is actually releasing the object in the array, that will crash next time you try to use it, while leaking the unused alloc'd item. Just remove the alloc and release lines from there.

[Georg]

Originally Posted by nobre84

Your first snippet looks ok... You may be leaking when disposing of your array or when moving the images.

In the second snippet you just load the images, you don't need to alloc them again. Your loadedImage release is actually releasing the object in the array, that will crash next time you try to use it, while leaking the unused alloc'd item. Just remove the alloc and release lines from there.

hihi nobre84, I really appreciate your comment, it had solved the problem!! let me know your App name in AppStore.

Another little problem I have is, the dealloc function is never get called, therefore I cannot excute again the subview by pressing the button on mainView:

- (IBAction)btnNewGame{

GameLevel = 1;


if (self.playGameView == nil) {
PlayGameViewController *playGameViewView = [[PlayGameViewController alloc] initWithNibName:@"PlayGameViewController" bundle:nil];
self.playGameView = playGameViewView;
[self.view insertSubviewlayGameViewView.view atIndex:20];
[self.view addSubviewlayGameViewView.view];
//[self.view removeFromSuperview];
[playGameViewView release];
}
NSLog(@"Testing ret count: %d", [self.view retainCount]);
}

which part I was doing wrong?