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

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

ただいまの
回答率

90.75%

  • JavaScript

    15230questions

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

  • jQuery

    6305questions

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

JavaScriptでテキストが浮遊する動き

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 365

goldenb

score 14

画面内を浮遊するテキストを作っているのですがブラウザの枠内からはみ出ないようにしたいです。
画面の上と左は大丈夫なのですが下と右がはみ出てしまいます。
ここの部分の修正方法をご教授お願いします。

    var movement01 = {x:9, y:2};
    var movement02 = {x:-2, y:9};
    var movement03 = {x:9, y:-2};
    var movement04 = {x:7, y:4};

    $(function redraw(){

        var winWidth = window.innerWidth;
        var winHeight = window.innerHeight;

        var posLeft01 = parseFloat($('.maru01').css('left'));
        var posTop01 = parseFloat($('.maru01').css('top'));
        var posLeft02 = parseFloat($('.maru02').css('left'));
        var posTop02 = parseFloat($('.maru02').css('top'));
        var posLeft03 = parseFloat($('.maru03').css('left'));
        var posTop03 = parseFloat($('.maru03').css('top'));
        var posLeft04 = parseFloat($('.maru04').css('left'));
        var posTop04 = parseFloat($('.maru04').css('top'));

        posLeft01 += movement01.x;
        posTop01 += movement01.y;
        posLeft02 += movement02.x;
        posTop02 += movement02.y;
        posLeft03 += movement03.x;
        posTop03 += movement03.y;
        posLeft04 += movement04.x;
        posTop04 += movement04.y;

        if(posLeft01 < 0 || posLeft01 > winWidth){
            movement01.x *= -1;
        }
        if(posTop01 < 0 || posTop01 > winHeight){
            movement01.y *= -1;
        }
        if(posLeft02 < 0 || posLeft02 > winWidth){
            movement02.x *= -1;
        }
        if(posTop02 < 0 || posTop02 > winHeight){
            movement02.y *= -1;
        }
        if(posLeft03 < 0 || posLeft03 > winWidth){
            movement03.x *= -1;
        }
        if(posTop03 < 0 || posTop03 > winHeight){
            movement03.y *= -1;
        }
        if(posLeft04 < 0 || posLeft04 > winWidth){
            movement04.x *= -1;
        }
        if(posTop04 < 0 || posTop04 > winHeight){
            movement04.y *= -1;
        }

        $('.maru01').css({left:posLeft01, top:posTop01});
        $('.maru02').css({left:posLeft02, top:posTop02});
        $('.maru03').css({left:posLeft03, top:posTop03});
        $('.maru04').css({left:posLeft04, top:posTop04});

        setTimeout(redraw, 60);
    });
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

端まで来たかの判定に左上の一点が使われているのが原因です。
$('.maru01').width() とすると .maru01 要素の幅が取れるので、
これを用いて「右端」「下端」判定を行うと改善します。

        if(posLeft01 < 0 || posLeft01 + $('.maru01').width() > winWidth){
            movement01.x *= -1;
        }
        if(posTop01 < 0 || posTop01 + $('.maru01').height() > winHeight){
            movement01.y *= -1;
        }
        if(posLeft02 < 0 || posLeft02 + $('.maru02').width() > winWidth){
            movement02.x *= -1;
        }
        if(posTop02 < 0 || posTop02 + $('.maru02').height() > winHeight){
            movement02.y *= -1;
        }
        if(posLeft03 < 0 || posLeft03 + $('.maru03').width() > winWidth){
            movement03.x *= -1;
        }
        if(posTop03 < 0 || posTop03 + $('.maru03').height() > winHeight){
            movement03.y *= -1;
        }
        if(posLeft04 < 0 || posLeft04 + $('.maru04').width() > winWidth){
            movement04.x *= -1;
        }
        if(posTop04 < 0 || posTop04 + $('.maru04').height () > winHeight){
            movement04.y *= -1;
        }

■追記: オブジェクト指向を導入した書き方

これ系の「初期値は違うけど同じロジックで動くたくさんのオブジェクト」は、オブジェクト指向を使うとさっぱり書けることが多いです。
これだとliの数が100個とかいっちゃってもそんなにコードを複雑化させずに対応できます。

class MovingLi {
    constructor(selector, movement) {
        this.element = $(selector);
        this.movement = movement;
    }
    move () {
        let posLeft = parseFloat(this.element.css('left'));
        let posTop = parseFloat(this.element.css('top'));
        posLeft += this.movement.x;
        posTop += this.movement.y;
        if(posLeft < 0 || posLeft + this.element.width() > window.innerWidth){
            this.movement.x *= -1;
        }
        if(posTop < 0 || posTop + this.element.height() > window.innerHeight){
            this.movement.y *= -1;
        }
        this.element.css({left:posLeft, top:posTop});
    }
}

$(function () {
    const movingLiArray = [
        new MovingLi('.maru01', {x:9, y:2}),
        new MovingLi('.maru02', {x:-2, y:9}),
        new MovingLi('.maru03', {x:9, y:-2}),
        new MovingLi('.maru04', {x:7, y:4})
    ];
    function redraw () {
        movingLiArray.forEach(movingLi => movingLi.move());
    }
    setInterval(redraw, 60);
});

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/29 00:45

    ありがとうございます。解決しました。
    ちなみにこのJavaScriptのソースで書き方を直した方が良いところなどございますでしょうか?
    まだ手探りで始めたばかりなのでご教授いただけると幸いです。

    キャンセル

  • 2018/05/29 01:16

    回答に追記しました! → 直したほうが良いところ

    キャンセル

  • 2018/05/29 01:24

    ありがとうございます。無駄に多い記述がなく見易いです。
    しかしメソッドやオブジェクトの役割がわからないので調べながら一行づつ理解していきます。
    またわからない事があったら質問させてください。
    よろしくお願いいたします。

    キャンセル

  • 2018/05/29 01:27

    はい、またよろしくお願いしますbb

    キャンセル

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

  • ただいまの回答率 90.75%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • JavaScript

    15230questions

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

  • jQuery

    6305questions

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