Почему мой шар (объекты) не сокращается/исчезает?

Ответ 1

В вашем коде несколько проблем.

Во-первых, в вашем определении:

var shrink = function(p) {
    for (var i = 0; i < 100; i++) {
        p.radius -= 1;
    }

    function asplode(p) {
         setInterval(shrink(p),100);
        balls.splice(p, 1);
    }
}

asplode является локальным для области внутри shrink и поэтому недоступен для кода в update, где вы пытаетесь его вызвать. Область JavaScript основана на функциях, поэтому update не может видеть asplode, поскольку она не находится внутри shrink. (В вашей консоли вы увидите сообщение об ошибке: Uncaught ReferenceError: asplode is not defined.)

Сначала вы можете вместо этого перемещать asplode вне shrink:

var shrink = function(p) {
    for (var i = 0; i < 100; i++) {
        p.radius -= 1;
    }
}

function asplode(p) {
     setInterval(shrink(p),100);
     balls.splice(p, 1);
}

Однако у вашего кода есть еще несколько проблем, выходящих за рамки этого вопроса:

• setInterval ожидает функцию. setInterval(shrink(p), 100) вызывает setInterval, чтобы получить возвращаемое значение с немедленным вызовом shrink(p). Вероятно, вы хотите

  setInterval(function() { shrink(p) }, 100)
  

• Ваш код for (var i = 0; i < 100; i++) { p.radius -= 1; }, вероятно, не делает то, что вы думаете. Это немедленно приведет к операции декремента 100 раз, а затем визуально покажет результат. Если вы хотите повторно отобразить мяч при каждом новом размере, вам нужно будет выполнить каждый отдельный декремент внутри отдельного обратного вызова времени (например, операцию setInterval).

• .splice ожидает числовой индекс, а не объект. Вы можете получить числовой индекс объекта с помощью indexOf:

  balls.splice(balls.indexOf(p), 1);
  

• К тому времени, когда ваш интервал работает в первый раз, оператор balls.splice уже произошел (это произошло примерно 100 мс, если быть точным). Я предполагаю, что не то, что вы хотите. Вместо этого вы должны иметь функцию декрементирования, которая неоднократно вызывается setInterval и, наконец, выполняет balls.splice(p,1) после p.radius == 0.

Ответ 2

setInterval(shrink(p),100);

Это не делает то, что вы думаете. Это вызывает shrink, передает его p, а затем передает результат на setInterval. shrink(p) возвращает undefined, поэтому эта строка фактически не помещает ничего на интервал.

Вероятно, вы хотите:

setInterval(function(){
    shrink(p)
}, 100);