Hsv to Rgb method

gistfile1.cs

        /// <summary>
        /// RGBへ変換する
        /// </summary>
        /// <returns>RGBオブジェクト</returns>
        public RGB ToRGB()
        {
            // まず、もしSが0.0と等しいなら、最終的な色は無色もしくは灰色である。
            if (Saturation == 0)
            {
                return new RGB(Value, Value, Value);
            }

            double r = 0, g = 0, b = 0;
            double f = 0;
            double p = 0, q = 0, t = 0;

            var h = Math.Min(360.0, Math.Max(0, Hue));
            var s = Math.Min(1.0, Math.Max(0, Saturation));
            var v = Math.Min(255.0, Math.Max(0, Value));

            h = (int)(h / 60);
            f = h - Hue;
            p = v * (1 - s);
            q = v * (1 - f * s);
            t = v * (1 - (1 - s) * s);

            if (h == 0)
            {
                r = v; g = t; b = p;
            }
            else if (h == 1)
            {
                r = q; g = v; b = p;
            }
            else if (h == 2)
            {
                r = p; g = v; b = t;
            }
            else if (h == 3)
            {
                r = p; g = q; b = v;
            }
            else if (h == 4)
            {
                r = t; g = p; b = v;
            }
            else if (h == 5)
            {
                r = v; g = p; b = q;
            }

            return new RGB(r, g, b);
        }

hが0より少ない、360より多い時の計算が抜けていたのと、fの求め方がおかしかったので修正してみた。

    /// <summary>
    /// RGBへ変換する
    /// </summary>
    /// <returns>RGBオブジェクト</returns>
    public RGB ToRGB()
    {
        // まず、もしSが0.0と等しいなら、最終的な色は無色もしくは灰色である。
        if (Saturation == 0)
        {
            return new RGB(Value, Value, Value);
        }

        double r = 0, g = 0, b = 0;
        double f = 0;
        double p = 0, q = 0, t = 0;

        //var h = Math.Min(360.0, Math.Max(0, Hue));
        // 角座標系で、Hの範囲は0から360までであるが、その範囲を超えるHは360.0で
        // 割った剰余（またはモジュラ演算）でこの範囲に対応させることができる。
        // たとえば-30は330と等しく、480は120と等しくなる。
        var h = Hue % 360;
        var s = Math.Min(1.0, Math.Max(0, Saturation));
        var v = Math.Min(255.0, Math.Max(0, Value));

        var hi = (int)(h / 60);
        f = (h / 60) - hi;
        p = v * (1 - s);
        q = v * (1 - f * s);
        t = v * (1 - (1 - s) * s);

        if (hi == 0)
        {
            r = v; g = t; b = p;
        }
        else if (hi == 1)
        {
            r = q; g = v; b = p;
        }
        else if (hi == 2)
        {
            r = p; g = v; b = t;
        }
        else if (hi == 3)
        {
            r = p; g = q; b = v;
        }
        else if (hi == 4)
        {
            r = t; g = p; b = v;
        }
        else if (hi == 5)
        {
            r = v; g = p; b = q;
        }

        return new RGB(r, g, b);
    }