Question

I want to append two text files together.

I have one file with a carriage return line feed at the end. Observe file A which is 28 bytes.

this is a line in the file\n

then I have another file which is the same thing without the new line. Observe file B which is 26 bytes.

this is a line in the file

I want to append the same file to itself (file A to A, and file B to B) and compare the byte counts.

However, when using StreamReader.ReadLine() on file A, I get a value returned but MSDN says:

A line is defined as a sequence of characters followed by a line feed (“\n”), a carriage return (“\r”) or a carriage return immediately followed by a line feed (“\r\n”). The string that is returned does not contain the terminating carriage return or line feed. The returned value is null if the end of the input stream is reached.

However, there is no crlf in the file.

How can I safely append these files without adding an extra line break at the end? For example, StreamWriter.WriteLine() will put an extra line break on file A when I don’t want it to. What would be an ideal approach?

Answer 1

StreamReader and StreamWriter (which derive from TextReader and TextWriter) are not suitable for situations requiring an exact form of binary data. They are high level abstractions of a file which consists of bytes, not text or lines. In fact, not only could you wind up with different number of newlines, but depending on the environment you might write out a line terminator other than the expected CR/LF.

You should instead just copy from one stream to another. This is quite easy actually.

var bytes = File.ReadAllBytes(pathIn);
var stream = File.Open(pathOut, FileMode.Append);
stream.Write(bytes, 0, bytes.Length);
stream.Close();

If the size of the file is potentially large, you should open both the input and output file at the same time and use a fixed-sized buffer to copy a block at a time.

using (var streamIn = File.Open(pathIn, FileMode.Read))
using (var streamOut = File.Open(pathOut, FileMode.Append)) {

    var bytes = new byte[BLOCK_SIZE];

    int count;
    while ((count=streamIn.Read(bytes, 0, bytes.Length)) > 0) {
        streamOut.Write(bytes, 0, count);
    }

}

Also worth noting is that the above code could be replaced by Stream.CopyTo which is new in .NET 4.