为什么 OR 在 IF 语句中没有按预期工作？

Question

我有一个将文件上传到 S3 的功能，并且只想允许上传 PNG 和（JPG、JPEG）图像。 我添加了以下检查；

 if(fileExtension.toUpperCase() != 'png'.toUpperCase() || fileExtension.toUpperCase() != 
 'jpeg'.toUpperCase() || fileExtension.toUpperCase() != 'jpg'.toUpperCase()) {
 return {
        statusCode: 500,
        body: JSON.stringify("Not a valid capture format.  Capture upload only accepts PNG and 
  (JPG,JPEG)"),
     };
 }
 else {
  // Upload capture
}

发生的情况是它返回下面的错误消息，说它不是有效的捕获格式，即使我希望它是。 例如， filePrefix是jpg因此我希望它进入else语句。

编辑

如何只允许将带有 PNG、JPG 或 JPEG 扩展名的文件上传到 S3？ 如果不是这些扩展名，我想返回一个错误，如上所示。

谢谢大家。

Answer 1

看来你的逻辑有点不对啊！

就目前而言，您正在检查文件扩展名是否不是“png”或“jpg”或不是“jpeg”。 因为它永远不可能全部为 3，并且测试必须每次都返回 true。

你应该做两件事之一。

1. 改变|| s 到&& s 以便：如果它不是“png”，也不是“jpg”，也不是“jpeg”，则返回错误

    if(fileExtension.toUpperCase() != 'png'.toUpperCase() && fileExtension.toUpperCase() != 
     'jpeg'.toUpperCase() && fileExtension.toUpperCase() != 'jpg'.toUpperCase()) {
        // return error
    } else {
        // process upload
    }

或者，

2. 反转逻辑并将!=更改为==

    if(fileExtension.toUpperCase() == 'png'.toUpperCase() || fileExtension.toUpperCase() == 'jpeg'.toUpperCase() || fileExtension.toUpperCase() == 'jpg'.toUpperCase()) {
        // process upload
    } else {
        // return error
    }