Застрял с анимацией JavaScript

Question

Я не писал код в возрасте, и я решил сделать забавный маленький веб-сайт, чтобы освежить мои навыки. К сожалению, я застрял в анимации глиняного голубя (я делаю интерактивную штуку). Я застрял на этом фрагменте кода уже несколько дней, и я не могу опустить голову. Любая помощь будет высоко оценена.

Вот сегмент мой код, где я застрял:

var uwifh = 1; 
 
if (uwifh == 1) { 
 
    var screenwidth = screen.width-'px'; 
 
    var screenheight = screen.height-'px'; 
 

 
    function lanceerklei() { 
 
     var e = document.getElementById('clay'); 
 
     e.style.visibility = 'visible'; 
 

 
     var positie_yinit= Math.floor(Math.random*(screenheight/1.7))+50; 
 
     var positie_y; 
 
     var positie_x = 10; 
 
     var paraboolfactor = (Math.floor(Math.random * 50) + 1)/1000; 
 
     var parabooltop = screenwidth/2; 
 
     var snelheid = Math.floor(Math.random*20)+20; 
 

 
     var elke20ms; 
 
     elke20ms= setInterval(function(){ 
 
      positie_x= positie_x += snelheid; 
 
      positie_y= (-1 * paraboolfactor) * (positie_x - parabooltop)^2; 
 
      e.style.left = positie_x + 'px'; 
 
      e.style.top = positie_y + positie_yinit + screenheight + 'px'; 
 
     }, 20); 
 
    }; 
 

 
    window.setTimeout(lanceerklei, 1000); 
 
};

#clay{ 
 
    position: fixed; 
 
    height: 150px; 
 
    width: 150px; 
 
    visibility: hidden; 
 
    z-index: 11; 
 
    top: 10px; 
 
    left: 10px; 
 
}

<img src="http://www.threecountiesclayshoot.co.uk/communities/0/004/013/041/930//images/4619113458.png" id="clay">

Это не весь код Если вы хотите отдохнуть, я дам остальное.

Вновь любая помощь приветствуется.

Answer 1

Я заметил две вещи, которые неверны в вашем коде. Во-первых, вы не можете использовать оператор - для удаления «px» из «100px», например. Если вы действительно хотите использовать screen.width (или свойство height), нет необходимости либо в том, что это числовое значение, а не начало строки, вы должны просто сделать var screenwidth = screen.width.

Второе, что Math.random является функцией и должно называться так, как это Math.random().

С этими исправлениями ваша анимация пришла в какую-то жизнь, и я думаю, что оставшаяся работа касается вашей логики для анимации.