Jquery条件不起作用

Question

问题是当我上传一个无效的文件扩展名和无效的文件大小。它不会提示无效的文件类型和文件大小，而是会根据满足2条件的第一个文件发出警报。

例如，当我上传example.php和另一个大于2048的文件时，它将提醒无效的文件类型。然后反之亦然。

我要的是，如果我上传example.php和具有大于2048的输出必须是无效的文件类型和文件大小的另一个文件。

这里是我的代码：

var fileextension = new RegExp("(.*?)\.(csv|doc|docx|gif|jpeg|jpg|pdf|png|ppt|pptx|txt|xls|xlxs|zip|mp3|mp4)$"); 
$('input[type=file]').change(function() { 
var val = $(this).val().toLowerCase(); 

for(var i=0; i<this.files.length; i++){ 
    var filename = this.files[i].name.toLowerCase(); 
    var fsize = this.files[i].size, 
     fsize = fsize/1024; // 1024 = 1mb 
     if(fsize > 2048){ 
      $(this).val(''); 
      swal('Opps','Invalid file size','warning'); 
     }else if(!fileextension.test(filename)){ 
      $(this).val(''); 
      swal('Opps','Invalid file type','warning'); 
     }else if(!fileextension.test(filename) && fsize > 2048){ 
      $(this).val(''); 
      swal('Opps','Invalid file type and file size','warning'); 
     } 
} 

Answer 1

创建一个临时字符串，例如message，并改变你的if块这样的：

var message = ""; 

if(fsize > 2048){ 
    message += "Invalid file size"; 
} 

if(!fileextension.test(filename)) { 
    message += message === "" ? "Invalid" : " and"; 
    message += " file type"; 
} 

然后就可以调用swal这样的：

if (message) { // Checks if there is something in `message` 
    $(this).val(''); 
    swal('Opps',message,'warning'); 
} 

编辑： 这里是更新的答案（这是在小提琴提到） ：

var fileextension = new RegExp("(.*?)\.(csv|doc|docx|gif|jpeg|jpg|pdf|png|ppt|pptx|txt|xls|xlxs|zip|mp3|mp4)$"); 

$('input[type=file]').change(function() { 
    var val = $(this).val().toLowerCase(), 
    invalidType = false, 
    invalidSize = false; 

    for (var i = 0; i < this.files.length; i++) { 
    var filename = this.files[i].name.toLowerCase(); 
    var fsize = this.files[i].size, 
     fsize = fsize/1024; // 1024 = 1mb 

    if (fsize > 2048) { 
     invalidSize = true; 
    } 

    if (!fileextension.test(filename)) { 
     invalidType = true; 
    } 
    } 

    var message = ""; 

    if (invalidType) { 
    message += "Invalid type"; 
    } 

    if (invalidSize) { 
    message += (message === "" ? "" : " and") + " size"; 
    } 

    if (message) { 
    swal('Opps', message, 'warning'); 
    } 

}); 

Answer 2

什么情况是，第一if条件计算结果为true被执行，程序不评估休息。在你的情况下，这会使if else循环中的最后一个条件无法访问。如果你希望你的最后一个条件先计算，然后移动它 - 是这样的：

var fileextension = new RegExp("(.*?)\.(csv|doc|docx|gif|jpeg|jpg|pdf|png|ppt|pptx|txt|xls|xlxs|zip|mp3|mp4)$"); 
$('input[type=file]').change(function() { 
var val = $(this).val().toLowerCase(); 

for(var i=0; i<this.files.length; i++){ 
    var filename = this.files[i].name.toLowerCase(); 
    var fsize = this.files[i].size, 
     fsize = fsize/1024; // 1024 = 1mb 
     if(!fileextension.test(filename) && fsize > 2048){ 
      $(this).val(''); 
      swal('Opps','Invalid file type and file size','warning'); 
     } 
     else if(fsize > 2048){ 
      $(this).val(''); 
      swal('Opps','Invalid file size','warning'); 
     } 
     else if(!fileextension.test(filename)){ 
      $(this).val(''); 
      swal('Opps','Invalid file type','warning'); 
     } 
} 

编辑 - 如果你希望你的输出是基于所有文件作为一组的状态，而不是单个文件，那么显然你不能在循环内处理你的输出。你需要做这样的事情：

var fileextension = new RegExp("(.*?)\.(csv|doc|docx|gif|jpeg|jpg|pdf|png|ppt|pptx|txt|xls|xlxs|zip|mp3|mp4)$"); 
$('input[type=file]').change(function() { 
var val = $(this).val().toLowerCase(); 

var sizeError=false; 
var extensionError=false; 

for(var i=0; i<this.files.length; i++){ 
    var filename = this.files[i].name.toLowerCase(); 
    var fsize = this.files[i].size, 
     fsize = fsize/1024; // 1024 = 1mb 
     if(fsize > 2048){ 
      sizeError=true; 
     } 
     if(!fileextension.test(filename)){ 
      extensionError=true; 
     } 
} 

现在使用这些布尔值extensionError和sizeError的for循环外检查和处理的输出。