Sprawdzanie poprawności rozszerzeń plików z tablicy

Question

Przeszukałem sieć WWW i SO, aby samodzielnie rozwiązać ten problem na wiele różnych sposobów, ale bez powodzenia.

Chciałbym pokazać komunikat, jeśli rozszerzenie pliku jest akceptowany, czy nie, w taki sam sposób, jak instrukcje "outpush.push".

Należy pobrać z ARRAY z zaakceptowanych rozszerzeń plików, takich jak JPG, PNG, GIF i wykryć, czy rozszerzenie pliku jest wielkie i zaakceptować (przekonwertować na małe litery).

Oto mój skrypt. Zastanawiam się, jak i gdzie w scenariuszu mogę zaimplementować taką funkcję?

function handleFileSelect(evt) { 

    var files = evt.target.files; // FileList object 
    var max_size = 5120; // Max file size 

    var output = []; 
    for (var i = 0, f; f = files[i]; i++) { 

     output.push('<li><strong><font size="3" color="FFFFFF">FILE: ', escape(f.name), '</strong> (', f.type || 'n/a', ') - ', 
      f.size, ' bytes, last modified: ', 
      f.lastModifiedDate ? f.lastModifiedDate.toLocaleDateString() : 'n/a', 
      '</font></li>'); 

     if(f.size > max_size) { 

      output.push('<font size="5" color="FFFF00"><b>ERROR!! Sorry, but the file that you selected is too large. Please upload a file that is no larger than ' + max_size + ' KB.'); 
     } 

     if(f.size < max_size) { 
      output.push('<font size="5" color="FFFF00"><b>FILE SIZE OK. CLICK TO SEND button below.</font>'); 

      output.push('<font size="5" color="FFFFFF"><hr><b>IMPORTANT: Do not close this window. Wait till you see the next page when finished uploading your file.</font>'); 

      document.getElementById("myButton").style.display="all"; 
     } 

    } 

    document.getElementById('list').innerHTML = '<ul>' + output.join('') + '</ul>'; 
} 

document.getElementById('files').addEventListener('change', handleFileSelect, false); 

Answer 1

Istnieje kilka problemów z kodem.

1) należy użyć if-else, zamiast wielu if stwierdzeń:

if (f.size > max_size) { 
    // ... 
} else { 
    // ... 
} 

2) all nie jest prawidłową wartością dla display użyć block lub inline:

document.getElementById("myButton").style.display = "block"; 

3) Używasz nieaktualnych tagów <font>. Zamiast tego użyj stylów i CSS. W twoim przypadku użyłbym klas, jeden dla msg, plus dodatkowe klasy dla error, important i ok.

4) W tym celu należy sprawdzić tablicy, wystarczy użyć indexOf():

var extensions = ["jpg", "jpeg", "txt", "png"]; // Globally defined 

... 

// Get extension and make it lowercase 
// This uses a regex replace to remove everything up to 
// and including the last dot 
var extension = f.name.replace(/.*\./, '').toLowerCase(); 

if (extensions.indexOf(extension) < 0) { // Wasn't found 
    output.push('<li class="msg error">ERROR!! Sorry, but the file that you selected is not a valid file type. Valid types are: ', valid, '</li>'); 
} else ... 

Demo: http://jsfiddle.net/jtbowden/L2Gps/

Answer 2

Możesz mieć tablicę jako coś takiego:

var file_types = {'jpg', 'png', 'gif'}; 

a następnie zrobić coś takiego w swoim dla pętli

if ($.inArray(f.ext, file_types) == -1) { 
output.push('<font size="5" color="FFFF00"><b>ERROR!! Incorrect file type! Accepted file types are : jpg, png, gif</b></font>'); 
} 

Więcej czytania o inArray().

Answer 3

Możesz wypróbować jedno z dwóch podejść - spróbuj zgarnąć rozszerzenie i dopasuj je do tablicy dozwolonych rozszerzeń lub wyrażenia regularnego.

var accepted_types = ['jpg', 'gif', 'png']; 
if (accepted_types.indexOf(f.ext) > 0) { 
    output.push(...); 
} 

chodzi o jQuery - możesz rozważyć następujący kod HTML na budowę

var file_list = $("<ul></ul>"); 
for (var i = 0, f; f = files[i]; i++) { 
    // work goes here, simply append the list elements into file_list 
    file_list.append($("<li>" ... "</li>")); 
} 

// to append your new list to id="list" 
$("#list").append(file_list); 
// or to replace the html with your new html 
$("#list").html(file_list.html());