C# string.format with decimal format.

Question

DriveInfo[] drives = DriveInfo.GetDrives();
for (int i = 0; i < drives.Length; i++)
{
    if (drives[i].IsReady)
    {
        Console.WriteLine("Drive {0} - Has free space of {1} GB",drives[i].ToString(),(drives[i].TotalFreeSpace/1024/1024/1024).ToString("N2"));
    }
}

Output:

Drive C:\ - Has free space of 70,00 GB
Drive D:\ - Has free space of 31,00 GB
Drive E:\ - Has free space of 7,00 GB
Drive F:\ - Has free space of 137,00 GB

All end up with ,00 but I need to show real size. So which format is suitable?

canon · Accepted Answer · 2014-01-23 18:57:14Z

4

The format string doesn't have anything to do with it. Your integer operations are discarding any remainder.

3920139012 / 1024 / 1024  / 1024 // 3

Specify decimals using the m suffix like so:

3920139012 / 1024m / 1024m / 1024m // 3.6509139575064182281494140625

Alternatively:

3920139012 / Math.Pow(1024, 3) // 3.65091395750642

This might be a little more clear:

var gb = Math.Pow(1024, 3);
foreach(var drive in DriveInfo.GetDrives())
{   
    if(drive.IsReady)
    {
        Console.WriteLine("Drive {0} - Has free space of {1:n2} GB",
            drive.Name,
            drive.TotalFreeSpace / gb);
    }
}

edited Jan 23, 2014 at 18:57

answered Jan 21, 2014 at 21:19

canon

41.8k10 gold badges77 silver badges102 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

D Stanley · Accepted Answer · 2014-01-21 21:20:20Z

4

Becasue you are doing integer division which truncates decimal remainders. Use floating-point division instead:

drives[i].TotalFreeSpace/1024.0/1024.0/1024.0

or

drives[i].TotalFreeSpace / (1024.0 * 1024.0 * 1024.0)

answered Jan 21, 2014 at 21:20

D Stanley

153k12 gold badges189 silver badges257 bronze badges

Collectives™ on Stack Overflow

C# string.format with decimal format.

2 Answers 2

Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Related