, если утверждение, кажется, никогда не оценивается false

Question

Цель: Пользователь загружает изображение, проверяющий проверяет, является ли он файлом изображения, который пользователь загрузил, возвращает сообщение, если это не изображение, если нет является.

Ошибка: При нажатии кнопки «Загрузить» независимо от того, является ли загруженный файл изображением или нет изображения, сообщение валидатора всегда возвращается.

Площадь фокуса: В классе Validator строка System.out.println(partValueContentType); написала тип контента на консоль, например. image/jpeg, но когда он протестирован в заявлении if, он вообще не оценивает тип содержимого.

 String partValueContentType = part.getContentType(); 
     System.out.println(partValueContentType); 

     if (!partValueContentType.equals("image/jpeg") 
       || !partValueContentType.equals("image/jpg") 
       || !partValueContentType.equals("image/gif") 
       || !partValueContentType.equals("image/png")) 
     { 
      FacesMessage msg = new FacesMessage("File is not an image.", 
        "Acceptable image types (jpeg, jpg, gif, png)"); 
      msg.setSeverity(FacesMessage.SEVERITY_ERROR); 
      throw new ValidatorException(msg); 
     } 

Как это вызвано и как я могу его решить?

Answer 1

Ваш если заявление поодаль:

String partValueContentType = part.getContentType(); 
System.out.println(partValueContentType); 

if (!(partValueContentType.equals("image/jpeg") 
     || partValueContentType.equals("image/jpg") 
     || partValueContentType.equals("image/gif") 
     || partValueContentType.equals("image/png"))) 
{ 
    FacesMessage msg = new FacesMessage("File is not an image.", 
      "Acceptable image types (jpeg, jpg, gif, png)"); 
    msg.setSeverity(FacesMessage.SEVERITY_ERROR); 
    throw new ValidatorException(msg); 
} 

С точки зрения проверки, вы можете проверить сам файл, чтобы убедиться, что это действительно картина (это не .zip скрытие как .jpeg) и, возможно, соблюдение ограничений на размер файлов ...

---

Или использовать HashSet:

String partValueContentType = part.getContentType(); 
System.out.println(partValueContentType); 
Set<String> acceptedMimeTypes = new HashSet<>(Arrays.asList("image/jpeg", "image/jpg", "image/gif", "image/png")); 

if (!acceptedMimeTypes.contains(partValueContentType)) 
{ 
    FacesMessage msg = new FacesMessage("File is not an image.", 
      "Acceptable image types " + Arrays.toString(acceptedMimeTypes.toArray())); 
    msg.setSeverity(FacesMessage.SEVERITY_ERROR); 
    throw new ValidatorException(msg); 
}