Сохранение SKSpriteNode в границах экрана

Question

Я пытаюсь проверить, останется ли мой SKSpriteNode в границах экрана во время жест перетаскивания. Я дошел до такой степени, что почти уверен, что моя логика к решению проблемы правильная, но моя реализация неверна. В принципе, прежде чем игрок перейдет из перевода, программа проверяет, не находится ли она в границах. Вот мой код:

-(CGPoint)checkBounds:(CGPoint)newLocation{ 
    CGSize screenSize = self.size; 
    CGPoint returnValue = newLocation; 
    if (newLocation.x <= self.player.position.x){ 
    returnValue.x = MIN(returnValue.x,0); 
    } else { 
     returnValue.x = MAX(returnValue.x, screenSize.width); 
    } 

    if (newLocation.y <= self.player.position.x){ 
    returnValue.y = MIN(-returnValue.y, 0); 
    } else { 
    returnValue.y = MAX(returnValue.y, screenSize.height); 
    } 

    NSLog(@"%@", NSStringFromCGPoint(returnValue)); 
    return returnValue; 
} 
-(void)dragPlayer: (UIPanGestureRecognizer *)gesture { 
      CGPoint translation = [gesture translationInView:self.view]; 

      CGPoint newLocation = CGPointMake(self.player.position.x + translation.x, self.player.position.y - translation.y); 
    self.player.position = [self checkBounds:newLocation]; 
} 

По какой-то причине мой плеер уходит с экрана. Я думаю, что мое использование макросов MIN & MAX может быть неправильным, но я не уверен.

Answer 1

Точно, вы перепутали MIN/MAX. Строка MIN(x, 0) вернет нижнее значение x или 0, что означает, что результат будет 0 или меньше.

В одной строке вы используете -returnValue.y, что не имеет смысла.

Вы можете (и должны для чтения) опустить if/else, потому что MIN/MAX, если он используется правильно, сделать, если/else здесь не нужно.