Iphone Code - CGPoint Question

Question

I have 10 moving objects (UIImageView),
is there a better way to write this code?

    - (void) jumpOnTimer {

        jumpBall1.center = CGPointMake(jumpBall1.center.x+pos1.x,jumpBall1.center.y+pos1.y);

        if(jumpBall1.center.x > 60 || jumpBall1.center.x < 0)
            pos1.x = -pos1.x;
        if(jumpBall1.center.y > 211 || jumpBall1.center.y < 82)
            pos1.y = -pos1.y;

        jumpBall2.center = CGPointMake(jumpBall2.center.x+pos2.x,jumpBall2.center.y+pos2.y);

        if(jumpBall2.center.x > 40 || jumpBall2.center.x < 0)
            pos2.x = -pos2.x;
        if(jumpBall2.center.y > 206 || jumpBall2.center.y < 82)
            pos2.y = -pos2.y;

and so on...

Answer 1

Judging by this piece of code, it looks like you have one controller that “owns” ten balls, and you want the balls to bounce according to a set of rules that are unique to each ball. A more object oriented approach would be:

@interface JumpBallClass
{
    CGPoint center;
    CGPoint speed;

    CGPoint lowerLimit;
    CGPoint upperLimit;
}
@property (assign) CGPoint lowerLimit;
@property (assign) CGPoint upperLimit;
- (void)update;
@end

@implementation JumpBallClass
- (void)update
{
    center.x += speed.x;
    center.y += speed.y;

    if (center.x > upperLimit.x || center.x < lowerLimit.x)
    { speed.x = -speed.x; }
    if (center.y > upperLimit.y || center.y < lowerLimit.y)
    { speed.y = -speed.y; }
}
@end

This setting allows you to configure all the balls once, setting their upper and lower limits:

[jumpBall1 setUpperLimit:CGPointMake(60, 211)];
[jumpBall1 setLowerLimit:CGPointMake(0, 82)];
...

And then just calling updateon each ball in the timer method:

- (void) jumpOnTimer {          

    [jumpBall1 update];          
    [jumpBall2 update];
    ...
}

This can be simplified by saving all the balls in NSArray:

NSArray * balls = [NSArray arrayWithObjects:jumpBall1, jumpBall2, ..., nil];

And then calling makeObjectsPerformSelector:

[balls makeObjectsPerformSelector:@selector(update)];

Answer 2

jumpBalls, . - :

JumpBallClass *myjumpballs[10];

for (i=0; i<10; i++) {
    myjumpballs[i].center = CGPointMake(myjumpballs[i].center.x+pos1.x,myjumpballs[i].center.y+pos1.y);

    if(myjumpballs[i].center.x > 60 || myjumpballs[i].center.x < 0)
        pos1.x = -pos1.x;
    if(myjumpballs[i].center.y > 211 || myjumpballs[i].center.y < 82)
        pos1.y = -pos1.y;
}

Answer 3

It looks like you are trying to manually revive. Look at using UIView animation instead