meine for-Schleife funktioniert nicht und ich weiß nicht warum

Question

Ich habe jQuery installiert und alle Variablen sind früher im Code festgelegt, aber alles nach der for-Schleife funktioniert nicht wie die Warnung ('Hallo'). Wenn irgendjemand etwas damit falsch sehen kann, sag es mir bitte.

bearbeiten

var max=438; 
localStorage.setItem('max', max); 
var pi=[]; 
pi=JSON.parse(localStorage.getItem('pi')); 
var i=localStorage.getItem('i'); 
var url1=Math.floor((Math.random() * max) + 1); 
var url2=Math.floor((Math.random() * max) + 1); 
if(url1==url2){ 

url2=url2+2; 
if(url2>max){ 
    url2=url2-max; 
} 
} 
document.getElementById('imgimg1').src='../img/'+url1+'.gif'; 
document.getElementById('imgimg2').src='../img/'+url2+'.gif'; 
var v1=url1-1; 
var v2=url2-1; 
function vote1() { 
    pi[v1]=pi[v1]+(pi[v2]/pi[v1]); 
    localStorage.setItem('pi',JSON.stringify(pi)); 
    location.reload(); 
} 
function vote2() { 
    pi[v2]=pi[v2]+(pi[v1]/pi[v2]); 
    localStorage.setItem('pi',JSON.stringify(pi)); 
    location.reload(); 
} 
$(document).load(function(){ 
    for (var i = 0; i <max; i++) { 
     if(pi[i]===undefined||pi[i]===null||pi[i]==0){ 
      pi[i]=1; 
     } 
    } 
}); 
alert(pi); 

dies ist der vollständige Code

Answer 1

mit diesen Versuchen:.!

/* assuming this */ 
var max = 4; // for example 
var pi = [0, 0, 0, 0]; 
/* end assuming this */ 

$(document).ready(function() { 
    for (var i = 0; i < max; i++) { 
     if(pi && pi[i] && pi[i] < 0) { 
      pi[i] = -1; 
     } else { 
      pi[i] = 1; 
     } 
    } 
    alert(JSON.stringify(pi)); 
}); 

Es sollte schreiben '[1, 1, 1, 1]'

[EDIT]

In Bezug auf Ihren gesamten Code, warum müssen Sie nach dem Dokument repetieren?

Warum init nicht pi wie folgt aus:

var max=438; 
localStorage.setItem('max', max); 
var pi=[]; 
pi=JSON.parse(localStorage.getItem('pi')); 
for(var i = 0; i < max; i++) { 
    pi[i] = 1; 
} 
[...] 

Answer 2

Unter der Annahme, alle Variablen richtig eingestellt sind, sollte Ihr Code sein:

$(document).load(function(){ 
    for (var i = 0; i <max; i++) { 
     if(pi[i]===undefined||pi[i]===null||pi[i]==0){ 
      pi[i]=1; 
     } 
    } 
    alert('hello'); 
}); 

Vielleicht wird dies helfen, obwohl ich nicht tun wissen, worauf Sie diese Variablen einstellen. Bitte beachten Sie, dass ich anstelle des Ladens von Dokumenten das jquery ready-Ereignis verwende.

<div id="message"></div> 
 

 
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> 
 

 
<script> 
 
    var max = 100; 
 
    var pi = []; 
 
    
 
    $(function() { 
 
    for (var i = 0; i <max; i++) { 
 
     if(pi[i]===undefined||pi[i]===null||pi[i]==0){ 
 
      pi[i] = 1; 
 
     } 
 
    } 
 
    $('#message').text(pi.length); 
 
}); 
 
</script>

Answer 3

$(document).onload(function(){ 
    for (var i = 0; i <max; i++) { 
     if(typeof pi[i]==="undefined"||pi[i]===null||pi[i]==0){ 
      pi[i]=1; 
     } 
    } 
}); 
alert('hello'); 

Answer 4

Ich nehme an, Sie "max" auf einige positive Zahl haben. Ich denke, das Problem ist, dass Sie $ (Dokument) codiert. onload (... wenn Sie $ codiert haben sollten (Dokument) bereit (... Hoffnung, die hilft PK