得到错误写入

Question

我正在读ZIP文件的内容，当我发现sample.xml中的文件，我编辑它的内容，并写入到输出zip文件

public class CopyEditZip { 

static String fileSeparator = System.getProperty("file.separator"); 

    public static void main(String[] args) { 
     System.getProperty("file.separator"); 

     ZipFile zipFile; 
     try { 
      zipFile = new ZipFile("c:/temp/source.zip); 
      ZipOutputStream zos = new ZipOutputStream(new 
            FileOutputStream(
              c:/temp/target.zip)); 

      for (Enumeration e = zipFile.entries(); 
         e.hasMoreElements();) 
       { 
        ZipEntry entryIn = (ZipEntry) e.nextElement(); 
        if (entryIn.getName().contains("sample.xml")) { 
         zos.putNextEntry(new ZipEntry("sample.xml")); 
         InputStream is = zipFile.getInputStream(entryIn); 
         byte[] buf = new byte[1024]; 
         int len; 
         while ((len = (is.read(buf))) > 0) { 
          String x = new String(buf); 
          if (x.contains("Input")) { 
           System.out.println("edit count"); 
           x = x.replace("Input", "output"); 
          } 
          buf = x.getBytes(); 
          zos.write(buf, 0, (len < buf.length) ? len 
           : buf.length); 
         } 
         is.close(); 
         zos.closeEntry(); 
       } 
       zos.close(); 
       zipFile.close(); 
      } catch (Exception ex) { 

     } 

     } 
    } 

现在输出中的sample.xml不正确。有一些数据被截断，一些丢失。这是否与缓冲区没有正确写入有关？任何其他替代方法来编辑文件并写出来？

编辑：我看到xml正在写入从xml更多的数据。 mt结束标记被称为代理，然后是几行数据。不知道它是如何在结束标记后写入更多数据的。

编辑：

我把一个计数器和线路的系统输出由网上看到while循环的每次迭代中一下就出来了。

这里的最后两行

18 
put.fileFtpDirectory"/><ConfigurableProperty uri="CDTSFileInput#File  
Input.fileFtpServer"/><ConfigurableProperty uri="CDTSFileInput#File  
Input.fileFtpUser"/><ConfigurableProperty uri="CDTSFileInput#File 
Input.longRetryInterval"/><ConfigurableProperty uri="CDTSFileInput#File 
Input.messageCodedCharSetIdProperty"/><ConfigurableProperty 
uri="CDTSFileInput#File Input.messageEncodingProperty"/> 
<ConfigurableProperty uri="CDTSFileInput#File Input.remoteTransferType"/> 
<ConfigurableProperty uri="CDTSFileInput#File Input.retryThreshold"/> 
<ConfigurableProperty uri="CDTSFileInput#File Input.shortRetryInterval"/> 
<ConfigurableProperty uri="CDTSFileInput#File Input.validateMaster"/> 
<ConfigurableProperty override="30" uri="CDTSFileInput#File 
Input.waitInterval"/><ConfigurableProperty override="no" 
uri="CDTSFileInput#FileInput.connectDatasourceBeforeFlowStarts"/> 
<ConfigurableProperty uri="CDTSFileInput#FileInput.validateMaster"/> 
<ConfigurableProperty override="/apps/cdts/trace/ExceptionTrace- 
CDTSFileInput-CDT.REF_EXT.Q01.txt" uri="CDTSFileInp 

19 
ut#FilePath_ExceptionTrace"/><ConfigurableProperty 
override="/apps/cdts/trace/SnapTrace-CDTSFileInput-CDT.REF_EXT.Q01.txt"  
uri="CDTSFileInput#FilePath_SnapTraceENV"/><ConfigurableProperty  
override="/apps/cdts/trace/SnapTrace-CDTSFileInput-CDT.REF_EXT.Q01.txt" 
uri="CDTSFileInput#FilePath_SnapTraceNOENV"/><ConfigurableProperty 
override="EXTERNAL" uri="CDTSFileInput#INPUTORIGIN"/> 
<ConfigurableProperty 
override="/apps/cdts/data_in/data_in_fileinput_gtr1" 
uri="CDTSFileInput#InputDirectory"/><ConfigurableProperty override="GTR" 
uri="CDTSFileInput#SUBMITTERID"/><ConfigurableProperty 
override="FILEINPT" uri="CDTSFileInput#SUBMITTERTYPE"/> 
<ConfigurableProperty override="" uri="CDTSFileInput#excludePattern"/> 
<ConfigurableProperty override="*" uri="CDTSFileInput#filenamePattern"/> 
<ConfigurableProperty override="no" 
uri="CDTSFileInput#recursiveDirectories"/></CompiledMessageFlow> 
</Broker>ileInput#FileInput.validateMaster"/><ConfigurableProperty 
override="/apps/cdts/trace/ExceptionTrace-CDTSFileInput- 
CDT.REF_EXT.Q01.txt" uri="CDTSFileInp 

的XML结束，但在最后的一部分，而是一个线再被追加。

Answer 1

读取和写入文本

如果该文件是文本文件，你不应该读它以字节为单位。你应该用一个阅读器来包装输入流，读取线条，并将它们写回到包装在输出流中的作者。

其中一个原因是该文件可能处于不是单字节的编码，如UTF-8。这意味着一个字符可以在一个缓冲区和下一个缓冲区之间分开。

另一个问题是Input这个词可能会在缓冲区之间分裂。所以你可能只需要在Inp之一，ut在下一个，你不会正确匹配它。读线是确保你不会停下来的好方法。

但是，使用ZipOutputStream编写文本会稍微简单一些，因为您没有为每个条目获取单独的输出流。因此，您需要从读取的行中提取字节，然后将这些字节写入zip文件 - 与您非常相似。

读取和写入的字节

即使文件恰好是ASCII，你有一对夫妇的问题，在你的读/写循环。首先，未成年人一个是你的循环条件应该是：

((len = (is.read(buf)) >= 0) 

你真当你-1只能终止循环。理论上，如果缓冲区大小为零，则可以在循环中间读取根本不读取任何字节的数据，但这并不意味着数据流已结束。所以>=，而不是>。

但是更糟糕的问题是，您读取的是len字节，但是您将整个缓冲区转换为字符串。所以如果你有一个1024字节的缓冲区，并且len只有50个，那么只有50个字节的缓冲区是最新读取的内容，其余的将来自上一次读取，或者是零。

因此，如果这是你读的东西，总是使用len字节。您应该使用

String x = new String(buf,0,len); 

不是

String x = new String(buf); 

此外，应注意的是，当你这样做：

buf = x.getBytes(); 

你的缓冲区不再1024个字节长。如果最初有1024个字节，并且在字符串中出现10个Input，则缓冲区现在将长1034个字节（假定为一个字节的编码）。 len不再相关 - 它会比数字更小。所以这是你失去角色的另一个原因。

编码

通常情况下，XML文件是UTF-8。当您将字节转换为字符串时，以及在创建读者和作者时，显式声明编码非常重要。否则，字符可能会被不适当地读取。

摘要

• 更喜欢一个文本文件，基于行的读取循环。
• 如果您读取的是字节而非行：如果您读取的是len字节，请使用len字节，而不是整个缓冲区。
• 如果更改数据，请勿使用旧的len。
• 使用编码。

所以新的循环的素描是：

for (Enumeration<? extends ZipEntry> e = zipFile.entries(); e.hasMoreElements();) { 
    ZipEntry entryIn = e.nextElement(); 
    if (entryIn.getName().contains("sample.xml")) { 
     zos.putNextEntry(new ZipEntry("sample.xml")); 
     try (BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(zipFile.getInputStream(entryIn), 
                         StandardCharsets.UTF_8))) { 
      String line; 
      while ((line = bufferedReader.readLine()) != null) { 
       if (line.contains("Input")) { 
        System.out.println("edit count"); 
        line = line.replace("Input", "output") 

       } 
       line += System.lineSeparator(); // Add newline back. 
       byte[] buf = line.getBytes(StandardCharsets.UTF_8); 
       zos.write(buf); 
      } 
     } 
    zos.closeEntry(); 
    } 
} 

注：

• 尝试 - 与资源用于打开缓冲的读者。它将自动关闭（带有其底层读取器和输入蒸汽）。
• 请勿使用原始类型Enumeration。使用适当的通配符，你将能够避免显式的强制转换。
• 由于您从整行创建了一个缓冲区，并且只有那一行，所以您可以编写完整的缓冲区并且不需要偏移量和长度。