Почему мои яйца исчезают? [закрыто]

202

Простите за смешное название. Я создал небольшую графическую демонстрацию, в которой 200 шариков подпрыгивают и сталкиваются, как на стенах, так и друг на друга. Вы можете увидеть, что у меня есть в настоящее время здесь: http://www.exeneva.com/html5/multipleBallsBouncingAndColliding/

Проблема в том, что всякий раз, когда они сталкиваются друг с другом, они исчезают. Я не уверен почему. Может кто-нибудь взглянуть и помочь мне?

ОБНОВЛЕНИЕ: Очевидно, что массив шаров имеет шары с координатами NaN. Ниже приведен код, где я помещаю шары в массив. Я не совсем уверен, как координаты получают NaN.

// Variables
var numBalls = 200;  // number of balls
var maxSize = 15;
var minSize = 5;
var maxSpeed = maxSize + 5;
var balls = new Array();
var tempBall;
var tempX;
var tempY;
var tempSpeed;
var tempAngle;
var tempRadius;
var tempRadians;
var tempVelocityX;
var tempVelocityY;

// Find spots to place each ball so none start on top of each other
for (var i = 0; i < numBalls; i += 1) {
  tempRadius = 5;
  var placeOK = false;
  while (!placeOK) {
    tempX = tempRadius * 3 + (Math.floor(Math.random() * theCanvas.width) - tempRadius * 3);
    tempY = tempRadius * 3 + (Math.floor(Math.random() * theCanvas.height) - tempRadius * 3);
    tempSpeed = 4;
    tempAngle = Math.floor(Math.random() * 360);
    tempRadians = tempAngle * Math.PI/180;
    tempVelocityX = Math.cos(tempRadians) * tempSpeed;
    tempVelocityY = Math.sin(tempRadians) * tempSpeed;

    tempBall = {
      x: tempX, 
      y: tempY, 
      nextX: tempX, 
      nextY: tempY, 
      radius: tempRadius, 
      speed: tempSpeed,
      angle: tempAngle,
      velocityX: tempVelocityX,
      velocityY: tempVelocityY,
      mass: tempRadius
    };
    placeOK = canStartHere(tempBall);
  }
  balls.push(tempBall);
}
Ян Пульс
источник
119
Это получит мой голос, даже если только за лучший вопрос года!
Алекс

Ответы:

97

Ваша ошибка изначально исходит из этой строки:

var direction1 = Math.atan2(ball1.velocitY, ball1.velocityX);

У вас есть ball1.velocitY(что есть undefined) вместо ball1.velocityY. Так что Math.atan2дает вам NaN, и это NaNзначение распространяется через все ваши расчеты.

Это не источник вашей ошибки, но есть еще кое-что, что вы можете изменить в этих четырех строках:

ball1.nextX = (ball1.nextX += ball1.velocityX);
ball1.nextY = (ball1.nextY += ball1.velocityY);
ball2.nextX = (ball2.nextX += ball2.velocityX);
ball2.nextY = (ball2.nextY += ball2.velocityY);

Вам не нужны дополнительные назначения, и вы можете просто использовать +=один оператор:

ball1.nextX += ball1.velocityX;
ball1.nextY += ball1.velocityY;
ball2.nextX += ball2.velocityX;
ball2.nextY += ball2.velocityY;
Павел
источник
20

В функции есть ошибка collideBalls:

var direction1 = Math.atan2(ball1.velocitY, ball1.velocityX);

Так должно быть:

var direction1 = Math.atan2(ball1.velocityY, ball1.velocityX);
Альф
источник