4

我正在尝试创建一个横向滚动条，但无法将Y对象的坐标设置为随机值。

我称我的对象为平台。我希望每个平台出现在不同的Y坐标上，并相信我的方法是正确的做法。但它工作得不是很好。

所有Y坐标都注销相同的数字，我不完全确定为什么？我的意思是我在实例化它们时显然是在添加间距。

我还注意到的一件事是，如果我不添加计时器并调用移动方法，它们确实会出现在正确的位置。所以它可能是调用函数之间的东西。

我发现的另一个问题是，当我再次调用平台时，只有一个平台遵循幻灯片功能要求它执行的操作，其他 2 个平台遵循这些点，但对其他任何内容都没有反应。

任何帮助都非常感谢！

//
//  C4WorkSpace.m
//  TheGame
//
//

#import "C4Workspace.h"

@implementation C4WorkSpace {

    C4Shape  *player ; // player
    CGPoint p, move; // CG point for moving platforms && Players
    int speed; // Speed of the platforms
    C4Timer *timer; // Timer
    NSMutableArray *platforms; // Platform Array

}

-(void)setup {


    speed = 5; // Speed Limit
    p = CGPointMake(self.canvas.width, 400); // Making 2 coordinates for the platform shape to follow
    move = CGPointMake(0, 0); // Making 2 coordinates for the user shape to follow
    platforms = [NSMutableArray array]; // Pointer of Array for platforms

    // Generating shapes

    for ( int i = 0; i < 3; i++)
    {
        C4Shape * s = [C4Shape rect:CGRectMake(0, 400, 50, [C4Math randomInt:50])]; // Making the platform
        p.x = self.canvas.width; // x - coordinate for the platforms
        p.y += 100; // y - coordinate of the platforms
        s.center =  p; // The Center of the Circle is P
        [platforms addObject:s]; // Adding platforms to the platforms array
        [self.canvas addShape:platforms[i]]; // Adding an instance of it
        timer = [C4Timer automaticTimerWithInterval:1.0f/30 target:self method:@"slide" repeats:YES]; // Timer to shoot it off ever frame

    }


    player = [C4Shape ellipse:CGRectMake(0, 0, 50, 50)]; // The shape of the player
    [self.canvas addSubview:player]; // Adding an instance of the player



}

//Moving the platform

-(void) slide {

    //Calling the platforms again to add movement

    for (C4Shape *s in platforms){

    // Adding boundries

    if (p.x <= 0 ) {
        p.x = self.canvas.width; // if it's smaller than the width of the cavas auto transport
        p.y = [C4Math randomInt:self.canvas.height]; // choose a different y coordinate for each

    }

    p.x-= speed; // Adding accelaration
    C4Log(@"The Y is .%2f",  p.y); // Logging the problem
    s.center = p; // making the shape follow the point

    }
}

-(void)touchesMoved:(NSSet *)touches withEvent:(UIEvent *)event {

    UITouch *place = [[event allTouches] anyObject]; // Get touches
    move = [place locationInView:place.view]; // Gets the location of the current mouse point
    player.center = move; // folllowing the move point

    [self collisionCheck]; // collision check
}


-(void) collisionCheck {
    //currently empty!

    }


@end
4

2 回答 2

3

y 坐标正在完美更新——在您正在检查的单点上，并将所有 C4Shapes 的中心设置到幻灯片方法中。

在设置中这工作正常，因为您在更改之前将 C4Shape s 的中心设置为 p，但是当您转到幻灯片方法中的 For 循环时，您只是记录和更新该点，该点从最后一个开始假设在调用 setup 和 slide 之间没有任何反应，请将您更新到 setup 中。p 是 C4Workspace 类的一个 ivar，因此每个实例都有一个。为了解决这个问题，我认为您应该将幻灯片中每次出现的 p 更改为 s.center，并去掉最后一行。

顺便说一句，你应该认真考虑重命名这些方法中的变量，它们很难理解——我实际上很困惑为什么 p 是一个 ivar 并且没有在 setup 中声明，这似乎是你唯一的地方需要它。

于 2013-11-02T05:25:10.793 回答
2

本关于仅更新p变量的答案是正确的。您要做的是检查每个单独形状的中心点并对其进行操作。

错误的原因是这个逻辑：

for(every shape in platforms) {
    check to see if a point p is off the screen
        if it is, then change its value to a random number
    then update the speed of p
    set the centerpoint of the current shape to p
}

上面的逻辑是你在这里编码的：

for (C4Shape *s in platforms) {
    if (p.x <= 0 ) {
        p.x = self.canvas.width;
        p.y = [C4Math randomInt:self.canvas.height];
    }
    p.x-= speed; // Adding accelaration
    s.center = p; // making the shape follow the point
}

问题在于：

s.center = p; // making the shape follow the point

因为是将形状的所有中心点设置为同一点。但是，只有最后一点会有所不同。

您的方法应如下所示：

-(void) slide {
    //Calling the platforms again to add movement
    for (C4Shape *currentShape in platforms) {
        CGPoint currentCenterPoint = currentShape.center;
        if (currentCenterPoint.x <= 0 ) {
            // if it's smaller than the width of the cavas auto transport
            currentCenterPoint.x = self.canvas.width;
            // choose a different y coordinate for each
            currentCenterPoint.y = [C4Math randomInt:self.canvas.height];
        }
        currentCenterPoint.x-= speed; //Adding accelaration
        currentShape.center = currentCenterPoint;
    }
}

此外，请注意此方法重命名变量，因此代码更具可读性。这是一个很好的做法，可以帮助记住正在发生的事情，并让其他人能够更轻松地阅读您的代码。

注意：您真的很接近能够将这种功能添加到它自己的类中以使您自己的事情变得更简单，干得好！

于 2013-11-02T21:06:41.660 回答