JSLint: Niespodziewane "dla". Nieoczekiwany "var"

Question

Wyszukiwanie i wypróbowywanie innych sugestii w StackOverflow. Niestety odpowiedzi nie działają dla mnie. Sugerują użycie "foreach" zamiast "for", ale jak mogę ... jeśli chcę iterować tylko 50 razy? : <

Cóż, po prostu wkleję kod i zobaczymy, czy jakiś dobry lud może mi pomóc.

JSLint nie można było ukończyć.

Nieoczekiwane "za". dla (var i = 1; i < = 50; i + = 1) {
wiersz 6 kolumna 8

Nieoczekiwany "var". for (var i = 1, i < = 50, i + = 1) {
linia 6 kolumna 13

"use strict"; 

    var campo = []; 
    var ronda = 0; 

    // Llenamos el campo de 50 humanos/maquinas/extraterrestres = 150 jugadores 
    for (var i=1;i<=50;i+=1){ 
     campo.push(new Human("h"+i)); 
     campo.push(new Machine("m"+i)); 
     campo.push(new Alien("e"+i)); 
    } 

    // Array.prototype.suffle para barajar el Array 
    Array.prototype.shuffle = function() { 
     var input = this; 

     for (var i=input.length-1;i>=0;i-=1){ 
      var randomIndex = Math.floor(Math.random()*(i+1)); 
      var itemAtIndex = input[randomIndex]; 

      input[randomIndex]=input[i]; 
      input[i] = itemAtIndex; 
     } 
    }; 

    // Barajamos el Array campo 
    campo.shuffle(); 

    // Comprobamos que quedan más de 1 jugador por ronda 
    while (campo.length>1) { 
     console.log("Iniciando ronda: " + ++ronda); 
     console.log(campo.length + " jugadores luchando."); 
     // Recorremos el campo, y luchamos 

     var muertos = 0; 

     for (var i=0; i<campo.length-1; i+=2){ 
      // Caso de numero impar de jugadores: 
      // Por ejemplo cuando solo quedan 3 jugadores. Pelean 1 vs 2. El 3 se libra. 
      // - Si siguen vivos y aguantan otra ronda, se barajan las posiciones otra vez y 
      // vuelven a pelear dos. Y el nuevo tercero no pelea. 
      // - Si uno de los dos muere, en la siguiente ronda ya solo quedan 2, y pelean normal. 

      campo[i].fight(campo[(i+1)]); 
      // # descomentar solo la siguiente linea para hacer comprobaciones # 
      // console.log("["+ campo[i].username + "] VS ["+ campo[(i+1)].username + "]"); 
      if (campo[i].health<=0) { 
       console.log("El " + campo[i].constructor.name + " llamado " + campo[i].showName() + " ha sido asesinado :<"); 
       var fallecido = campo.splice(i, 1); 

       // # descomentar solo la siguiente linea para hacer comprobaciones # 
       //console.log(fallecido[0]); 
       i--; // como el array se hace pequeño, hay que corregir el error para que no se salte jugadores 
       muertos++; 
      } else { 
       if (campo[(i+1)].health<=0) { 
        console.log("El " + campo[(i+1)].constructor.name + " llamado " + campo[(i+1)].showName() + " ha sido asesinado :<"); 
        var fallecido = campo.splice((i+1), 1); 

        // # descomentar solo la siguiente linea para hacer comprobaciones # 
        // console.log(fallecido[0]); 
        i--; // como el array se hace pequeño, hay que corregir el error para que no se salte jugadores 
        muertos++; 
       } 
       else { 
        // # descomentar solo la siguiente linea para hacer comprobaciones # 
        // console.log("Siguen vivos"); 
       } 
      } 
     } 

     console.log("Fin de ronda!") 
     if (muertos === 1) { 
      console.log("Ha muerto " + muertos + " jugador."); 
     } else { 
      console.log("Han muerto " + muertos + " jugadores."); 
     } 

     // Al final de la ronda barajamos de nuevo 
     campo.shuffle(); 
    } 

    if (campo.length === 1) { 
     console.log("Vaya!! Ha sido una memorable batalla!"); 
     console.log("Después de tantos bits derramados y de " + ronda + " rondas... el jugador '" + campo[0].constructor.name + "' llamado '" + campo[0].showName() + "' se ha alzado con la victoria!!"); 
    } 

są inne w kodzie, ale wydaje się, aby zatrzymać u pierwszy. Z góry dziękuję! Zapomniałem powiedzieć, że kod działa PERFECT. Ale właśnie sprawdzałem to z JSLintem, również "tolerowanie" dla ostrzeżeń w JSLint nie działa.

Answer 1

Jeśli zdecydujesz się tolerować for, następną rzeczą jest to ostrzeżenie o to globalny deklaracja var i. Ponieważ masz pętlę for na najwyższym poziomie, i staje się dostępny wszędzie w twoim programie.

Po prostu tolerowałbym for i zawijam go w IIFE. W ten sposób i jest dostępna tylko wewnątrz tej funkcji i nie wycieka do zasięgu globalnego.

(function() { 
    var i = 0; 
    for (i=1;i<=50;i+=1) { 
     campo.push(new Human("h"+i)); 
     campo.push(new Machine("m"+i)); 
     campo.push(new Alien("e"+i)); 
    } 
})(); 

---

Można też, wyjąwszy przy użyciu istniejącego wdrożenia tam, utworzyć funkcję, która uogólnia „Repeat n razy” definicji.

function repeat(fn, n) { 
    var i = 0; 
    for (;i < n; i += 1) { 
    fn(); 
    } 
} 

wykorzystania w przypadku wyglądałby następująco:

function initialize() { 
    campo.push(new Human("h"+i)); 
    campo.push(new Machine("m"+i)); 
    campo.push(new Alien("e"+i)); 
} 

// then later 
repeat(initialize, 50); 

Answer 2

jslint jest nadgorliwy (niektórzy twierdzą), oczekuje, że wszystkie instrukcje var będą na górze funkcji.

Możesz powiedzieć jslint, że nie dbasz o tę regułę, dodając komentarz do instrukcji w wierszu powyżej, w którym deklarujesz zmienną.

// Llenamos el campo de 50 humanos/maquinas/extraterrestres = 150 jugadores 
/*jslint for:true */ 
for (var i=1;i<=50;i+=1){ 

Albo można przenieść wszystkie swoje var i; do górnej części pliku/funkcji