Javascript для цикла не будет работать

Question

Это мой код:

<div class="div"> 
<div id="picture"><img src="news1.jpg" /></div> 
<div id="picture1"><img src="news2.jpg" /></div> 
<div id="picture2"><img src="news3.jpg" /></div> 
</div> 
<script type="text/javascript"> 

var element = document.getElementById("picture") 
var element1 = document.getElementById("picture1") 
var element2 = document.getElementById("picture2") 


var zdjecia = new Array("element", "element1", "element2"); 
    for(var i=0; i < zdjecia.Length; i++) 
     { 
     zdjecia[i].style.opacity="1"; 
     } 
</script> 

Не могли бы вы дать мне некоторые советуют, почему этот цикл не будет работать?. Если я найду проблему, я добавлю в нее параметр settimeout и setTimeInterval, но в данный момент я не могу найти, где проблема с этим циклом.

Answer 1

Остальные показали вам два кодирования сделанные вами ошибки (неправильная капитализация на .length и используя строки вместо прямых ссылок на переменные), но я хотел бы предложить это изменение кода:

var items = ["picture", "picture1", "picture2"]; 
for (var i = 0; i < items.length; i++) { 
    document.getElementById(items[i]).style.opacity = 1; 
} 

Answer 2

Вместо добавления элементов в виде строк, добавить элементы, как они

var zdjecia = new Array(element, element1, element2); 

и свойство длины должны быть доступны с zdjecia.length

Answer 3

уборщицей подход

<div class="div"> 
    <div class="picture"><img src="news1.jpg" /></div> 
    <div class="picture"><img src="news2.jpg" /></div> 
    <div class="picture"><img src="news3.jpg" /></div> 
</div> 

<script> 
    var zdjecia = document.getElementsByClassName('picture'); 
    for(var i = 0; i < zdjecia.length; i++) { 
    zdjecia[i].style.opacity = '1'; 
    } 
</script> 

Произошел код

Вам необходимо выполнить цикл через фактические элементы DOM, а не строки с одинаковыми именами переменных var zdjecia = new Array(element, element1, element2);.

Кроме того, JavaScript чувствителен к регистру, и вы случайно заглавны length. zdjecia.Length должно быть zdjecia.length.

[1, 2, 3].Length // => undefined 
[1, 2, 3].length // => 3 

Кроме того, желательно использовать массив в JavaScript буквального (квадратная форма скобки), чтобы инстанцировании новый массив:

var zdjecia = [element, element1, element2]; 

массива в JavaScript буквальный более предсказуем, чем прямое использование. Рассмотрим следующий пример:

new Array(2, 1) // => [2, 1] 
new Array(2) // => [undefined, undefined] 

Резерв прямое использование new Array() для помещения в предварительное выделение массива.

Answer 4

Исправленный код:

var zdjecia = new Array(element, element1, element2); //Array of objects, not strings 
for(var i=0; i < zdjecia.length; i++)  //Use .length instead of .Length 
    { 
    zdjecia[i].style.opacity="1"; 
    } 

веселит

Answer 5

У вас есть несколько незначительных ошибок. Во-первых, вы добавляете элементы как строки, а не как сами элементы. Во-вторых, вы капитализировали «длину» в своем цикле for. Не делай этого.

var zdjecia = new Array(element, element1, element2); 

for(var i=0; i < zdjecia.length; i++){ 
zdjecia[i].style.opacity="1"; 
}