Калькулятор\классы\дроби

[Alex11223]

Не могу понять в чем ошибка в калькуляторе для дробей.

int main()
{
	fraction frac1, frac2, fracsumm;
	char ch='a';
	do
	{
		
		switch(fracsumm.calc(frac1, frac2)) 
		{
		case '+': fracsumm.fadd(frac1, frac2); break;
		case '-': fracsumm.fsub(frac1, frac2); break;
		case '*': fracsumm.fmul(frac1, frac2); break;
		case '/': fracsumm.fdiv(frac1, frac2); break;
		default: continue;
		}
		fracsumm.display();
		cout << "Continue?(y/n)"; cin >> ch;
	}while(ch!='n');
	return 0;
}

char calc(fraction frac1, fraction frac2)const
	{
		char dummychar;
		char sign;
		cout << "\nEnter: " ;   cin >> frac1.numerator >> dummychar >> frac1.denumerator >> sign
			>> frac2.numerator >> dummychar >> frac2.denumerator;
		return sign;
	}

Конструктор класса fraction(): numerator(0), denumerator(1) {}

Почему-то значения полей всех объектов после ввода в switch (вроде же то, что там не противоречит синтаксису?) остаются теми же, что были инициализированы конструктором.

Весь код http://pastebin.com/7tAzXvar

[blacktener]

Лафоре? // ех....

[Alex11223]

Да
Еще раз дебаггером просмотрел, теперь увидел, что frac1 и frac2 почему-то теряют свои значения после завершения функции (calc) O_o

[netrino]

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

Да
Еще раз дебаггером просмотрел, теперь увидел, что frac1 и frac2 почему-то теряют свои значения после завершения функции (calc) O_o

Так ведь ясно дело, вы же в локальные переменные данные вводите. Естественно, по завершении функции, они разрушаются)
Чтобы было, как задумано - нужно передавать по ссылке

...calc(fraction& frac1, fraction& frac2)...

А вообще, ввод должен бы быть в отдельной функции, а никак не в calc.

...

И, честно говоря, довольно странная реализация класса fraction.

[Сtrl]

denumerator

Хы-хы, denominator же.

[Alex11223]

Ага, уже сам понял про локальные переменные, спс )

А вообще, ввод должен бы быть в отдельной функции, а никак не в calc.

Так это вроде и есть функция для ввода 2 дробей и действия между ними, название только мб немного некорретное. Или что вы имели в виду?

И, честно говоря, довольно странная реализация класса fraction.

А как лучше можно было?)

Хы-хы, denominator же.

Спсиабо, ибо как усатвонили бртиаскние уечные, члеовек чтиает тлоько преыве и псоелдние бквуы слов.)

[netrino]

Я к тому, что fraction по поведению должен напоминать примитивные типы, тем более, что функционал его очень с ними схож. У вас он используется как-то непонятно и совсем не логично. Функция ввода, если на то пошло, должна либо изменять существующий объект, либо быть статичной и принимать как аргумент некий объект и производить в него ввод.

class fraction {
...
    static void read(fraction& frac) {
        std::cin >> frac.numer >> frac.denom;
    }
 // или лучше
    static fraction read() {
        fraction frac;
        std::cin >> frac.numer >> frac.denom;
        return frac;
    }
// ну и самый логичный вариант:
     std::istream& operator>> (std::istream& in, fraction& frac) {
         return in >> frac.numer >> frac.denom;
     }
};

Кроме того, поведение должно полностью совпадать с ожидаемым (то есть с тем, как себя ведут примитивные числовые типы, такие как int и double).
То есть должно быть можно использовать его так:

   fraction frac_1(1, 2), frac_2(3, 4);
   fraction frac_3 = fraction::add(frac_1, frac_2); // исходные данные не должны меняться
// или
   fraction frac_3 = frac_1.add(frac_2); // исходные данные не меняются
// ну и на крайний случай
   frac_1.add(frac_2); // frac_1 меняется. это лучше, чем так как у вас, но тем не менее плохо.
// наиболее понятный и логичный способ:
   fraction frac_3 = frac_1 + frac_2; // исходные данные не меняются.
// для вывода тоже неплохо бы перегрузить оператор << и выводить ответ так:
   std::cout << frac_1 << frac_2 << frac_3 << std::endl;

Ну и так далее, в результате могло бы получится как-нибудь так:

#include <iostream>

class fraction {
public:
  fraction(int num = 0, int den = 1) {
    int g = gcd(num, den);
    numer = num / g;
    denom = den / g;
  }

  //
  // unary operators
  //

  fraction operator- () {
    return fraction(-numer, denom);
  }

  fraction operator+ () {
    return *this;
  }

  //
  // binary operators
  //

  fraction operator+ (const fraction& other) {
    return fraction(numer * other.denom + other.numer * denom, 
		    denom * other.denom);
  }

  fraction operator- (const fraction& other) {
    return fraction(numer * other.denom - other.numer * denom,
		    denom * other.denom);
  }

  fraction operator* (const fraction& other) {
    return fraction(numer * other.numer, denom * other.denom);
  }

  fraction operator/ (const fraction& other) {
    return fraction(numer * other.denom, denom * other.numer);
  }

  //
  // assign operators
  //

  fraction& operator+= (const fraction& other) {
    return *this = *this + other;
  }

  fraction& operator-= (const fraction& other) {
    return *this = *this - other;
  }

  fraction& operator*= (const fraction& other) {
    return *this = *this * other;
  }

  fraction& operator/= (const fraction& other) {
    return *this = *this / other;
  }

  //
  // I/O operators
  //

  friend std::ostream& operator<< (std::ostream& out, const fraction& f) {
    return out << f.numer << '/' << f.denom;
  }

  friend std::istream& operator>> (std::istream& in, fraction& f) {
    char delimiter;
    return in >> f.numer >> delimiter >> f.denom;
  }

private:
  int gcd(int x, int y) {
    return y == 0 ? x : gcd(y, x % y);
  }

private:
  int numer;
  int denom;
};


int main() {
  using namespace std;

  fraction frac1, frac2, result;
  char ch = 'a';
  
  do {

    cin >> frac1 >> frac2;
    cin.ignore();
    
    switch (cin.get()) {
    case '+': result = frac1 + frac2; break;
    case '-': result = frac1 - frac2; break;
    case '*': result = frac1 * frac2; break;
    case '/': result = frac1 / frac2; break;
    default : continue;
    }

    cout << result << endl;
    cout << "Continue?(y/n)";
    cin  >> ch;

  } while(ch != 'n');
}

[Alex11223]

Ух, сколько новых слов. Видимо стоит вернуться к этому после повышения моего уровня знаний