2013-03-01 7 views
6

Ho cercato sul Web e SO per capire questo da solo con molti modi diversi, ma senza successo.Convalida le estensioni di file da un array

Vorrei mostrare un messaggio se l'estensione del file è accettata o meno, nello stesso modo delle istruzioni "outpush.push".

Questo dovrebbe essere preso da un ARRAY delle estensioni di file accettate come JPG, PNG, GIF e rilevare se l'estensione del file è in maiuscolo e accettarlo (convertirlo in lettere minuscole).

Ecco il mio script. Mi chiedo come e dove nella sceneggiatura potrei implementare una funzione del genere?

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); 
+0

Dov'è il jQuery? Tutto questo sembra JavaScript di base. –

+0

Hai ragione. Modificherò la mia domanda. Pensavo che era. –

+0

Ci sono già delle risposte qui sotto, ma aggiungerò che hai 'if (f.size> max_size)' e 'if (f.size

risposta

7

Ci sono un paio di problemi con il codice.

1) è necessario utilizzare un if-else, invece di più if dichiarazioni:

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

2) all non è un valore valido per display, utilizzare block o inline:

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

3) Stai utilizzando tag obsoleti <font>. Invece, usa stili e CSS. Nel tuo caso, utilizzerei le classi, una per un msg, più classi aggiuntive per error, important e ok.

4) Per fare il check matrice, basta usare 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/

+0

Grazie Jeff, lo apprezzo molto. Saluti –

2

Si può avere l'array come qualcosa di simile a questo:

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

e poi fare qualcosa di simile nel tuo ciclo for

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>'); 
} 

Maggiori lettura su inArray().

+0

Le altre risposte sono buone per vanilla JavaScript. Terrò questa risposta per riferimento – RenegadeMatrix

0

È possibile provare uno dei due approcci: provare ad afferrare l'estensione e confrontarla con una serie di estensioni consentite o con un'espressione regolare.

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

Per quanto riguarda il jQuery - si potrebbe considerare quanto segue per costruire il vostro HTML

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()); 
Problemi correlati