てきとうなさいと べぇたばん

JavaScriptのリファクタリングをしたのだ

変更する前のソース。 変更したあとの現在のソース。大体半分超ほど。

このサイトのJavaScriptを書き換えたかった

ちょっとコードが汚かったと思った。jQueryを使っているのに、jQueryのような書き方になっていなかった。

修正内容

  • 画像ビューワーの自作プラグインから、jQuery.mousewheelを削除して、ライブラリの外から操作するよう修正
  • 画像ビューワーの現在見ている画像の管理方法を、配列で保存するのではなく、jQueryで探すように修正
  • 出来る限り同じコードを関数にして再利用

これで、修正前が88行だったものが、修正後には47行にまで減らせた。めでたしめでたし。

疑問点

  • 「#hoge aが一つでも存在したら」という条件で何かを実行しようとすると、eachメソッドを使わざるを得ない状況になった。loadとかでどうにかならなかったものか
  • jQueryのnextとprevは、nextで次がないとき、prevで前が無いときに反対方向へ進むときにはどうしたら良かったのだろうか。