Почему метод ToString не рабоатет?

Question

[keddad]

У меня есть некоторый класс

spoiler

using System;

namespace untc
{
    class Fraction
    {

        private long numerator;
        private ushort denumerator;

        public long Numerator { get => numerator; }
        public ushort Denumerator { get => denumerator; }

        public override string ToString()
        {
            return String.Format("{0}/{1}", this.numerator, this.denumerator);
        }

        private void optimise()
        {
            long n = this.numerator;
            long d = this.denumerator;
            long remainder;
            while (d != 0)
            {
                remainder = n % d;
                n = d;
                d = remainder;
            }

            this.numerator /= n;
            this.denumerator /= (ushort)n;
        }

        public static Fraction operator +(Fraction a, Fraction b)
        {
            return new Fraction(a.Numerator * b.Denumerator + b.Numerator * a.Denumerator, a.Denumerator * b.Denumerator);
        }

        public static Fraction operator -(Fraction a, Fraction b)
        {
            return new Fraction(a.Numerator * b.Denumerator - b.Numerator * a.Denumerator, a.Denumerator * b.Denumerator);
        }

        public static Fraction operator *(Fraction a, Fraction b)
        {
            return new Fraction(a.Numerator * b.Numerator, a.Denumerator * b.Denumerator);
        }

        public static Fraction operator /(Fraction a, Fraction b)
        {
            return new Fraction(a.Numerator * b.Denumerator, a.Denumerator * b.Numerator);
        }

        public static bool operator ==(Fraction a, Fraction b)
        {
            return a.Numerator == b.Numerator && a.Denumerator == b.Denumerator;
        }

        public static bool operator !=(Fraction a, Fraction b)
        {
            return a.Numerator != b.Numerator || a.Denumerator != b.Denumerator;
        }

        public Fraction(long numerator, long denumerator)
        {
            if (denumerator == 0)
            {
                throw new System.ArgumentException();
            }

            if (numerator < 0 && denumerator < 0)
            {
                numerator = Math.Abs(numerator);
                denumerator = Math.Abs(denumerator);
            }

            if (denumerator < 0 && denumerator > 0 || denumerator > 0 && numerator < 0)
            {
                numerator = -Math.Abs(numerator);
                denumerator = Math.Abs(denumerator);
            }

            optimise();
        }
    }
}

Когда я хочу распечатать его и делаю Console.Write(new Fraction(1, 2)) или Console.Write(new Fraction(1, 2).ToString()), я получаю пустую консоль вместо ожидаемой строчки. Что я делаю не так?

Answer 1

[Илья]

у вас ToString определен как

return String.Format("{0}/{1}", this.numerator, this.denumerator);

в нем используются поля класса
но в конструкторе вы поля класса не заполняете

сейчас

if (numerator < 0 && denumerator < 0)
            {
                numerator = Math.Abs(numerator);
                denumerator = Math.Abs(denumerator);
            }

            if (denumerator < 0 && denumerator > 0 || denumerator > 0 && numerator < 0)
            {
                numerator = -Math.Abs(numerator);
                denumerator = Math.Abs(denumerator);
            }

должно быть

if (numerator < 0 && denumerator < 0)
            {
                this.numerator = Math.Abs(numerator);
                this.denumerator = Math.Abs(denumerator);
            }

            if (denumerator < 0 && denumerator > 0 || denumerator > 0 && numerator < 0)
            {
                this.numerator = -Math.Abs(numerator);
                this.denumerator = Math.Abs(denumerator);
            }

Можете помечать приватные поля нижним подчеркиванием, тогда не будете путать их с локальными переменными, например _numerator, _denumerator

Comments

[keddad]

Ой, это было очень глупо, спасибо. Странно, что я не получил варнингов при компиляции. Как это вообще сработало, если переменные не были определены? Разве он не должен был упасть с чем то в духе NullPointerException?

[azm13]

keddad, нет, при объявлении поля/переменной примитивного типа (private long numerator;) оно инициализируется значением по умолчанию (в данном случае 0)

[Meloman19]

keddad, это же структуры, они всегда принимают значения по умолчанию)