Allocation of memory within app

[shadylane315]

It seems i'm not allocating any memory at all in my application.

Here is my code.

#import "ApotheosisViewController.h"

@implementation ApotheosisViewController

@synthesize scrollView1;

- (BOOL)shouldAutorotateToInterfaceOrientation: (UIInterfaceOrientation)interfaceOrientation {
	return (interfaceOrientation == UIInterfaceOrientationLandscapeRight);
}

const CGFloat kScrollObjHeight	= 320.0;
const CGFloat kScrollObjWidth	= 480.0;
const NSUInteger kNumImages		= 40;


- (void)layoutScrollImages
{
	UIImageView *view = nil;
	NSArray *subviews = [scrollView1 subviews];
	
	// reposition all image subviews in a horizontal serial fashion
	CGFloat curXLoc = 0;
	for (view in subviews)
	{
		if ([view isKindOfClass:[UIImageView class]] && view.tag > 0)
		{
			CGRect frame = view.frame;
			frame.origin = CGPointMake(curXLoc, 0);
			view.frame = frame;
			
			curXLoc += (kScrollObjWidth);
		}
	}
	
	// set the content size so it can be scrollable
	[scrollView1 setContentSize:CGSizeMake((kNumImages * kScrollObjWidth), [scrollView1 bounds].size.height)];
}

- (void)viewDidLoad
{
	self.view.backgroundColor = [UIColor viewFlipsideBackgroundColor];
	
	// 1. setup the scrollview for multiple images and add it to the view controller
	//
	// note: the following can be done in Interface Builder, but we show this in code for clarity
	[scrollView1 setBackgroundColor:[UIColor blackColor]];
	[scrollView1 setCanCancelContentTouches:NO];
	scrollView1.indicatorStyle = UIScrollViewIndicatorStyleBlack;
	scrollView1.clipsToBounds = NO;		// default is NO, we want to restrict drawing within our scrollview
	scrollView1.scrollEnabled = YES;
	
	// pagingEnabled property default is NO, if set the scroller will stop or snap at each photo
	// if you want free-flowing scroll, don't set this property.
	scrollView1.pagingEnabled = YES;
	
	// load all the images from our bundle and add them to the scroll view
	NSUInteger i;
	for (i = 1; i <= kNumImages; i++)
	{
		NSString *imageName = [NSString stringWithFormat:@"image%d.jpg", i];
		UIImage *image = [UIImage imageNamed:imageName];
		UIImageView *imageView = [[UIImageView alloc] initWithImage:image];
		
		// setup each frame to a default height and width, it will be properly placed when we call "updateScrollList"
		CGRect rect = imageView.frame;
		rect.size.height = kScrollObjHeight;
		rect.size.width = kScrollObjWidth;
		imageView.frame = rect;
		imageView.tag = i;	// tag our images for later use when we place them in serial fashion
		[scrollView1 addSubview:imageView];
		[imageView release];
}
	[self layoutScrollImages];	// now place the photos in serial layout within the scrollview
	
}

- (void)dealloc
{	
	[scrollView1 release];
	
	[super dealloc];
}

- (void)didReceiveMemoryWarning
{
	// invoke super's implementation to do the Right Thing, but also release the input controller since we can do that	
	// In practice this is unlikely to be used in this application, and it would be of little benefit,
	// but the principle is the important thing.
	//
	[super didReceiveMemoryWarning];
}

@end

The line of code that is giving me trouble is this one.

UIImage *image = [UIImage imageNamed:imageName];

I'm a newbie and i'm impressed i got this far but the memory allocation really has me by the balls. any help would be awesome.

I have all my images named like so:

image1, image,2, image3, etc

[harryvermeulen]

Like you said, you have to allocate memory for the class you're making an object off.

Try something like this:

CGRect imageRect = CGRectMake(0, 0, 1600, 1200);
myImage = [[UIImageView alloc] initWithFrame:imageRect];
[myImage setImage:[UIImage imageNamed:@"picture.jpg"]];

[shadylane315]

ok awesome. Would i have to repeat this for all 40 images?

Originally Posted by harryvermeulen

Like you said, you have to allocate memory for the class you're making an object off.

Try something like this:

CGRect imageRect = CGRectMake(0, 0, 1600, 1200);
myImage = [[UIImageView alloc] initWithFrame:imageRect];
[myImage setImage:[UIImage imageNamed:@"picture.jpg"]];