JSでローテーバナーを作ってますが不具合について教えてください

質問

下記のサイトを参考にローテバナーを作ってます。
http://codess.heteml.jp/sample/ad/rotation.html

ですが、参考サイトのHTMLに記述のスクリプトを外部ファイルにすると
動作しないのです。

検証したら一番最後の行だけ(他は外部JSのまま）
Event.observe(window, "load", pageInit);
をHTMLに直書きすれば動作るのです。

この記述も含めて外部ファイルにして動作するには
どうしたら良くて、何が原因で動かないのでしょうか？
ご教示いただければと思います。
よろしくお願いします。

<script type="text/javascript">
var ads = null;
var currentIndex = null;
/**
* 広告初期化処理
*/
function adsInit(adsId) {
// 複数の広告を含む広告枠
var obj = $(adsId);

if (obj) {
// クラス名 ad で広告オブジェクトを検索
ads = obj.getElementsByClassName("ad");

// 最初のバナー以外を隠す
for (i=1; i<ads.length; i++) {
ads[i].hide();
}
currentIndex = 0;
}

// 広告枠を表示（非表示クラスを解除）
obj.className = "";
}
/**
* ローテーション初期化処理
*/
function rotationInit() {
setInterval(rotation, 5000);
}
/**
* ローテーション
*/
function rotation() {
// 表示中の広告を隠す
ads[currentIndex].hide();

currentIndex++;
if (currentIndex >= ads.length) {
currentIndex = 0;
}

// 次の広告を表示
ads[currentIndex].show();
}
/**
* ページ初期化処理
*/
function pageInit() {
adsInit("ads");
rotationInit();
}
Event.observe(window, "load", pageInit);
</script>

回答No.1

fujillin · ベストアンサー

一番ありそうなのが、

prototypeかjqueryを使っていると思うけど、読み込む順番が後になっているとか…？

お礼

順番を逆にしたらFFで表示されました！
ありがとうございます。
ただ、IE7では上手く動作しないので、CSSの記述が問題のような感じです。
もうちょっと調べてみます。