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

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

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

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

解決済

jQueryで変数の定義を1回で済ませたい

yuta_saito
yuta_saito

総合スコア2

jQuery

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

1回答

0評価

0クリップ

287閲覧

投稿2022/01/06 07:58

編集2022/01/09 15:09

前提・実現したいこと

ページを開いて1秒経ったらバナーを表示。
スクロール中はバナーを非表示にし、スクロールが止まったらバナーを表示。
フッターに入った場合、バナーは非表示。
slideinというclassをつけることで表示非表示を切り替えています。

発生している問題・エラーメッセージ

jQueryで変数が下の関数で読み込まれません...
いろいろ調べたのですが、原因が特定できず、ご教示いただけますでしょうか...

該当のソースコード

jQuery

$(function(){ var fixedBanner = $('.fixed-banner'); fixedBanner.delay(1000).queue(function(){ fixedBanner.addClass('slidein'); }); $(window).scroll(function () { var check = window.pageYOffset; var docHeight = $(document).height(); var dispHeight = $(window).height(); fixedBanner.removeClass('slidein'); if(check > docHeight - dispHeight - 134) { fixedBanner.removeClass('slidein'); } }); $(window).on("scrollstop",function(){ var check = window.pageYOffset; var docHeight = $(document).height(); var dispHeight = $(window).height(); if(check > docHeight - dispHeight - 134) { fixedBanner.removeClass('slidein'); } else { fixedBanner.addClass('slidein'); } }); });

scrollstop関数は別で関数を作成しており、スクロールが止まったら発火するようにしております。
下記の変数を2回宣言しているため、できれば1回で済ませたいです。

jQuery

var check = window.pageYOffset; var docHeight = $(document).height(); var dispHeight = $(window).height();

試したこと

上記の3行を外に出してみたが、意図した挙動にならない。

jQuery

$(function(){ var fixedBanner = $('.fixed-banner'); var check = window.pageYOffset; var docHeight = $(document).height(); var dispHeight = $(window).height(); fixedBanner.delay(1000).queue(function(){ fixedBanner.addClass('slidein'); }); $(window).scroll(function () { fixedBanner.removeClass('slidein'); if(check > docHeight - dispHeight - 134) { fixedBanner.removeClass('slidein'); } }); $(window).on("scrollstop",function(){ if(check > docHeight - dispHeight - 134) { fixedBanner.removeClass('slidein'); } else { fixedBanner.addClass('slidein'); } }); });

お手数をおかけ致しますが、よろしくお願いいたします。

良い質問の評価を上げる

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

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

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

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

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

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

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

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

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

int32_t

2022/01/06 08:12

> jQueryで変数が下の関数で読み込まれません... 「変数が読み込まれない」とはどういう意味でしょうか。どういう動作を期待していて、実際はどうなりますか?
yuta_saito

2022/01/06 14:59

ご返信いただき、ありがとうございます! 質問が言葉足らずで申し訳ありません。 期待している動作としては、上記に記載した 「ページを開いて1秒経ったらバナーを表示。 スクロール中はバナーを非表示にし、スクロールが止まったらバナーを表示。 フッターに入った場合、バナーは非表示。」 であり、「該当のソースコード」に記載しているソースコードで実装できました。 ただ、変数を2回定義しているため、「変数の定義を1回で済ませたい」というのが、今回やりたいことです。 お手数をお掛け致しますが、何卒よろしくお願いいたします。
int32_t

2022/01/06 15:23

> 変数の定義を1回で済ませたい それをこの質問のタイトルにすべきかと思います。
yuta_saito

2022/01/09 15:10

返信が遅くなり申し訳ありません。 大変失礼いたしました! タイトルを修正いたしました!

まだ回答がついていません

会員登録して回答してみよう

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

ただいまの回答率
87.20%

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

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

質問する

関連した質問

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

jQuery

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