C#での分数の計算について

Question

初めまして。Unity、C#を勉強中の初心者です。
こちらのサイトのチャレンジ問題prob8-6.について質問させていただきたいと思います。

2つの分数同士の足し算をし、その結果を分数で表示するプログラムを作りなさい。このとき、分子、分母共に最大値が10で、最小値は分母が2、分子が１とする。それらの数値をランダムに発生させ、以下のように結果を表示させなさい。ただし、計算結果は、分子と分母がきちんと約分されていることとする。また、分子が分母の数で割り切れる場合は、整数として表示するものとする。

例
1/5 + 2/3 = 13/15 ← 通常のケース
2/3 + 3/8= 1.1/24 ← 帯分数になるケース
1/6 + 1/3= 1/2 ← 約分されるケース
1/2 + 1/2 = 1 ← 整数になるケース

自分なりにコードを作成し(後述)デバッグを行ってみたところ、上記設問で求められている計算結果が問題なく表示できているようなのでホッと一安心…なのですが、さらに見やすくわかりやすいコードにするにはどうすればいいのかわからず悩んでおります。
改良点・改善点等ございましたら教えていただければ幸いです。

該当のソースコード

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace Problem8_6
{
    class Program
    {
        static void Main(string[] args)
        {
            Random rand = new Random();
            int denominator1 = rand.Next(2, 11); // 分母１
            int numerator1 = rand.Next(1, 11);   // 分子１
            int denominator2 = rand.Next(2, 11); // 分母２
            int numerator2 = rand.Next(1, 11);   // 分子２
            int n = 0;                           // 整数部分をnとする
            Console.Write("{0}/{1} + {2}/{3} = ", numerator1, denominator1, numerator2, denominator2);

            int resultDenominator = denominator1 * denominator2;
            int resultNumerator = numerator1 * denominator2 + numerator2 * denominator1;

            int m = Gcd(resultDenominator, resultNumerator);
            resultDenominator /= m;
            resultNumerator /= m;

            while (resultNumerator >= resultDenominator)
            {
                resultNumerator -= resultDenominator;
                n++;
            }
            if (resultNumerator % resultDenominator == 0 && n != 0)
            {
                Console.WriteLine("{0}", n);
            }
            else if (n != 0)
            {
                Console.WriteLine("{0}.{1}/{2}", n, resultNumerator, resultDenominator);
            }
            else
            {
                Console.WriteLine("{0}/{1}", resultNumerator, resultDenominator);
            }
        }
        static int Gcd(int a, int b)   // 最大公約数を求める
        {
            int x;
            while ((x = a % b) != 0)
            {
                a = b;
                b = x;
            }
            return b;
        }
    }
}

Answer 1

while (resultNumerator >= resultDenominator)
{
resultNumerator -= resultDenominator;
n++;
}

商と余りで書いた方が素直である（＝わかりやすい）ように思います．

n = resultNumerator / resultDenominator;
resultNumerator = resultNumerator % resultDenominator;

直後のif

if (resultNumerator % resultDenominator == 0 && n != 0)

は，
if( resultNumerator==0 )で．

Comments

[4CDAxx]

回答いただきましてありがとうございます。
ご指摘いただいた内容を基にコードを組みなおすことができました。

Answer 2

######修正前

            int n = 0;                           // 整数部分をnとする
            
            while (resultNumerator >= resultDenominator)
            {
                resultNumerator -= resultDenominator;
                n++;
            }
            if (resultNumerator % resultDenominator == 0 && n != 0)
            {
                Console.WriteLine("{0}", n);
            }
            else if (n != 0)
            {
                Console.WriteLine("{0}.{1}/{2}", n, resultNumerator, resultDenominator);
            }
            else
            {
                Console.WriteLine("{0}/{1}", resultNumerator, resultDenominator);
            }

######修正後

            // int n = 0;を消去
            // whileループを使わずif文で動作するよう条件式を変更

            if (resultNumerator % resultDenominator == 0)
            {
                Console.WriteLine("{0}", resultDenominator / resultDenominator);
            }
            else if (resultNumerator > resultDenominator)
            {
                Console.WriteLine("{0}.{1}/{2}", resultNumerator / resultDenominator, resultNumerator % resultDenominator, resultDenominator);
            }
            else
            {
                Console.WriteLine("{0}/{1}", resultNumerator, resultDenominator);
            }