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

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

ただいまの
回答率

90.51%

  • jQuery

    8133questions

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

初心者(自分)が書いたjQueryのソースをきれいにしたい。

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 166
退会済みユーザー

退会済みユーザー

アニメーションGIFをWEBページで表示させる際に動画埋め込み再生風のデザインで実装したいと考えています。
静止画の上に配置した、なんちゃってPNG再生ボタンを押すとYouTubeのようにアニメーションGIFが再生し、再生終了後は元に戻ります。4秒ほどしかないので停止ボタンはありません。

一応見た目は実現したのですが、jsのソースがたぶん汚いです。
cssとHTMLはなるべく追加変更せずにjsのみでもう少しスマートに書けそうなところが
ありましたらご指南お願いいたします。

このような順番になっています。

  1. 静止画(01.jpg)を表示
  2. 静止画の上下左右中央に再生ボタン(#btnImg、03.png)が配置されています。
  3. 03.pngのボタンはマウスオーバーで透過状態に変化があったりカーソルが指アイコンになったりします。
  4. 再生ボタンをクリックするとアニメーションGIF(02.gif)が再生されます。
  5. アニメーションGIF再生中はボタン(03.png)が消えます。
  6. 再生ボタンをクリックしてから指定時間(5秒)が過ぎると
  7. 1の静止画にボタンが乗っている状態に戻ります
<!DOCTYPE html>
<html lang="ja">
<head>
<title></title>
<link href="./base.css" rel="stylesheet">
<script src="./jquery-3.2.1.min.js"></script>
<script src="./test.js"></script>
</head>
<body>

<div class="relative">
<img id="changeImg" src="01.jpg">
<img id="btnImg" class="absolute" src="03.png">
</div>

</body>
</html>
.relative {
    position: relative;
    width: 600px;
}
.absolute {
  margin: auto;
  position: absolute;
  top: 0;
  left: 0;
  right: 0;
  bottom: 0;
}

.relative.btn-hidden #btnImg {
  display: none; 
} 
$(function() {
  $("#btnImg").css({ opacity: ".65" });

  $("#btnImg").hover(
    function() {
      $(this)
        .stop()
        .animate({ opacity: "1" });
    },
    function() {
      $(this)
        .stop()
        .animate({ opacity: ".65" });
    }
  );

  $("#btnImg").css("cursor", "pointer");

 var $outer = $('.relative');

  $("#btnImg").on('click', function() {
    $("#changeImg").attr({ src: "02.gif" });
    $outer.addClass('btn-hidden');
    $(this).css("cursor", "auto");


    setTimeout(function() {
      $("#changeImg").attr({ src: "01.jpg" });
      $("#btnImg").attr({ src: "03.png" });
      $("#btnImg").css({ opacity: ".65" });
      $outer.removeClass('btn-hidden');
      $("#btnImg").css("cursor", "pointer");
    }, 5000);

  });
});
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • mts10806

    2018/11/12 20:58

    作業依頼に等しいですし単に「見て直して」だけだと回答者の負担が大きすぎます。同じプロジェクトの新人が書いたコードならまだしも、赤の他人の全く馴染みのない要件のコードです。想定通り動いているのならそれでよしとし、もし質問をしたいのであればもっとポイントを絞るべきではないでしょうか

    キャンセル

回答 1

0

hover
非同期
click
setTimeout
内の処理を全て外で関数にして、
var
onloadのcss変更
は上にまとめるのはどうでしょうか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

  • jQuery

    8133questions

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