Двойная перегрузка операторов istream и ostream в C++

У меня есть класс Fraction, который позволяет вводить дробь в виде c/d. Я могу нормально выводить и вводить дроби, но когда я изменяю их с помощью своей пользовательской функции, показанной ниже, она вообще ничего не делает.

У меня есть следующие перегруженные операторы >> и ‹‹:

  ostream& operator<<(ostream &out, const Fraction &f)
{
    char x = '/';
    out << f.num;
    out << x;
    out << f.den;
    return out;
}

istream& operator>>(istream &in, Fraction &r)
{
    //in >> r;
    int whole = 0, num, den = 1;
    char next;
    in >> num;
    next = in.peek();
    if(next == '+'){
        in.get();
        whole = num;
        in >> num;
        next = in.peek();
    }
    if(next == '/'){
        in.get();
        in >> den;
    }
    if(whole != 0){
        num += (whole * den);
    }
    if(den == 0){
        den = 1;
    }
    r.num = num;
    r.den = den;

    return in;
}

Кроме того, у меня есть функция, которая делает две дроби так, чтобы они имели один и тот же общий знаменатель:

void setEqualDen(Fraction a, Fraction b){
    int tempa = a.den;
    int tempb = b.den;
    a.den *= tempb;
    b.den *= tempa;
    a.num *= tempb;
    b.num *= tempa;
}

Затем я пытаюсь вывести результат в main следующим образом:

setEqualDen(Fa, Fb);
    cout << "The fractions are " << Fa << " , " << Fb <<   
             endl;

Это не работает. Есть ли необходимый шаг, такой как двойная перегрузка операторов ‹‹ и >> в C++, или мой синтаксис просто что-то упускает?


c++ class overloading ostream istream
person JCoder    schedule 24.02.2015    source источник
comment
Что вы подразумеваете под не работает? Если вы имеете в виду, что результат не соответствует вашим ожиданиям, то это потому, что setEqualDen копирует свои аргументы, а не берет их по ссылке. Если проблема не в этом, объясните, что вы подразумеваете под двойной перегрузкой, и опубликуйте сообщения об ошибках, ожидаемый результат и т. д.   -  person Praetorian    schedule 24.02.2015
comment
Ваша функция ввода не работает с этим текстом: 1+/25.   -  person Thomas Matthews    schedule 24.02.2015
comment
Вместо того, чтобы предполагать количество пробелов, вы должны пропустить его.   -  person Thomas Matthews    schedule 24.02.2015
comment
Что вы имеете в виду под не работает? Если вы имеете в виду, что результат не такой, как вы ожидаете, то это потому, что setEqualDen делает копии своих аргументов, а не берет их по ссылке. Если это не проблема, то объясните, что вы подразумеваете под двойной перегрузкой, и опубликуйте сообщения об ошибках, ожидаемый результат и т. д. — Praetorian///////////////////Я писал выше. Проблема в том, что ничего не происходит. Он полностью игнорирует функцию.   -  person JCoder    schedule 24.02.2015

Ответы (2)


arrow_upward
2
arrow_downward

Вам нужен & в определении функции, потому что вам нужно передать по ссылке, так как вы изменяете свой файл `Fractions.

void setEqualDen(Fraction &a, Fraction &b){
    int tempa = a.den;
    int tempb = b.den;
    a.den *= tempb;
    b.den *= tempa;
    a.num *= tempb;
    b.num *= tempa;
}
person TriHard8    schedule 24.02.2015
comment
Это, вероятно, решит проблему OP, но нет причин для setEqualDen быть функцией-членом, поскольку она не обращается к экземпляру, на котором она вызывается. - person Praetorian; 24.02.2015
comment
@Praetorian - да, это хорошо, что в этом нет необходимости. Каковы предпочтения здесь, поскольку функция действительна только для этого конкретного класса? Я бы сказал, сделайте его членом, чтобы вы могли перемещать класс и не беспокоиться о воссоздании функции в другой программе. - person TriHard8; 24.02.2015
comment
@TriHard8 TriHard8 Ошибка, вероятно, заключается в том, что он пытается вызвать ее без экземпляра Fraction. Это должна быть свободная функция в том же пространстве имен, что и Fraction. Бесплатные функции Google улучшают инкапсуляцию - person Praetorian; 24.02.2015
comment
@Praetorian, согласен. Вы должны получить кредит за ответ. Я буду Google, что для моего собственного обучения. - person TriHard8; 24.02.2015
comment
Я бы поставил вопрос, на который ответил Преторианец, но он пока не работает. Потоковые функции не позволяют работать другим функциям. - person JCoder; 24.02.2015
comment
@JCoder - Что ты имеешь в виду? Вы сказали, что ваша проблема связана с setEqualDen, а не с потоковыми функциями. Вам нужно отредактировать исходный пост и четко указать, о чем вы спрашиваете, а также добавить больше кода. Не видя кода вашего класса, трудно дать вам окончательный ответ. - person TriHard8; 25.02.2015

arrow_upward
0
arrow_downward

Вам нужно проверить наличие ошибок ввода и пропустить пробелы. Я предлагаю использовать временное для содержания первой цифры, так как это может быть либо целое число, либо числитель. Дифференциация должна происходить после обнаружения «/».

std::istream& operator>>(std::istream& inp, Fraction& f)
{
  int temp = 0;
  f.num = 0;
  f.den = 1;
  inp >> temp;  // Could be whole number or numerator.
  if (inp)
  {
    int whole = temp;
    int numerator = 0;
    inp >> numerator;
    if (!inp)
    {
      // Let's assume that the input failed because of the '/' symbol.
      numerator = temp;
      whole = 0;
      inp.clear();
    }
    else
    {
      inp.ignore(1000, '/'); // Skip over the '/'
    }
    int denominator = 1;
    inp >> denominator;
    if (denominator == 0) denominator = 1;
    numerator += whole * denominator;
    f.num = numerator;
    f.den = denominator;
  }
  return inp;
}
person Thomas Matthews    schedule 24.02.2015