Question

我的问题如下，我正在尝试通过servlet将文件上传到我的服务器，但上传的文件不一样，与原版文件相比有差异。

例如，如果我上传TXT文件，例如

# Root logger option
log4j.rootLogger=DEBUG, stdout, file

# Redirect log messages to console
log4j.appender.stdout=org.apache.log4j.ConsoleAppender
log4j.appender.stdout.Target=System.out
log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
log4j.appender.stdout.layout.ConversionPattern=%d{yyyy-MM-dd HH:mm:ss} %-5p     %c{1}:%L - %m%n

# Rirect log messages to a log file
log4j.appender.file=org.apache.log4j.RollingFileAppender
log4j.appender.file.File=C:\\filesLog\\log.log
log4j.appender.file.MaxFileSize=5MB
log4j.appender.file.MaxBackupIndex=10
log4j.appender.file.layout=org.apache.log4j.PatternLayout
log4j.appender.file.layout.ConversionPattern=%d{yyyy-MM-dd HH:mm:ss} %-5p         %c{1}:%L - %m%n

上传文件有一个新行

... NULNULNULNULNULNUL

（用Notepad ++打开）。

如果文件是.exe，则上传的文件不起作用。

我的代码如下

protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
    try {

        boolean isMultipart = ServletFileUpload.isMultipartContent(request);
        System.out.println("ES UN REQUEST MULTIPART " + isMultipart);
        // Set factory constraints
        // Create a new file upload handler
        ServletFileUpload upload = new ServletFileUpload();
        // Set overall request size constraint
        // MAX REQUEST SIZE
        // Parse the request
        // List / FileItem / items = upload.parseRequest(request);
        FileItemIterator iter = upload.getItemIterator(request);
        while (iter.hasNext()) {
            // FileItem item = (FileItem) iter.next();
            FileItemStream itemS = (FileItemStream) iter.next();

            if (itemS.isFormField() == false) {
                // Process a file upload
                String fileName = itemS.getName();
                System.out.println("FILE NAME " + fileName);
                // Process a file upload
                File uploadedFile = new File("C:\\FilesUpload\\" + UUID.randomUUID());
                // InputStream uploadedStream = item.getInputStream();

                FileOutputStream fout = new FileOutputStream(uploadedFile);
                BufferedOutputStream bout = new BufferedOutputStream(fout);
                BufferedInputStream bin = new BufferedInputStream(itemS.openStream());
                byte buf[] = new byte[2048];
                Long contador = 0L;
                while ((bin.read(buf)) != -1) {
                    bout.write(buf);
                    contador = contador + 1L;
                }
                bout.close();
                bin.close();
            }
        }
    } catch (Exception e) {
        e.printStackTrace();
    }
}

Answer 1

在这部分代码中：

 while ((bin.read(buf)) != -1) {
        bout.write(buf);
        contador = contador + 1L;
    }

您没有检查已读取的字节数。最后一个缓冲区读取，如果文件的长度不是完全可被2048整除，则只能部分读取。在这种情况下，bin.read(buf)将返回读取的字节数，您只应在编写buf时使用该数字，否则您将在buf中写入所有2048个字节，包括如果这是第一次读取，则来自先前读取或零的值。

所以它应该是这样的：

 int numBytesRead;
 while ((numBytesRead = bin.read(buf)) != -1) {
        bout.write(buf,0,numBytesRead);
        contador = contador + 1L;
 }

这将使它准确地写出它所读取的内容而不再是它。

顺便说一句，如果您使用的是Java 7及更高版本，那么您可以使用Files.copy将整个流直接复制到您的文件中，而不是完成所有这些操作。

Files.copy(itemS.openStream(),
           Paths.get("C:\\FilesUpload\\" + UUID.randomUUID()),
           StandardCopyOption.REPLACE_EXISTING );

Answer 2

检查

bout.write(buf)

即使部分内容为空，我也会编写整个缓冲区。

请注意，buf为2048字节..

Answer 3

您忽略了bin.read(buf)返回的值，表示读取的字节数。无法保证读取操作填满了缓冲区。因此，您对bout.write(buf)的许多调用都在编写垃圾数据。

将InputStream保存到文件是一种更简单的方法（因此不易出错）：

Files.copy(bin, uploadedFile.toPath());

请参阅documentation。

Java上传的文件不一样

3 个答案: