計算結果の小数点丸め処理を今より縮める方法

Question

C#
コード

今このように書いていて、　
これをさらに縮めたいと考えているのですがなにかいい方法はありますか？？
三項条件演算子なども勧めらましたがよくわからなくて、、、

//小数点第一位選択時
if (rdbFirst_decimal.Checked)
{

            switch (cmbCalc.Text)
            {
                //端数をより小さい値に切り捨てるならFloorメソッド
                case "切り捨てる":
                    total = Math.Floor(total);
                    break;

                //より大きい値に切り上げるならCeilingメソッド
                case "切り上げる":
                    total = Math.Ceiling(total);
                    break;

                // 四捨五入するにはMathクラスのRoundメソッド
                case "四捨五入":
                    total = Math.Round(total);
                    break;
            }
        }

        //小数点第二位選択時
        //  ( / 10 ) をして切り捨てを行う桁を整数まで移動し、切り捨て後元に戻すようにする。
        
        if (rdbScond_decimal.Checked)
        {
            switch (cmbCalc.Text)
            {
                case "切り捨てる":
                    total = Math.Floor(total * 10.0) / 10.0;
                    break;

                case "切り上げる":
                    total = Math.Ceiling(total * 10.0) / 10.0;
                    break;

                case "四捨五入":
                    total = Math.Round(total * 10.0) / 10.0;
                    break;
            }
        }

Answer 1

まるめるロジックは関数として切り出してしまった方がいいと思います。

        private void Hoge(decimal total)
        {
            var decimals = rdbFirst_decimal.Checked ? 0 : 1;
            var total = RoundDecimal(total,decimals,cmbCalc.Text);
        }


        private decimal RoundDecimal(decimal target, int decimals,string calcStatus)
        {
            var i =new decimal(Math.Pow(10,decimals));

            switch (calcStatus)
            {
                case "切り捨てる":
                    return Math.Floor(target * i)/i;

                case "切り上げる":
                    return Math.Ceiling(target*i)/i;

                case "四捨五入":
                    return Math.Round(target*i)/i;

                default:
                    return target;
            }
        }

Comments

[xlostdjx]

ご回答ありがとうございます。　
もう一つ質問なんですが　
計算結果をラベルに出力しているのですがその計算結果を右寄せにする方法を教えてください、
またテキストボックスではTextAlignで指定して右寄せにしてのですが
ラベルは lblText.TextAlign = ContentAlignment.BottomRight;
このようにしたのですが変わりませんでした。

[xlostdjx]

改善できました！！

[xlostdjx]

あと
ComboBox（丸め区分）をコード上で値の設定を行う事についてなんですけど
おすすめのやり方をおしえていただきたいです。

[ponpu1601]

ComboBoxにはUIに表示されるものと、選択された時に取得できる値を別々に設定できます。
表示されるのは「四捨五入」等わかりやすい言葉にしておいて、
選択されたときの値はEnum等で定義したものにしておくと、
コードの変更も少なく、わかりやすいものになります。
WindowsFormsならこちら
https://dobon.net/vb/dotnet/control/tbdisplaymember.html

WPFならこちら
http://iyemon018.hatenablog.com/entry/2015/10/17/203939
が参考になりそうです。

Answer 2

double keta=1.0;

if (rdbScond_decimal.Checked){
  keta=10.0;
}else if(rdTherd_decimal.Checked){
  keta=100.0;
}else if(rdForth_decimal.Checked){
  keta=1000.0;
}
switch (cmbCalc.Text)
{
  case "切り捨てる":
  total = Math.Floor(total * keta) / keta;
  break;

  case "切り上げる":
  total = Math.Ceiling(total * keta) / keta;
  break;

  case "四捨五入":
  total = Math.Round(total * keta) / keta;
  break;
}

ちょっと拡張

Comments

[xlostdjx]

ご回答ありがとうございます。　
もう一つ質問なんですが　
計算結果をラベルに出力しているのですがその計算結果を右寄せにする方法を教えてください、
またテキストボックスではTextAlignで指定して右寄せにしたのですが
ラベルは lblText.TextAlign = ContentAlignment.BottomRight;
このようにしたのですが変わりませんでした。

[xlostdjx]

改善できました！！

[y_waiwai]

http://codepanic.itigo.jp/cs/textbox_align.html

Label の場合はAutoSizeをOFFにしてサイズを決めうちにしないと思うようにならないですね

[xlostdjx]

あと
ComboBox（丸め区分）をコード上で値の設定を行う事についてなんですけど
おすすめのやり方をおしえていただきたいです。

[y_waiwai]

ComboBoxにはSelectedIndexというプロパティがあって、選択されている項目の番号を返します
それを使えば、項目の表示を変えても、コードの修正はしないで済みます