[5h00p3r_MAN]

I'm new to programming on the iPhone, and programming in general. I've only used Visual Basic and MATLAB before, so this may sound a bit stupid.

So basically, I'm just getting experience on making apps for the iPhone, and I want to try to make an app that will fill up the screen with 1x1 pixel labels one label at a time.

Fill_ScreenAppDelegate.h

#import <UIKit/UIKit.h>

@interface Fill_ScreenAppDelegate : NSObject <UIApplicationDelegate> {
    UIWindow *window;
	CGRect frame;
	UILabel *label;
}

@property (nonatomic, retain) IBOutlet UIWindow *window;

- (void)addPixel:(CGRect) frame

@end

Fill_ScreenAppDelegate.m

#import "Fill_ScreenAppDelegate.h"

@implementation Fill_ScreenAppDelegate

@synthesize window;

- (void)addPixel:(CGRect) frame {
	UILabel *label = [[UILabel alloc] initWithFrame:frame];
	[window addSubview:label];
	[label dealloc];
}

- (void)applicationDidFinishLaunching:(UIApplication *)application {    

	int xx = 1;
	int yy = 1;

    while(yy <= 480) {
		
		while(xx<= 320) {
		if (xx < 320)
			xx = xx + 1;
			CGRect frame = [CGRectMake(xx, yy, 1, 1)];
			[Fill_ScreenAppDelegate addPixel:frame];

		}
		
		xx = 1;
		yy = yy + 1;
		
		CGRect frame = [CGRectMake(xx, yy, 1, 1)];
		[Fill_ScreenAppDelegate addPixel:frame];

	}
		
    [window makeKeyAndVisible];
}


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


@end

Here is an image of the warnings and errors :

[NetMage]

First of all, I think creating 153,600 labels is likely to run the iPhone out of memory.

Second, CGRectMake is a function, not a message call, so should not be in brackets.

Third, Fill_ScreenAppDelegate is a class, not an object, which means it only responds to + messages, not - messages (hence the warning). Use the special name self to send a message to the current object.

Fourth, you should never call dealloc, you call release. The framework calls dealloc when it determines there are no more references to the released object to actually return memory.

Fifth, you don't need instance variables for frame and label since you are just using them locally in methods.

Sixth, you are resetting xx to one every time through your while loop on xx which means you won't get anywhere.

Attached is code that may compile (untested) but wouldn't bet on running:

#import "Fill_ScreenAppDelegate.h"

@implementation Fill_ScreenAppDelegate

@synthesize window;

- (void)addPixel:(CGRect) frame {
	UILabel *label = [[UILabel alloc] initWithFrame:frame];
	[window addSubview:label];
	[label release];
}

- (void)applicationDidFinishLaunching:(UIApplication*) application {

    int xx, yy;

    for (yy = 0; yy < 480; ++yy) {
        for (xx = 0; xx < 320; ++xx) {
	    CGRect frame = CGRectMake(xx, yy, 1, 1);
	    [self addPixel:frame];
	}
    }

    [window makeKeyAndVisible];
}

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

@end