2016-06-24 12 views
3

Ich benutze eine Eingabe mehrere Dateien und dann eine for-Schleife, um die Dateien mit FileReader zu durchlaufen. JFehler hier: https://jsfiddle.net/zLq8rsos/. Es zeigt den Dateinamen und den Inhalt korrekt an, aber ich verstehe nicht, warum das Zählen nicht funktioniert. Wenn ich eine Datei wähle, ist sie mit 1 nummeriert (warum nicht 0?). Wenn ich zwei Dateien auswähle, werden beide als "2" gezählt. Was mache ich falsch?Anzahl Eingabedateien in for Schleife

function showDetails(file, content, n) { 
 
    var start = content.substring(0, 9); 
 
    var message = "File " + n + " is " + file.name + " and starts with " + start + " .<br>"; 
 
    $('#results').append(message); 
 
} 
 

 
$(document).ready(function() { 
 
    $('#files').on('change', function(evt) { 
 
    var files = evt.target.files; 
 
    if (files) { 
 
    \t $('#results').text(""); 
 
     for (var i = 0, f; f = files[i]; i++) { 
 
     var reader = new FileReader(); 
 
     reader.onload = (function(theFile) { 
 
      return function(e) { 
 
      var content = e.target.result; 
 
      showDetails(theFile, content, i); 
 
      }; 
 
     })(f); 
 
     reader.readAsText(f, "UTF-8"); 
 
     } 
 
    } else { 
 
     console.log("Failed to load file(s)"); 
 
    }; 
 
    }); 
 
});
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> 
 
<input type="file" id="files" name="files[]" multiple> 
 
<div id="results"> 
 
</div>

Antwort

2

Verwenden Verschlüsse ändern IIFE

function showDetails(file, content, n) { 
    var start = content.substring(0, 9); 
    var message = "File " + n + " is " + file.name + " and starts with " + start + " .<br>"; 
    $('#results').append(message); 
} 

$(document).ready(function() { 
    $('#files').on('change', function(evt) { 
    var files = evt.target.files; 
    if (files) { 
     $('#results').text(""); 
     for (var i = 0, f; f = files[i]; i++) { 
     var reader = new FileReader(); 
     reader.onload = (function(theFile,count) { 
      return function(e) { 
      var content = e.target.result; 
      showDetails(theFile, content, count); 
      }; 
     })(f,i+1); 
     reader.readAsText(f, "UTF-8"); 
     } 
    } else { 
     console.log("Failed to load file(s)"); 
    }; 
    }); 
}); 
+0

Danke und @ Nikhil. Offensichtlich sind Iteration und Ausführen des Codes in der Schleife zwei verschiedene Dinge. Diese Lösung scheint mehr KISS zu sein als Nikhil, die einen zusätzlichen Funktions-Wrapper hinzufügt, also stimme ich dem zu. Verschlüsse sind neu für mich, interessante Sachen. Fügen Sie diesen Link hinzu: http://www.w3schools.com/js/js_function_closures.asp. – Mko

2

Problemstellung - In for-Schleife, wenn Sie versuchen, den Wert von i zugreifen zu können, hat sie bereits abgeschlossen, die Iterationen und damit den Wert, weil 1 für 1 Datei und 2 für 2 Dateien.

Sie können Ihre for-Schleife

for (var i = 0, f; f = files[i]; i++) { 
     (function(i){ // Added 
      var reader = new FileReader(); 
      reader.onload = (function(theFile) { 
       return function(e) { 
        var content = e.target.result; 
        showDetails(theFile, content, i); 
       }; 
      })(f); 
      reader.readAsText(f, "UTF-8"); 
     })(i+1); 
} 

Als Referenz aktualisieren - https://jsfiddle.net/zLq8rsos/1/

0

JSFIDDLE

Alles, was Sie brauchen, ist zu Aktualisieren der Zählung durch die Schleife jedes Mal

function showDetails(file, content, n) { 
 
    var start = content.substring(0, 9); 
 
    var message = "File " + n + " is " + file.name + " and starts with " + start + " .<br>"; 
 
    $('#results').append(message); 
 
} 
 

 
$(document).ready(function() { 
 
    $('#files').on('change', function(evt) { 
 
    var files = evt.target.files; 
 
    if (files) { 
 
    \t $('#results').text(""); 
 
     for (var i = 0, f; f = files[i]; i++) { 
 
     var reader = new FileReader(); 
 
     var x=1; 
 
     reader.onload = (function(theFile) { 
 
      return function(e) { 
 
      var content = e.target.result; 
 
      showDetails(theFile, content, x); 
 
      x++; 
 
      }; 
 
     })(f); 
 
     reader.readAsText(f, "UTF-8"); 
 
     } 
 
    } else { 
 
     console.log("Failed to load file(s)"); 
 
    }; 
 
    }); 
 
});
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> 
 
<input type="file" id="files" name="files[]" multiple> 
 
<div id="results"> 
 
</div>

0

Verwendung jquery jeder, starten Sie den Index der Datei mit 0

function showDetails(file, content, n) { 
 
    var start = content.substring(0, 9); 
 
    var message = "File " + n + " is " + file.name + " and starts with " + start + " .<br>"; 
 
    $('#results').append(message); 
 
} 
 

 
$(document).ready(function() { 
 
    $('#files').on('change', function(evt) { 
 
    var files = evt.target.files; 
 
    if (files) { 
 
    \t $('#results').text(""); 
 
     $.each(files, function(i, f){ 
 
     var reader = new FileReader(); 
 
     reader.onload = (function(theFile) { 
 
      return function(e) { 
 
      var content = e.target.result; 
 
      showDetails(theFile, content, i); 
 
      }; 
 
     })(f); 
 
     reader.readAsText(f, "UTF-8"); 
 
     }); 
 
    } else { 
 
     console.log("Failed to load file(s)"); 
 
    }; 
 
    }); 
 
});
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> 
 
<input type="file" id="files" name="files[]" multiple> 
 
<div id="results"> 
 
</div>

+0

Danke für die jQuery Appreach, funktioniert genauso gut. – Mko