setTimeoutで画像を動かす

Question

ゲーム作成などにおいて、制限時間を表すと言う目的で、アイコン画像をA地点からB地点に動かそうと思っています。B地点にたどり着いたらGAMEOVERと言うものです。
マリオを動かす際に、style.left += "50px"と言う風にしてしまうと、setTimeoutにしても、そこからさらに動くことはなく、ずっと同じ場所で止まってしまいます。
予想ですが、"50px"とすると、数値を足しあげるというよりは、そこの場所指定になってしまうのかな、と思っています。
setTimeoutなどで、毎秒少しずつ画像を動かす処理を考察していただけると幸いです。

function MarioMove() {
  timerId = setTimeout(step, 1000);
}

function step() {
 Mario.style.left += "50px";
 if(time === 0){
   clearTimeout(timerId);
   countdown = false;
 } else {
   timerId = setTimeout(step, 1000);
 }
}

Answer 1

例えば、Mario.style.leftに"10px"と入っていたとしましょう。
Mario.style.left += "50px";で行われるのは文字列と文字列の足し算ですから、"10px50px"という文字列がMario.style.leftに入ろうとしますが、そんな指定の仕方はないので無効になります。
やるとしたら、parseInt()などでMario.style.leftを数値にして、50を足してから、後ろに"px"を足して文字列にして、Mario.style.leftに入れてあげればいいのではないでしょうか。

Mario.style.left = (parseInt(Mario.style.left) + 50) + "px";

Comments

[Asylum35aO2]

文字列と数列、基本的なことでしたねお恥ずかしい。
お陰さまでまた１つ成長できました。ありがとうございました。

Answer 2

style.left += "50px"

style.leftがnullだったときに加算できません
また文字列だったときにも加算できません
50pxに50pxを足すと50px50pxですね

たとえばこう

<script>
var time=5;
var Mario;
window.addEventListener('DOMContentLoaded', function(e){
  Mario=document.querySelector('#mario');
  MarioMove();
});
function MarioMove() {
  timerId = setTimeout(step, 1000);
}

function step() {
  var l=Mario.style.marginLeft;
  l=(reg=l.match(/\d+/))?reg[0]:"0"; //数値を含めばその数、なければ0
  l=parseInt(l);                     //文字列を数値化
  l+=50;                             //数値に50を加算
  l+="px";                           //単位を付加
  console.log(l); 
  Mario.style.marginLeft = l;        //値を代入
  time--;
  if(time === 0){
   clearTimeout(timerId);
 } else {
   timerId = setTimeout(step, 1000);
 }
}
</script>

<img id="mario" src="https://placehold.jp/ff0000/00ffff/100x100.png?text=mario">

※コメントつけておきました

解説

l=(l.match(/\d+/))?reg[0]:"0";
```javascript
上記分解すると
```javascript
reg=l.match(/\d+/)

\dは0-9の数字それに+がついてるので1文字以上の数字
それをスラッシュで挟んでmatchにあたえると
変数lの文字列の中にある数字を拾い、結果をregという変数に配列として返します。

l=条件?reg[0]:"0";

条件の中でregに数値がはいっていればその数値であるreg[0]をlに代入します
そうでないとき（つまりlに数値が含まれないとき）にはlには"0"を代入します
これを三項演算子といいます

結果としてlの中から数値だけを取り出してlに入れ直しています

Comments

[Asylum35aO2]

確かに文字列と数字は並べないですね。基本的な事が分かっていませんでした。ご指摘感謝致します。
御手数ですが、
l=(reg=l.match(/\d+/))?reg[0]:"0";
の処理の説明をもう少し頂けますか？
全体的に、特にmatch(/\d+/))辺りが難しいです。

[yambejp]

追加の質問気が付きませんでした。追記しておきます