質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

CSS

CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。

Q&A

解決済

1回答

1359閲覧

スクリプトが動かない

nosonosolife

総合スコア42

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

CSS

CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。

0グッド

0クリップ

投稿2016/10/12 12:31

「balloon」という名前の要素が画面内をジグザグしながら行ったり来たりするスクリプトが動かないのですが、原因が分かりません。
ご教授よろしくお願いいたします。

javascript

1function Moving(){ 2var base_x; 3var base_y; 4var max_speed = 3; 5var index_x = 0; 6var index_y = 0; 7var vx1 = 1; 8var vx2 = 1; 9var vy1 = 1; 10var vy2 = 1; 11vx1 = Math.ceil(Math.random() * max_speed); 12 13if(Math.round(Math.random)) vx2 *= -1; 14vy1 = Math.ceil(Math.random() * max_speed); 15 16if(Math.round(Math.random)) vy2 *= -1; 17base_x = $("#balloon").css("left"); 18base_y = $("#balloon").css("top"); 19 20setInterval(function(){ 21 index_x += (vx1 * vx2); 22 index_y += (vy1 * vy2); 23 24 if((index_x > 100) || (index_x < -100)){ 25 if(index_x > 100){ 26 index_x = 100; 27 }else{ 28 index_x = -100; 29 } 30 31 vx1 = Math.ceil(Math.random() * max_speed); 32 vx2 *= -1; 33 vy1 = Math.ceil(Math.random() * max_speed); 34 } 35 36 if((index_y > 20) || (index_y < -20)){ 37 if(index_y > 20){ 38 index_y = 20; 39 }else{ 40 index_y = -20; 41 } 42 43 vx1 = Math.ceil(Math.random() * max_speed); 44 vy1 = Math.ceil(Math.random() * max_speed); 45 vy2 *= -1; 46 } 47 48 $("#balloon").css('left', base_x + index_x ); 49 $("#balloon").css('top', base_y + index_y ); 50},200); 51} 52Moving();

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

HTMLとCSSが提示されていないので勘での回答ですが、

JavaScript

1base_x = $("#balloon").css("left"); 2base_y = $("#balloon").css("top");

ここは "100px" のように、単位付きで取得する可能性があります。そうなると、

JavaScript

1$("#balloon").css('left', base_x + index_x ); 2$("#balloon").css('top', base_y + index_y );

最後に足し合わせるときに "150px6" のような結果となるかもしれません。parseIntするなどして、数値に変換しておいたほうが良いでしょう。

また、数値同士の計算ができたとして、原則としてこのプロパティは単位が無いと有効な値として認識されませんので、最後は単位を付けたほうが良いかと思います。

投稿2016/10/12 13:56

sii_side

総合スコア849

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問