The 'short' keyword is the same as Int16 Type in .net.  

Better said, you're trying to take the result of your conversion (an int16)
and divide it (which, turns it into an int32 because any plainly typed number
is an int32), then place it back into an int16 without converting the result
to int16--returning a casting error.

Placing the division inside the convert statement works because you're taking
the result of the division (an int32) then explicitly converting it to an
int16 (though there's the potential to lose data).

That should work, though I have to ask--why an int16?  Are you sure there
will never be an overflow--especially on file sizes?  Int16s only range from
+/-32768, so be wary.

Here's a quick example:

           FileInfo info = new FileInfo(@"C:\WindowsServer2003-KB340178-SP2-x86-ENU.msi");

           Console.WriteLine(Convert.ToInt16((info.Length / 1024) / 1024)
+ " MB");
           Console.WriteLine(Convert.ToInt16(info.Length / 1024) + " KB");
           Console.WriteLine(info.Length + " Bytes");

HTH.

-dl

--
David R. Longnecker
http://blog.tiredstudent.com

Because dividing by 1000 causes the expression to be an int, not a short.  
Then you try to assign the whole thing to a short, which can't be done  
implicitly (as the error says).

I personally would just use a single cast on the division expression and  
forget the Convert class altogether in this case.  But the solution you  
found is fine too.

Pete

Anodes has brought this to us :

Note: FileInfo.Length returns the number of *bytes*, so you need to
divide it by 1.000.000 (or rather 1024 * 1024) to get the number of
MegaBytes.

Hans Kesting