Почему у меня возникают недетерминированные ошибки удаления файлов при использовании DiskFileItem? - PullRequest
1 голос
/ 29 ноября 2011

Мой сервлет загрузки продолжает выдавать исключение, в котором говорится, что файл, который я пытаюсь заменить (ближе к концу моего кода), не может быть удален случайно (на первый взгляд).Я не знаю, что является причиной этого, так как я не использую никаких потоков, и файл не открыт в моем браузере.Кто-нибудь знает, что может быть причиной этого?Я совершенно не в курсе этого, так как код кажется мне правильным.Это первый раз, когда я использовал DiskFileItem, поэтому я не уверен, есть ли какие-то нюансы для обработки.

Имейте в виду, что иногда это работает, иногда нет.Я заблудился.

Проблемная область:

File destination = new File(wellnessDir + File.separator + fileName + ".pdf");

  System.out.println("destination file exists: " + destination.exists());
  System.out.println("file to be moved exists: " + uploadedFile.exists());

  if(destination.exists()){
    boolean deleted = destination.delete();
    if(!deleted)
      throw new Exception("Could not delete file at " + destination);
  }        

Мои системные выходы всегда говорят, что существует и файл, и место назначения.Я пытаюсь загрузить файл для перезаписи существующего файла.

Полный код: (& pastebin )

private void uploadRequestHandler(ServletFileUpload upload, HttpServletRequest request)
  {
    // Handle the request
    String fileName = "blank";
    try{         
      List items = upload.parseRequest(request);
      //Process the uploaded items
      Iterator iter = items.iterator();
      File uploadedFile = new File(getHome() + File.separator + "temp");
      if(uploadedFile.exists()){
        boolean tempDeleted = uploadedFile.delete();
        if(!tempDeleted)
          throw new Exception("Existing temp file could not be deleted.");
      }
      //write the file
      while (iter.hasNext()) {
        DiskFileItem item = (DiskFileItem) iter.next();
        if(item.isFormField()){
          String fieldName = item.getFieldName();
          String fieldValue = item.getString();
          if(fieldName.equals("fileName"))
            fileName = fieldValue;
            //other form values would need to be handled here, right now only need for fileName
        }else{
          item.write(uploadedFile);
        }
      }
      if(fileName.equals("blank"))
        throw new Exception("File name could not be parsed.");
      //move file
      File wellnessDir = new File(getHome() + File.separator + "medcottage" + File.separator + "wellness");
      File destination = new File(wellnessDir + File.separator + fileName + ".pdf");

      System.out.println("destination file exists: " + destination.exists());
      System.out.println("file to be moved exists: " + uploadedFile.exists());

      if(destination.exists()){
        boolean deleted = destination.delete();
        if(!deleted)
          throw new Exception("Could not delete file at " + destination);
      }        
      FileUtil.move(uploadedFile, new File(wellnessDir + File.separator + fileName + ".pdf"));
      writeResponse();
    } catch (Exception e) {
      System.out.println("Error handling upload request.");
      e.printStackTrace();
    }
  }

изменить: добавить,getHome () и "home" на самом деле не в коде, это просто для защиты моего домашнего пути

1 Ответ

0 голосов
/ 09 декабря 2011

После долгих испытаний и обострений, наконец, попробовал его на другой машине, тот же код, работал отлично.Имеет отношение к переносу доменов на мою рабочую машину, и это мешает с разрешениями.

...