UIImageView dynamically to array

[sgr_81]

Hello All,

Im very new on this, I have been locking for 2 days if it is possible to do this:

in my .h file I have:
@interface puzzle2ViewController : UIViewController {
IBOutlet UIImageView *pza1;
IBOutlet UIImageView *pza2;
.....
}
@property(assign) IBOutlet UIImageView *pza1;
@property(assign) IBOutlet UIImageView *pza2;

in my .m file I have:
#define kNumeroDePiezas 2
piezas = [[NSMutableArray alloc] initWithCapacity:kNumeroDePiezas];
for(int i = 0; i < kNumeroDePiezas; i++) {
UIImageView *myView = [NSString stringWithFormat:@"pza%d", i];
[piezas addObject:myView];
}

As you can see I want to add the UIImageView dynamically to a NSMutableArray. Why? because I whant it dynamic, now I have 2 images but probably I will have 10 or 20 images.

The images are in the interface builder with reference outlets already seted up.

Hope somebody can help me, thanks.

[Duncan C]

Originally Posted by sgr_81

Hello All,

Im very new on this, I have been locking for 2 days if it is possible to do this:

in my .h file I have:
@interface puzzle2ViewController : UIViewController {
IBOutlet UIImageView *pza1;
IBOutlet UIImageView *pza2;
.....
}
@property(assign) IBOutlet UIImageView *pza1;
@property(assign) IBOutlet UIImageView *pza2;

in my .m file I have:
#define kNumeroDePiezas 2
piezas = [[NSMutableArray alloc] initWithCapacity:kNumeroDePiezas];
for(int i = 0; i < kNumeroDePiezas; i++) {
UIImageView *myView = [NSString stringWithFormat:@"pza%d", i];
[piezas addObject:myView];
}

As you can see I want to add the UIImageView dynamically to a NSMutableArray. Why? because I whant it dynamic, now I have 2 images but probably I will have 10 or 20 images.

The images are in the interface builder with reference outlets already seted up.

Hope somebody can help me, thanks.

Yes, you can certainly store image views into an array. An NSArray will hold ANY object.

However, your code is creating strings, and saving those to an array.

This line:

UIImageView *myView = [NSString stringWithFormat:@"pza%d", i];

Should generate a compiler warning, because you're assigning a string to a variable of type UIImageView.

You probably meant to create your image views from images that you load with imageNamed or imageWithContentsOfFile.
Something like this:

NSString* docsDir = [NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES) lastObject];
NSString* filename = [NSString stringWithFormat:@"pza%d", i];
NSString* pathName = [docsDir stringByAppendingPathComponent: filename];
UIImage* anImage = [UIImage imageWithContentsOfFile: pathName];
UIImageView* anImageView = [[[UIImageView alloc] initWithImage: anImage] autorelease];

[imageViewArray addObject: anImageView];

[sgr_81]

wow!, how fast you are Duncan, thanks so much,

just one more question, I have already tried your solution, does it will work if i have this:


The images are placed in the interface builder and referenced.

i changed my code accordingly:
piezas = [[NSMutableArray alloc] initWithCapacity:kNumeroDePiezas];
for(int i = 0; i < kNumeroDePiezas; i++) {
NSString* docsDir = [NSSearchPathForDirectoriesInDomains(NSDocumentDire ctory, NSUserDomainMask, YES) lastObject];
NSString* filename = [NSString stringWithFormat:@"pza%d", i];
NSString* pathName = [docsDir stringByAppendingPathComponent: filename];
UIImage* anImage = [UIImage imageWithContentsOfFile: pathName];
UIImageView* anImageView = [[[UIImageView alloc] initWithImage: anImage] autorelease];

[piezas addObject: anImageView];

NSLog(@"the array is %f",piezas);

And I get in the console:

2011-04-09 15:16:54.810 puzzle2[14422:207] the array is 0.000000
2011-04-09 15:16:54.812 puzzle2[14422:207] the array is 0.000000

Thanks

[Duncan C]

Originally Posted by sgr_81

wow!, how fast you are Duncan, thanks so much,

just one more question, I have already tried your solution, does it will work if i have this:


The images are placed in the interface builder and referenced.

i changed my code accordingly:
piezas = [[NSMutableArray alloc] initWithCapacity:kNumeroDePiezas];
for(int i = 0; i < kNumeroDePiezas; i++) {
NSString* docsDir = [NSSearchPathForDirectoriesInDomains(NSDocumentDire ctory, NSUserDomainMask, YES) lastObject];
NSString* filename = [NSString stringWithFormat:@"pza%d", i];
NSString* pathName = [docsDir stringByAppendingPathComponent: filename];
UIImage* anImage = [UIImage imageWithContentsOfFile: pathName];
UIImageView* anImageView = [[[UIImageView alloc] initWithImage: anImage] autorelease];

[piezas addObject: anImageView];

NSLog(@"the array is %f",piezas);

And I get in the console:

2011-04-09 15:16:54.810 puzzle2[14422:207] the array is 0.000000
2011-04-09 15:16:54.812 puzzle2[14422:207] the array is 0.000000

Thanks

I don't understand what you're trying to do. Are you saying that you have a bunch of image views that you defined in Interface builder and you want to put THOSE into an array?

In that case, I would suggest using tags. There is a UIView method viewWithTag that finds a subview with a particular numeric tag. You could set up your image views in IB with tags that start at 1 and go up from there, and then use code like this:

UIImageView* anImageView;
for(int i = 0; i < kNumeroDePiezas; i++) 
{
  anImageView = [[self view] viewWithTag: i+1];
  if (anImageView)
    [piezas addObject: anImageView];
}
NSLog(@"the piazzas array contains %d items. Contents = %@",[piazzas count], piazzas);

Note that you should use the "%@" format string to display an array (or any other object.) The "%f" format string you used in your log statement is for displaying a floating point value, and you will get odd results if you try to log an object with that format.

[sgr_81]

Thats exactly what I was looking for!!!! I read something about tags but I did not put atention, that is why I will never found a solution. thanks so much for your help so I can continue with my little project.

[sgr_81]

I forgot to tell you im receiving this alert



Is that normal?

[baja_yu]

Typecast it to avoid the compiler warning, if you are sure that the pointers will be pointing to UIImageView objects.

[sgr_81]

Originally Posted by baja_yu

Typecast it to avoid the compiler warning, if you are sure that the pointers will be pointing to UIImageView objects.

thanks!!!!! it removes the warning, this is the code now:

piezas = [[NSMutableArray alloc] initWithCapacity:kNumeroDePiezas];
for(int i = 0; i < kNumeroDePiezas; i++)
{
UIImageView *anImageView = (UIImageView *)[self.view viewWithTag:i+1];
if (anImageView)
[piezas addObject: anImageView];
}

[Duncan C]

Originally Posted by sgr_81

thanks!!!!! it removes the warning, this is the code now:

piezas = [[NSMutableArray alloc] initWithCapacity:kNumeroDePiezas];
for(int i = 0; i < kNumeroDePiezas; i++)
{
UIImageView *anImageView = (UIImageView *)[self.view viewWithTag:i+1];
if (anImageView)
[piezas addObject: anImageView];
}

If you want to be really careful, you could add a runtime type check to make sure the view you grab is really an image view, like this:

piezas = [[NSMutableArray alloc] initWithCapacity:kNumeroDePiezas]; 
for(int i = 0; i < kNumeroDePiezas; i++) 
{
  UIImageView *anImageView = (UIImageView *)[self.view viewWithTag:i+1];
  if (anImageView && [anImageView isMemberOfClass: [UIImageView class]])
    [piezas addObject: anImageView];
}

[sgr_81]

Originally Posted by Duncan C

If you want to be really careful, you could add a runtime type check to make sure the view you grab is really an image view, like this:

piezas = [[NSMutableArray alloc] initWithCapacity:kNumeroDePiezas]; 
for(int i = 0; i < kNumeroDePiezas; i++) 
{
  UIImageView *anImageView = (UIImageView *)[self.view viewWithTag:i+1];
  if (anImageView && [anImageView isMemberOfClass: [UIImageView class]])
    [piezas addObject: anImageView];
}

very nice improvement!!!!! I already applied to my code!!! thanks