2011-08-02 32 views
0

我有一個循環不會使用Prototype + Scriptaculous運行。它爲我的數組中的第一個對象運行一次然後停止。PrototypeJS循環將不會運行

var myMessages = new Object(); 
     myMessages = ['success','info','warning','error']; // define the messages types  
     function hideAllMessages() 
     { 
       var messagesHeights = new Array(); // this array will store height for each 

enter code here 


     // This one runs just once for the first item in the array 
       var i = 0; 
       myMessages.each(function(element) { 
        alert(element); 
        messagesHeights[i] = $('.' + element).getHeight(); 
        i++; 
        $$('.' + element + ' message').invoke('hide'); 
       }); 


       //This won't run at all=============================== 
       for (var index = 0; index < myMessages.length; index++) 
       { 
         messagesHeights[index] = $('.' + myMessages[index]).getHeight(); 
         $('x').hide(); 
         //$('.' + myMessages[i]).css('top', -messagesHeights[i]); //move element outside viewport  
       } 
     } 
+0

看看你的瀏覽器的控制檯。有什麼錯誤信息?我敢打賭,你有一個分配給數組'messagesHeights'的問題,它還沒有長度。 – clockworkgeek

+0

@clockworkgeek - 你說得對,他可能有錯誤信息,但最有可能的是'$('。'+ element)'返回null。允許以這種方式分配給數組。 –

+0

你說得很對。 'element.getHeight()'就足夠了。整個片段還有更多問題,將高度保持爲陣列沒有意義。 – clockworkgeek

回答

2

我不是一個原型用戶,但這裏是我看的更遠:

$是標識。我相信你需要$$這裏:

$$('.' + element) 

這將返回一個數組,所以我認爲你需要invoke()這樣的:

$$('.' + element).invoke('getHeight'); 

此外,.each()通過索引作爲第二個參數回調,所以你不需要維護你自己的i

myMessages.each(function(element, i) { 

而且,這樣的:

$$('.' + element + ' message') 

...似乎在尋找與標籤命名message元素。我假設你想要一個班級。

$$('.' + element + ' .message').invoke('hide'); 
+0

[工作示例](http://jsfiddle.net/ANQ3c/)。我刪除了'for'循環,因爲它好像只是複製'.each()'。但它似乎與'$'和'$$'有相同的問題。 – user113716