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

Question

前提・実現したいこと

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

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

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回で済ませたいです。

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

試したこと

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

$(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');
    }
  });
});

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

Comments

[int32_t]

> jQueryで変数が下の関数で読み込まれません...

「変数が読み込まれない」とはどういう意味でしょうか。どういう動作を期待していて、実際はどうなりますか?

[yuta_saito]

ご返信いただき、ありがとうございます！
質問が言葉足らずで申し訳ありません。

期待している動作としては、上記に記載した
「ページを開いて1秒経ったらバナーを表示。
スクロール中はバナーを非表示にし、スクロールが止まったらバナーを表示。
フッターに入った場合、バナーは非表示。」
であり、「該当のソースコード」に記載しているソースコードで実装できました。

ただ、変数を2回定義しているため、「変数の定義を1回で済ませたい」というのが、今回やりたいことです。

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

[int32_t]

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

それをこの質問のタイトルにすべきかと思います。

[yuta_saito]

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