Не ловится исключение

@troll_hunter · Регистрация: 02.04.2017

Author24 — интернет-сервис помощи студентам

Здравствуйте, уважаемые форумчане.

Продолжаю проходить онлайн-курс по C++.
На данный момент задача реализовать калькулятор обыкновенных дробей.

Не могу понять, почему в одном случае exception ловится, а в другом - нет.
Подав на вход, например, "4/5 / 0/8", хочу получить "Division by zero", но получаю "0/1".

Мой код:

C++

#include <iostream>
#include <sstream>
#include <iomanip>
#include <algorithm>
#include <map>
#include <set>
#include <vector>
#include <exception>
 
using namespace std;
 
 
int CalcGCD(int a, int b) {
 
    while (a > 0 && b > 0) {    // пока a и b больше нуля
        if (a > b) {            // если а больше b
            a = a % b;          // а = остаток от деления a на b
        }
        else {                  // если b > a
            b = b % a;          // b = остаток от дления b на a
        }
    }
    return a + b;
}
 
int UnionDenominator(int a, int b) {
    if (a > b) {
        if (a % b == 0) {
            return a;
        }
        else {
            return a * b;
        }
    }
    else {
        if (b % a == 0) {
            return b;
        }
        else {
            return b * a;
        }
    }
}
 
void EnsureZero(int a) {
    if (a == 0) {
        throw invalid_argument("Invalid argument");
    }
}
 
 
class Rational {
public:
    Rational() {
        // Реализуйте конструктор по умолчанию
        numerator = 0;
        denominator = 1;
        //full = 0;
    }
 
    Rational(int new_numerator, int new_denominator) {
        // Реализуйте конструктор
        //full = 0;
 
        EnsureZero(new_denominator);
 
        if ((new_numerator < 0 && new_denominator < 0) || (new_numerator > 0 && new_denominator < 0)) {
            new_numerator *= (-1);
            new_denominator *= (-1);
        }
 
        numerator = new_numerator / CalcGCD(new_numerator, new_denominator);     // функция CalcGCD считает наименьший общий делитель
        denominator = new_denominator / CalcGCD(new_numerator, new_denominator);
 
    }
 
    int Numerator() const {
        // Реализуйте этот метод
        return numerator;
    }
 
    int Denominator() const {
        // Реализуйте этот метод
        return denominator;
    }
 
    //int Full() const {
    //  return full;
    //}
 
   // Duration operator+(const Duration& lhs, const Duration& rhs) {
   //   return Duration(lhs.hour + rhs.hour, lhs.min + rhs.min);
   //}
 
    bool CompareDenominators(const Rational& lhs, const Rational& rhs) {
        if (lhs.denominator == rhs.denominator) {
            return true;
        }
        else {
            return false;
        }
 
 
    }
 
    Rational& operator =(const Rational& rhs) {
        denominator = rhs.denominator;
        numerator = rhs.numerator;
        return *this;
    }
 
    friend istream& operator >>(istream& stream, Rational& rational);
 
private:
    // Добавьте поля
    int numerator, denominator;
    //int full;
};
 
 
istream& operator >>(istream& stream, Rational& rational) {
 
 
    int a = 0;
    int b = 1;
 
    if (!(stream >> a) || !stream.ignore(1) || !(stream >> b))          // если из потока не считалось в а или не пропустился один символ, или не считалось в b, то
        return stream;                                                  // то возвращаем поток
    EnsureZero(b);
    rational.numerator = a / CalcGCD(a, b);                             // CalcGCD считает наибольший общий делитель
    rational.denominator = b / CalcGCD(a, b);
 
 
    // Rational r(new_nominator, new_denominator);
    return stream;
   }
 
 
bool operator ==(const Rational& lhs, const Rational& rhs) {
    if ((lhs.Denominator() == rhs.Denominator()) && (lhs.Numerator() == rhs.Numerator())) {
        return true;
    }
    else {
        return false;
    }
}
 
 
 
Rational operator +(const Rational& lhs, const Rational& rhs) {
    if (lhs.Denominator() == rhs.Denominator()) {
        return Rational(lhs.Numerator() + rhs.Numerator(), rhs.Denominator());
    }
    else {
        int z = UnionDenominator(lhs.Denominator(), rhs.Denominator());
        return Rational((lhs.Numerator() * (z / lhs.Denominator())) + (rhs.Numerator() * (z / rhs.Denominator())), z);
    }
}
 
Rational operator -(const Rational& lhs, const Rational& rhs) {
    if (lhs.Denominator() == rhs.Denominator()) {
        return Rational(lhs.Numerator() + rhs.Numerator(), rhs.Denominator());
    }
    else {
        int z = UnionDenominator(lhs.Denominator(), rhs.Denominator());
        return Rational((lhs.Numerator() * (z / lhs.Denominator())) - (rhs.Numerator() * (z / rhs.Denominator())), z);
    }
}
 
Rational operator *(const Rational& lhs, const Rational& rhs) {
    return Rational(lhs.Numerator() * rhs.Numerator(), lhs.Denominator() * rhs.Denominator());
}
 
Rational operator /(const Rational& lhs, const Rational& rhs) {
 
    if (rhs.Numerator() == 0) {
    throw domain_error("Division by zero");
    }
    return Rational(lhs.Numerator() * rhs.Denominator(), lhs.Denominator() * rhs.Numerator());
}
 
 
bool operator <(const Rational& lhs, const Rational& rhs) {
    if (lhs.Denominator() == rhs.Denominator()) {
        return lhs.Numerator() < rhs.Numerator();
        }
        else if (lhs.Numerator() == rhs.Numerator()) {
        return rhs.Denominator() < lhs.Denominator();
        }
        else {
            int z = UnionDenominator(lhs.Denominator(), rhs.Denominator());
            return (lhs.Numerator() * (z / lhs.Denominator())) < (rhs.Numerator() * (z / rhs.Denominator()));
        }
}
 
ostream& operator <<(ostream& stream, const Rational& rational) {
    //stream << setfill('0');
    stream << rational.Numerator() << '/' << rational.Denominator();
    return stream;
}
 
void PrintRational(const Rational& rational) {
    //if (rational.Full() != 0) {
    //  cout << rational.Full() << " ";
    //}
    //if ()
    cout << rational.Numerator() << "/" << rational.Denominator();
}
 
 
 
int main() {
 
 
    try {
        Rational r(1, 0);
        cout << "Doesn't throw in case of zero denominator" << endl;
        return 1;
    } catch (invalid_argument&) {
    }
 
    try {
        auto x = Rational(1, 2) / Rational(0, 1);
        cout << "Doesn't throw in case of division by zero" << endl;
        return 2;
    } catch (domain_error& r) {
    }
 
    cout << "OK" << endl;
 
 
   Rational right;
   Rational left;
   char op;
 
   try {
   cin >> right >> op >> left;
   if (op == '+') {
       cout << left + right;
       }
   else if (op == '-') {
       cout << left - right;
   }
   else if (op == '/') {
       cout << left / right;
   }
   else {
       cout << left * right;
       }
   }
   catch (const exception& ex) {
       cout << ex.what();
   }
 
   return 0;
}

Вот этот кусок выдаёт "ОК":

C++

    try {
        auto x = Rational(1, 2) / Rational(0, 1);
        cout << "Doesn't throw in case of division by zero" << endl;
        return 2;
    } catch (domain_error& r) {
    }
 
    cout << "OK" << endl;

Перегразка оператора / совершена:

C++

Rational operator /(const Rational& lhs, const Rational& rhs) {
 
    if (rhs.Numerator() == 0) {
    throw domain_error("Division by zero");
    }
    return Rational(lhs.Numerator() * rhs.Denominator(), lhs.Denominator() * rhs.Numerator());
}

Но почему-то не срабатывает исключение здесь (пробовал писать даже отдельный try для него):

C++

1
2
3

else if (op == '/') {
       cout << left / right;
}

@oleg-m1973 · 21.06.2020, 11:42

Сообщение от troll_hunter

Вот этот кусок выдаёт "ОК":

А что он по-твоему должен выдавать? Сделай

C++

try
{
    auto x = Rational(1, 2) / Rational(0, 1);
    cout << "Doesn't throw in case of division by zero" << endl;
    return 2;
} 
catch (const domain_error& r) 
{
    cout << "domain_error" << endl;
    return -1;
}

@TheCalligrapher · 21.06.2020, 11:52

Сообщение от troll_hunter

Подав на вход, например, "4/5 / 0/8", хочу получить "Division by zero", но получаю "0/1".

Это с чего это вдруг?

У вас же вводится

C++

1	cin >> right >> op >> left;

именно в таком порядке, а потом делится left / right, именно в таком порядке. То есть тут нет никакого деления на 0. Вы делите 0, а не на 0.

Почему у вас порядок ввода вдруг стал задом-наперед?

@troll_hunter · 22.06.2020, 10:46 **[ТС]**

oleg-m1973, здесь проблем нет.
Я, наоборот, говорил о том, что выводится "ОК", но считает неверно.

TheCalligrapher указал на ошибку. Я почему-то перепутал левую и правую часть. И изначально считывал в right вместо left, из-за этого и логика деления (и вычитания) поломалась.

TheCalligrapher, благодарю!

Новые блоги и статьи
Объекты и их имена hw_wired 28.01.2025 Введение: Основы объектного подхода и их роль в информатике В современном мире информационных технологий объектный подход стал фундаментальным принципом организации и обработки информации. Этот. . .	Элементы алгоритмизации hw_wired 28.01.2025 Основы алгоритмизации В современном мире алгоритмы играют фундаментальную роль в развитии информационных технологий и программирования. Понимание основ алгоритмизации является ключевым элементом в. . .	Человек и информация hw_wired 28.01.2025 Введение: роль информации в познании мира В современном мире информация играет фундаментальную роль в процессе познания окружающей действительности. Она представляет собой совокупность сведений об. . .	Компьютер и информация hw_wired 28.01.2025 Эволюция вычислительных машин История развития вычислительной техники начинается задолго до появления первых электронных устройств. Человечество всегда стремилось упростить процесс вычислений и. . .	Информационные технологии hw_wired 28.01.2025 Введение в современные технологии работы с информацией В современном мире информационные технологии стали неотъемлемой частью практически всех сфер человеческой деятельности. Они существенно. . .	Информация вокруг нас hw_wired 28.01.2025 Основные понятия информации В современном мире понятие информации является фундаментальным и охватывает практически все сферы человеческой деятельности. Информация представляет собой совокупность. . .
Компьютер для начинающих hw_wired 28.01.2025 Введение в мир компьютерных технологий В современном мире информация стала одним из важнейших ресурсов человечества, определяющим развитие общества и технологий. Наша жизнь неразрывно связана с. . .	[golang] 189. Rotate Array alhaos 28.01.2025 Повороты рукоятки, целочисленный слайс нужно сдвинуть на целое положительное число. Мне очень нравится решение на GO / / https:/ / leetcode. com/ studyplan/ top-interview-150/ package topInterview . . .	КуМир: решение задач на матрицы bytestream 28.01.2025 КуМир представляет собой среду для обучения программированию, которая включает в себя мощные инструменты для работы с матрицами. Матрица в программировании - это двумерный массив, состоящий из. . .	КуМир: решение задач на строки bytestream 28.01.2025 В системе программирования КуМир работа со строковыми данными является одним из важнейших аспектов создания программ. Строки представляют собой последовательности символов, заключенные в кавычки,. . .	КуМир: решение геометрических задач bytestream 28.01.2025 Программирование геометрических задач в среде КуМир становится всё более актуальным в обучении школьников и студентов. КуМир — это разработанная в России обучающая программная среда, предназначенная. . .	КуМир, исполнитель Водолей: Задачи и решения bytestream 28.01.2025 КуМир — это образовательная среда для обучения программированию. Она предлагает пользователям разнообразные инструменты для разработки и отладки программ, что особенно ценно для студентов и. . .

@troll_hunter 0 / 0 / 0 Регистрация: 02.04.2017 Сообщений: 10
	22.06.2020, 10:46 [ТС]	4
	oleg-m1973, здесь проблем нет. Я, наоборот, говорил о том, что выводится "ОК", но считает неверно. TheCalligrapher указал на ошибку. Я почему-то перепутал левую и правую часть. И изначально считывал в right вместо left, из-за этого и логика деления (и вычитания) поломалась. TheCalligrapher, благодарю! 0

Не ловится исключение

Решение