KMGT変換で「999999」にすると1Mになってしまう

Question

前提

数値をKMGTに変換する処理を作成しました。
ただ「999999」を変換すると1Mになってしまいます。

おそらく下記の部分が不具合の原因かと思いますが、修正方法がわかりません。

            //有効桁数３桁＋指数表記で文字列化
　　　　//※ここが原因？　
            var s = d.ToString("0.00E000");

ご教授いただければ幸いです。

実現したいこと

「999999」を変換すると"999.9K"になるのが理想です。

該当のソースコード

 /// <summary>
        /// 単位をつけた文字列に変換します。
        /// 12 -> 12
        /// 1234 -> 1.23k
        /// 123456 -> 123k
        /// 12345678 -> 12.3m
        /// 
        /// 単位：k, m, b, t, A, B, C, ... Z, AA, AB, AC, ...
        /// 
        /// </summary>
        /// <param name="d"></param>
        /// <returns></returns>
        public static string ToReadableString(this double d)
        {
            //マイナスは扱う気無し
            if (d <= 0) return "0";
            //表示上は整数として見せるので、suffix(kmbtABC等)がつかない場合は少数部を破棄
            if (d <= 1000) return ((int) d).ToString();

            //有効桁数３桁＋指数表記で文字列化
　　　　//※ここが原因？　
            var s = d.ToString("0.00E000");

            //有効数字
            float f = float.Parse(s.Substring(0, 4));

            //10の指数
            var e = int.Parse(s.Substring(5, 3));

            //中途半端な指数は数値に掛け合わせておく
            for (var i = 0; i < e % 3; i++)
            {
                f *= 10;
            }

            return f.ToString("###.##") + LevelToSuffix(e / 3);
        }

        /// <summary>
        /// 10^3ごとにつけるSuffixの一覧
        /// </summary>
        private static string[] _suffixes;

        private static string LevelToSuffix(int level)
        {
            if (_suffixes == null)
            {
                const string AtoZ = " ABCDEFGHIJKLMNOPQRSTUVWXYZ";
                var list = new List<string>();
                list.Add("");
                list.Add("k");
                list.Add("m");
                list.Add("b");
                list.Add("t");
                for (var x = 0; x < AtoZ.Length; x++)
                for (var y = 1; y < AtoZ.Length; y++)
                {
                    var str = string.Format("{0}{1}", AtoZ[x], AtoZ[y]).Trim();
                    list.Add(str);
                }

                _suffixes = list.Take(308).ToArray(); //doubleは指数308まで
            }

            return _suffixes[level];
        }

Answer 1

何らかの課題でしょうか。だとすれば設問に問題がありますね。
というのも、コードを見るに非常に大きい数まで扱いたいようですが、引数がdoubleである限り正確な値が扱えるのはわずか2^53≒9007兆までです。(この記法でいうなら9.007A)
正確でない値をどうするかで考えられるスタンスがいくつかあるので、どういう回答を求められているのか(どうせ上の方は正確でないのに999999が1Mになるのも許容しないのか)、引数がdoubleなのは変えられないのかあたりが分からないとどうにも答えようがありません。

まあ、(引数を文字列にしたいところだが無理なら)適当にやるならdouble最大の17桁で文字列化してから文字列処理をすれば、文字列化が正しく行われる前提で、doubleとして最大限正しい値は出ると思います。
文字列化に正しさを丸投げしているわけですが、(大昔は誤った出力が出ることもあったようだが)最近のまともな環境ならたぶん大丈夫でしょう。

Answer 2

指数表記にこだわらなければ、いかようにでも作れると思いますが・・・

using System;
using System.Collections.Generic;
using System.Linq;

internal static class Program
{
    static void Main(string[] args) {
        print(999999d);
        print(1000000d);
        print(12d);
        print(1234d);
        print(123456d);
        print(12345678d);
        Console.ReadKey();
    }

    static void print(double d) {
        Console.WriteLine($"{d} => {d.ToReadableString()}");
    }

    public static string ToReadableString(this double d) {
        var sign = Math.Sign(d);
        var value = Math.Abs(d);
        int i;
        for (i = 0; i < suffixes.Length - 1; i++) {
            if (value < 1000) {
                break;
            }
            value = Math.Truncate(value / 10) / 100;
        }
        var str = (value * 1000).ToString();
        if (str.Length > 3) {
            str = str.Substring(0, 3) + new string('0', str.Length - 3);
        }
        value = double.Parse(str) / 1000;
        return (sign * value).ToString() + suffixes[i];
    }

    private static string[] _suffixes;
    private static string[] suffixes {
        get {
            if (_suffixes == null) {
                const string AtoZ = " ABCDEFGHIJKLMNOPQRSTUVWXYZ";
                var list = new List<string>();
                list.Add("");
                list.Add("k");
                list.Add("m");
                list.Add("b");
                list.Add("t");
                for (var x = 0; x < AtoZ.Length; x++)
                    for (var y = 1; y < AtoZ.Length; y++) {
                        var str = string.Format("{0}{1}", AtoZ[x], AtoZ[y]).Trim();
                        list.Add(str);
                    }
                _suffixes = list.Take(308).ToArray();
            }
            return _suffixes;
        }
    }
}

ところで、頭３桁のみ有効桁にするのであれば、999999 → 999k になるのでは？

Answer 3

「999999」を変換すると"999.9K"になるのが理想です。

　一般的な丸め処理では、そうはなりません。

　通常、有効桁数を減らす場合、四捨五入など、誤差が小さくなるように変換します。
なので、999999 を "0.00E000" で文字列にする場合、まず、丸め処理で、1000000になるので、ご希望の結果にはなりません。
質問者さんの仕様を満たすためには、文字列にする前に、切り捨てによる丸め処理を行うことが必要です。

Comments

[MissingPiece598]

下記のようにした見たのですが、うまくいきません。
「文字列にする前に、切り捨てによる丸め処理」とはどのようにしたらよいでしょうか。
お手数ですが、ご教授いただけないでしょうか。

//dが「999999」
　　　　string temp = d.ToString();
//有効桁数３桁
int significantDigit = int.Parse(temp.Substring(0, 3));
//指数表記で文字列化
var suffix = significantDigit.ToString("0.00E000");

[YT0014]

順番が逆です。
文字列返還の前に切り捨てです。
桁数不明なので、割と面倒な処理をしないと切り捨てできません。
未チェックですが、以下にコード
// 上から３桁目の桁数の取得
int digit = (int)Math.Log10(d) - 2;
// 上から３桁目を示す１０の乗数を計算
double base = Math.Pow(10.0, (double)digit);
// 切り捨てで上位３桁のみにする
double d2 = Math.Floor(d / base) * base;

[MissingPiece598]

ご回答ありがとうございます。
999kになりました、変換すると"999.9K"にする方法自力で見つけたいと思います。

ご教示いただきありがとうございます。

[YT0014]

探す前に、こちらで提示したコードを、詳細に理解されることをお勧めします。
(int)Math.Log10(d) は、何を求めているのか？
なぜ、２を引いているのか？
Math.Pow(10.0, (double)digit) は、何を求めているのか？
Math.Floor(d / base) * base は、何を求めているのか？
以上の質問に答えられますか？
参考までに、各関数のヘルプを挙げておきます。
https://learn.microsoft.com/ja-jp/dotnet/api/system.math.log10?view=net-7.0
https://learn.microsoft.com/ja-jp/dotnet/api/system.math.pow?view=net-7.0
https://learn.microsoft.com/ja-jp/dotnet/api/system.math.floor?view=net-7.0