4

我有一组要合并的文件。这是我尝试过的,但没有奏效。

 public static void joinf(File f1, File f2){

    try{

        InputStream in = new FileInputStream(f1);


        OutputStream out = new FileOutputStream(f2,true);

        byte[] buf = new byte[8192];
        int len;
        while ((len = in.read(buf)) > 0){
            out.write(buf, 0, len);
        }
        in.close();
        out.close();
        System.out.println("File copied.");
    }
    catch(FileNotFoundException ex){
        System.out.println(ex.getMessage() + " in the specified directory.");
        System.exit(0);
    }
    catch(IOException e){
        System.out.println(e.getMessage());            
    }
}


public void pro(File a,File[]b){
    for(int i=0;i<b.length;i++){


        joinf(a,b[i]);
    }
}
4

2 回答 2

19

使用IOUtils来执行此操作。看我的例子:

import java.io.BufferedInputStream;
import java.io.BufferedOutputStream;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;

import org.apache.commons.io.IOUtils;

public class SourceCodeProgram {

    public static void main(String[] args) throws Exception {
        IOCopier.joinFiles(new File("D:/d.txt"), new File[] {
                new File("D:/s1.txt"), new File("D:/s2.txt") });
    }
}

class IOCopier {
    public static void joinFiles(File destination, File[] sources)
            throws IOException {
        OutputStream output = null;
        try {
            output = createAppendableStream(destination);
            for (File source : sources) {
                appendFile(output, source);
            }
        } finally {
            IOUtils.closeQuietly(output);
        }
    }

    private static BufferedOutputStream createAppendableStream(File destination)
            throws FileNotFoundException {
        return new BufferedOutputStream(new FileOutputStream(destination, true));
    }

    private static void appendFile(OutputStream output, File source)
            throws IOException {
        InputStream input = null;
        try {
            input = new BufferedInputStream(new FileInputStream(source));
            IOUtils.copy(input, output);
        } finally {
            IOUtils.closeQuietly(input);
        }
    }
}

如果您不能使用 IOUtils lib,请编写您自己的实现。例子:

class IOUtils {
    private static final int BUFFER_SIZE = 1024 * 4;

    public static long copy(InputStream input, OutputStream output)
            throws IOException {
        byte[] buffer = new byte[BUFFER_SIZE];
        long count = 0;
        int n = 0;
        while (-1 != (n = input.read(buffer))) {
            output.write(buffer, 0, n);
            count += n;
        }
        return count;
    }

    public static void closeQuietly(Closeable output) {
        try {
            if (output != null) {
                output.close();
            }
        } catch (IOException ioe) {
            ioe.printStackTrace();
        }
    }
}
于 2013-02-03T13:53:03.107 回答
0

在其当前形式中,您的pro方法将文件附加a到每个b文件。原因是您的joinf(a, b)方法将 file 附加a到 file b,而不是相反。

要解决它,这个:

joinf(a,b[i]);

应该改成这样:

joinf(b[i], a);

或类似的东西。


仅供参考:您的代码中还有其他各种问题。请检查它们,以便您下次可以编写更好的代码...

  • 在某些情况下,您的 joinf 方法可能会泄漏文件描述符 - 资源应该在finally.
  • 像这样调用System.exit()库方法使得重用变得困难。
  • 您正在丢弃堆栈跟踪。这将使您的应用程序的现场部署副本的调试变得困难。
  • 处理 IOException 的策略是错误的。如果你在那里得到了一个意外的 IOException,那就是大错特错了。打印并继续是一个非常糟糕的主意。
  • 名字projoinf是糟糕的选择。
  • 没有 javadocs
于 2013-02-03T13:42:32.470 回答