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

私はあなたのコードで間違っている2つのことに気付きました。まず、 ' - px'を '100px'から切り離すために - 演算子を使用することはできません。 screen.width（またはheightプロパティ）を実際に使用する場合は、数字で始まる文字列ではないので、最初にvar screenwidth = screen.widthを実行する必要はありません。

2番目のことは、Math.randomは関数であり、このように呼び出す必要があることです。Math.random()。

あなたのアニメーションは何らかの人生になったので、残りの作業はアニメーションのロジックに関係していると思います。