C++. Проверка знаменателя

[Александр222]

Вопрос. Как сделать так, чтобы знаменатель корректно проверялся на 0. Здесь код, где я проверяю его через if. Еще была попытка через try. Суть в том, что это работает, но программа все равно вычисляет дробь. А надо, чтобы (в случае знаменателя = 0) выводил mistake. Как можно решить этот вопрос?

Rational.cpp

#include <iostream>
#include "Rational.h"

Rational::Rational(int Numerator, int Denominator): Numerator(Numerator), Denominator(Denominator) {}
Rational::Rational(const Rational& R): Numerator{ R.Numerator }, Denominator{ R.Denominator } {}

void Rational::SetNumerator(){}
void Rational::SetDenominator(){}

int Rational::GetNumerator() const { return Numerator; }
int Rational::GetDenominator() const { 
    
    if (Denominator == 0) {
        std::cout << "mistake!";
        return 0;
    }
    else { return Denominator; }
    
}

Rational operator + (const Rational& a, const Rational& b) {
    Rational R (a.GetNumerator() * b.GetDenominator() + b.GetNumerator() * a.GetDenominator(), a.GetDenominator() * b.GetDenominator());

    R.reduce();
    return R;
}

Rational operator - (const Rational& a, const Rational& b) {
    Rational R (a.GetNumerator() * b.GetDenominator() - b.GetNumerator() * a.GetDenominator(), a.GetDenominator() * b.GetDenominator());

    R.reduce();
    return R;
}

Rational operator * (const Rational& a, const Rational& b) {
    Rational R(a.GetNumerator() * b.GetNumerator(), a.GetDenominator() * b.GetDenominator());

    R.reduce();
    return R;
}

Rational operator / (const Rational& a, const Rational& b) {
    Rational R (a.GetNumerator() * b.GetDenominator(), a.GetDenominator() * b.GetNumerator());

    R.reduce();
    return R;
}

bool operator > (const Rational& a, const Rational& b) { return (a.GetNumerator()/b.GetNumerator()) > (a.GetDenominator()/b.GetDenominator()); }
bool operator < (const Rational& a, const Rational& b) { return (a.GetNumerator() / b.GetNumerator()) < (a.GetDenominator() / b.GetDenominator()); }
bool operator == (const Rational& a, const Rational& b) { return (a.GetNumerator() == b.GetNumerator() && a.GetDenominator() == b.GetDenominator()); }
bool operator != (const Rational& a, const Rational& b) { return !(a == b); }

std::ostream& operator << (std::ostream& out, const Rational& R) { 
    out << "Numerator, Denominator: " << R.Numerator << ", " << R.Denominator;
    return out;
};

std::istream& operator >> (std::istream& in, Rational& R) {
    in >> R.Numerator >> R.Denominator;
    return in;
};

Rational.h

#ifndef RATIONAL_H
#define RATIONAL_H
#include <iostream>
#include <sstream>

class Rational {

private:
    int Numerator{};
    int Denominator{};
    
    void reduce() {        
        int N = NOD(Numerator < 0 ? -Numerator : Numerator, Denominator);
        Numerator /= N;
        Denominator /= N;                
    }
    int NOD(int a, int b) const {
        while (a) {
            int t = b % a;
            b = a;
            a = t;
        }
        return b;
    }  
public:
    Rational() = default;
    Rational(int Numerator, int Denominator);
    Rational(const Rational& R);

    void SetNumerator();
    void SetDenominator();

    int GetNumerator() const;
    int GetDenominator() const;
    
    friend Rational operator + (const Rational& a, const Rational& b);
    friend Rational operator - (const Rational& a, const Rational& b);
    friend Rational operator * (const Rational& a, const Rational& b);
    friend Rational operator / (const Rational& a, const Rational& b);

    friend bool operator > (const Rational& a, const Rational& b);
    friend bool operator < (const Rational& a, const Rational& b);
    friend bool operator == (const Rational& a, const Rational& b);
    friend bool operator != (const Rational& a, const Rational& b);

    friend std::ostream& operator << (std::ostream& out, const Rational&);
    friend std::istream& operator >> (std::istream& in, Rational&);
};
#endif

[BDA]

Нужно кидать исключение, а не сразу печатать сообщение. Возможно стоит "не давать" создавать дроби с нулевым знаменателем (кидать исключение в конструкторе). Потом уже ловить исключение и печатать сообщение. Операторы сравнения на больше и меньше сейчас неверно работают. Во-первых, там получается целочисленное деление. Во-вторых, сравнивать стоит, наверное, a.GetNumerator() * b.GetDenominator() и a.GetDenominator() * b.GetNumerator(), чтобы не использовать вещественные числа.

[Александр222]

BDA, Спасибо!