One Byte Missing After Using FileStream

[Terry]

In the code below, I open a file using filestream, copy it to a byte
array, and the write that array out to a new file. But when I check
the size of the original file and the new file, the new file has one
less byte than the original file (although the file itself still works
fine). I was hoping someone could explain what is going on (why is
there a byte missing, and what might that byte be).

string originalFile = @"C:\MP3Test\Or iginal.mp3";
string copiedFile = @"C:\MP3Test\Co pied.mp3";

// Read file into byte array
FileStream fs1 = new FileStream(orig inalFile, FileMode.OpenOr Create,
FileAccess.Read );
byte[] mp3ByteArray = new byte[fs1.Length];
fs1.Read(mp3Byt eArray, 0, System.Convert. ToInt32(fs1.Len gth));
fs1.Close();

// Write same byte array to new file
int arraySize = new int();
arraySize = mp3ByteArray.Ge tUpperBound(0);
FileStream fs2 = new FileStream(copi edFile, FileMode.OpenOr Create,
FileAccess.Writ e);
fs2.Write(mp3By teArray, 0, arraySize);
fs2.Close();

// Compare size of original an new file
FileInfo fileInfo1 = new FileInfo(origin alFile);
FileInfo fileInfo2 = new FileInfo(copied File);

Console.WriteLi ne("Size of original file: " +
fileInfo1.Lengt h.ToString());
// output====> 20629 bytes
Console.WriteLi ne("Size of copied file: " +
fileInfo2.Lengt h.ToString());
// output====> 20628 bytes. One byte less than original file.

Thanks,

Terry

[Nicholas Paldino [.NET/C# MVP]]

Terry,

The reason you are getting this is because you are using a call to
GetUpperBound(0 ) on the array. This will return the index of the last
element in the array, which, in a zero-based array, turns out to be the
length - 1.

You can just do this:

int arraySize = mp3ByteArray.Le ngth;

And it will work out fine.

Hope this helps.
--
- Nicholas Paldino [.NET/C# MVP]
- mv*@spam.guard. caspershouse.co m

"Terry" <te*******@yaho o.com> wrote in message
news:11******** *************@g 44g2000cwa.goog legroups.com...

In the code below, I open a file using filestream, copy it to a byte
array, and the write that array out to a new file. But when I check
the size of the original file and the new file, the new file has one
less byte than the original file (although the file itself still works
fine). I was hoping someone could explain what is going on (why is
there a byte missing, and what might that byte be).

string originalFile = @"C:\MP3Test\Or iginal.mp3";
string copiedFile = @"C:\MP3Test\Co pied.mp3";

// Read file into byte array
FileStream fs1 = new FileStream(orig inalFile, FileMode.OpenOr Create,
FileAccess.Read );
byte[] mp3ByteArray = new byte[fs1.Length];
fs1.Read(mp3Byt eArray, 0, System.Convert. ToInt32(fs1.Len gth));
fs1.Close();

// Write same byte array to new file
int arraySize = new int();
arraySize = mp3ByteArray.Ge tUpperBound(0);
FileStream fs2 = new FileStream(copi edFile, FileMode.OpenOr Create,
FileAccess.Writ e);
fs2.Write(mp3By teArray, 0, arraySize);
fs2.Close();

// Compare size of original an new file
FileInfo fileInfo1 = new FileInfo(origin alFile);
FileInfo fileInfo2 = new FileInfo(copied File);

Console.WriteLi ne("Size of original file: " +
fileInfo1.Lengt h.ToString());
// output====> 20629 bytes
Console.WriteLi ne("Size of copied file: " +
fileInfo2.Lengt h.ToString());
// output====> 20628 bytes. One byte less than original file.

Thanks,

Terry

[Jon Skeet [C# MVP]]

kids_pro <ki******@yahoo .com> wrote:

I am having the similiar problem.
But the different between the original file and the copy one is alot more.

int bufferSize = 102400;
byte[] read = new byte[bufferSize];

Stream fs = new
FileStream("E:\ \anders_hejlsbe rg_linq_2005.wm v",FileMode.Ope n);

Stream fs2 = new
FileStream("D:\ \anders_hejlsbe rg_linq_2005.wm v",FileMode.Cre ate);

int chunk=0;
while((chunk=fs .Read(read,0,bu fferSize))>0){
if(chunk < bufferSize){

/* truncate the byte it should reach end of file */
byte[] finalByte = new byte[chunk];
Array.Copy(read ,finalByte,chun k);
fs.Write(finalB yte,0,chunk);
}else{
fs2.Write(read, 0,bufferSize);
}

read = new byte[bufferSize];
}
fs.Close();
fs2.Close();

Please refer to attach screenshot for the size different.
Why the size is different? Have I done any mistakes?

Well, I'm not sure why you're copying the array in the case where you
haven't read the whole file in one go - just use
fs2.Write(read, 0, chunk)

However, the error in your program is that by introducing the extra
complexity, you've got your other Write call to the wrong stream -
you're writing to the stream you've been reading from!

Note that you should also use "using" statements to make sure that your
FileStreams get closed even if an exception is thrown.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[Terry]

Thanks for the insights Nicholas and Jon. I'll spend some more time
studying this code and your responses so I understand it better.

This code was extracted from two methods I created to save .mp3's to a
database, and then write them out to a file again. I uncovered the
error when I wrote a unit test to compare the file before and after
saving to the database.

Thanks again.