Координата Y не обновляется должным образом

Question

Я пытаюсь создать боковой скроллер, и у меня возникли проблемы с установкой координат моих объектов на случайные значения 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 

Answer 1

Ответ Бен об обновлении только переменной 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; 
    } 
} 

Кроме того, обратите внимание, что этот метод переименовывает переменные, так код более читаемым. Это хорошая практика, помогающая помнить, что происходит, а другим людям легче читать ваш код.

ПРИМЕЧАНИЕ: вы действительно близки к возможности добавлять эту функциональность к собственному классу, чтобы сделать вещи проще для вас, хорошая работа!

Answer 2

Координата обновляется отлично - на одной точке, которую вы проверяете и заходящую все центры ваших C4Shapes' в методе скольжения.

В настройке это прекрасно работает, поскольку вы устанавливаете центр C4Shape s на p перед его изменением, но когда вы переходите к циклу For в методе слайдов, вы просто регистрируете и обновляете эту точку, которая начинается в последнем месте, когда вы обновили его в настройке, если ничего не происходит между тем, когда вызывается установка и слайд. p является ivar вашего класса C4Workspace, поэтому вы получаете один экземпляр. Чтобы исправить это, я считаю, что вы должны менять каждое появление p в слайде на s.center и избавляться от последней строки.

Кстати, вы должны серьезно подумать о переименовании переменных в этих методах, их очень сложно понять - я на самом деле очень смущен, почему p является ivar и не объявлен в настройке, что, кажется, единственное место, в котором вы нуждаетесь.