Question

我有一个实用程序函数，它将以适当的形式显示文件大小，就像Windows资源管理器那样，即;将它转换为最接近的KB，MB，GB等。我想知道我写的代码是否正确，以及它是否可以更简单。

我写的功能如下：

public static function formatFileSize(bytes:int):String
    {
        if(bytes < 1024)
            return bytes + " bytes";
        else
        {
            bytes /= 1024;
            if(bytes < 1024)
                return bytes + " Kb";
            else
            {
                bytes /= 1024;
                if(bytes < 1024)
                    return bytes + " Mb";
                else
                {
                    bytes /= 1024;
                    if(bytes < 1024)
                        return bytes + " Gb";
                }
            }
        }
        return String(bytes);
    }

虽然它目前对我有用，但我觉得它可以用更简单的方式编写，甚至可以优化。

提前致谢

Answer 1

这是一种更简单的方法：

private var _levels:Array = ['bytes', 'Kb', 'MB', 'GB', 'TB', 'PB', 'EB', 'ZB', 'YB'];

private function bytesToString(bytes:Number):String
{
    var index:uint = Math.floor(Math.log(bytes)/Math.log(1024));
    return (bytes/Math.pow(1024, index)).toFixed(2) + this._levels[index];
}

我将它包含在yottabytes中以获得完整性：）

Answer 2

@J_A_X有最好的方法来做到这一点，但是对于将来，当你发现你有像你一样的嵌套if...else...if语句时，我建议你提前返回。

public static function formatFileSize(bytes:int):String
{
    if(bytes < 1024)
        return bytes + " bytes";

    bytes /= 1024;
    if(bytes < 1024)
        return bytes + " Kb";

    bytes /= 1024;
    if(bytes < 1024)
        return bytes + " Mb";

    bytes /= 1024;
    if(bytes < 1024)
        return bytes + " Gb";

    return String(bytes);
}

用于将字节转换为kb，mb，gb等的ActionScript代码

2 个答案: