jQuery offsetをリサイズ時に随時取得したい

Question

サイドバーをfixedを使い、追尾させたいです。

jQueryでleft位置をサイドバーに与えてやるのですが、ウィンドウサイズを変更したときに前回取得したoffset.leftがそのまま適応されてしまいます。

ですので、リサイズした時に随時offsetを取得できないかと調べつつコードを書いてみたのですが、うまくいきませんでした。

それが下記のソースです。

$(function(){

    function fix() {
        window.onload = function () {

            var Box = $('.sidebar'),
                offset = Box.offset();
            var BoxL = offset.left;
            var BoxT = offset.top;

            $(window).scroll(function () {
                if($(window).scrollTop() < offset.top - 20) {
                    fixBox.css({
                        position: 'absolute',
                        top: 'inherit',
                        left: 'inherit',
                    });
                } else {
                    fixBox.css({
                        position: 'fixed',
                        top: '20px',
                        left: fixBoxL,
                    });
                }
            });
        }
    }

    $(window).on('resize', function() {
        fix();
    });

    fix();

});

理想は、ロードした時ではなく、リサイズした瞬間にfixBoxLにoffset.leftの値が入ることです。

Answer 1

fix()内部でwindow.onloadを書き換えていますが、window.onloadは1度しか実行されませんので、実行後に書き換えても何も起きません。

取るべき手段としては、

• サイズ取得ルーチンをfix()の中に直接書く
• fix()をjQuery実行の1回と、onresizeで実行させる

のようになります。なお、resizeは頻繁に起きるイベントですので、タイマーなどでメイン処理の実行を減らしたほうがいいかもしれません。

Answer 2

レビューしました。

• jQueryの$(fn)自体がwindow.onload = fnの完全上位互換みたいなもの

window.onload = fnは後勝ちで上書きされ、前の関数が動作しなくなる害悪設定です。二度と使う事はないので忘れよう

• 単なる関数はホイスト（巻き上げ）に気をつけよう

おすすめは普通に変数宣言してそれに突っ込む方法

• fixは外に出そう。

内部で宣言されている変数は全てfix外で管理し、fixは値を書き換えるのに集中

• fixBoxやfixBoxLが未定義だね。fix関数の中身だから頭にfix付けてみただけ？
• $(window).on('resize', fn)の第二引数に無名関数を作って渡してるけど、thisを使わない設計なのでそのままfixを渡してしまったほうが簡潔
• jQueryオブジェクトは慣習的に変数名の頭に$を付ける人が多い
• オブジェクトのケツカンマはChromeでは動くけど、IEでは動作しなかったような…まぁ優先度低
• オフセット値を書き換えたら画面更新しないと駄目でしょ

---

上記を踏まえてコードをリファクタリングしました。
※fixBox、fixBoxLは$BoxとBoxLなんだろうなと解釈したので変数名を変えてます。

$(function() {
    var $Box = $('.sidebar'),
        offset, BoxL, BoxT;
    var fix = function() {
        offset = $Box.offset();
        BoxL = offset.left;
        BoxT = offset.top;
    }
    var render = function() {
        if ($(window).scrollTop() < offset.top - 20) {
            $Box.css({
                position: 'absolute',
                top: 'inherit',
                left: 'inherit',
            });
        } else {
            $Box.css({
                position: 'fixed',
                top: '20px',
                left: BoxL,
            });
        }
    }
    var refresh = function() {
      fix();
      render();
    }

    refresh();
    $(window).on('resize', refresh);
    $(window).on('scroll', render);
});