Проблема с функциями класса

Question

У меня возникли проблемы с передачей параметра класса с помощью функции main. У меня есть синтаксис для функции печати. Функция, с которой возникает проблема im, передает f2 в функцию fracmult. Если у кого-нибудь есть какие-либо объяснения или идеи для кода, которые приведут к тому, что числители и деномы будут умножены на f1 и f2 вместе, это будет отлично. Примечание: это мой первый раз работает с классами

class fraction 

{ 
    private: 
     int numerator; 
     int denom; 
     bool positive; 

    public: 
     void inputFrac(); 
     void printFrac(); 

     fraction fracMult(fraction b); 
     fraction fracDiv(fraction b); 
     fraction fracAdd(fraction b); 
     fraction fracSub(fraction b); 
}; 

void fraction::printFrac() 
{ 
    if (!positive) 
    { 
    cout << "-"; 
    } 
    cout << numerator << "/" << denom; 
} 
void fraction::inputFrac() 
{  
    cout<<"Please input the numerator "; 
    cin>>numerator; 
    cout<< "Please input the denominator "; 
    cin>>denom; 
    cout<<"Is the fraction positive? (Y or N); 
    cin>>positive; 
} 

fraction fraction::fracMult(fraction b) 
{ 

    numerator=b.numerator; 
    denom=b.denom; 
} 
fraction fraction::fracAdd(fraction& b) 
{ 
    fraction temp; 
    temp.numerator=numerator + b.numerator; 
    temp.denom=denom + b.denom; 

    return temp; 
} 
fraction fraction::fracDiv(fraction& b) 
{ 
    fraction temp; 
    temp.numerator = numerator * b.denom; 
    temp.denom = denom * b.numerator; 

    return temp; 

} 
fraction fraction::fracSub(fraction& b) 
{ 

    fraction temp; 
    temp.numerator = numerator - b.numerator; 
    temp.denom = denom - b.denom; 

    return temp; 
} 

int main(int argc, char** argv) { 

    fraction f1, f2, fresult; 

    f1.inputFrac(); //input the first fraction 
    f2.inputFrac(); //input the second fraction 
    cout<<endl; 
    f1.printFrac(); 
    cout<<endl; 
    f2.printFrac(); 
    cout<<endl; 

    cout << "The result of a * b is: "; 

    fresult = f1.fracMult(f2); // calculate a * b 
    fresult.printFrac(); // print out the result 

Answer 1

дробью :: fracMult вам необходимо иметь следующее:

fraction fraction::fracMult(const fraction& b) 
{ 
    if(positive==b.positive) positive=false; 

    numerator *=b.numerator; 
    denom *=b.denom; 

    return *this; 
} 

в качестве альтернативы вы могли бы:

fraction fraction::fracMult(const fraction&b) const 
{ 
    fraction output; 
    output.numerator = this->numerator *b.numerator; 
    output.denom = this->denom *b.denom ; 
    output.positive=false; 
    if(this->positive==b.positive) output.positive=true; 

    return output; 

} 

Answer 2

Когда вы собрали ваш код, было ли предупреждение, указывающее, что вы выходите из функции, ожидающей возврата значения, но не возвращающего значения?

fraction fraction::fracMult(fraction b) 
{ 
    numerator=b.numerator; 
    denom=b.denom; 
} 

Вы определили функцию, которая возвращает fraction, но вы не смогли вернуть значение. Таким образом, функция производит неопределенное поведение.

Исправление состоит в том, чтобы вернуть объект фракции.

fraction fraction::fracMult(fraction b) 
{ 
    numerator=b.numerator; 
    denom=b.denom; 
    return *this; 
} 

Тем не менее, это кажется противоречивым. Если вы собираетесь умножить объект на значение и вернуть новый fraction, тогда необходимо создать копию this, а затем вернуть копию. Вы не хотите изменять текущую долю.

fraction fraction::fracMult(fraction b) 
{ 
    fraction temp(*this); 
    temp.numerator=b.numerator; 
    temp.denom=b.denom; 
    return temp; 
} 

Если вы действительно хотите изменить текущую часть, а затем возвращает ссылку на *this, а не новый объект:

fraction& fraction::fracMult(fraction b) 
{ 
    numerator=b.numerator; 
    denom=b.denom; 
    return *this; 
} 

Но вот почему есть operator * и operator *=. Вам нужно решить, какую из этих операций должна выполнять ваша функция. Либо изменение текущего объекта, либо вычисление с использованием данных текущего объекта и возврат нового объекта.

Answer 3

class fraction 
{ 
private: 
    int numerator; 
    int denom; 
char choice; 

public: 
void inputFrac(); 
void printFrac(); 

fraction fracMult(fraction b); 
fraction fracDiv(fraction b); 
fraction fracAdd(fraction b); 
fraction fracSub(fraction b); 
}; 

void fraction::printFrac() 
{ 
if (choice=='y'||choice=='Y') 
{ 
cout << "-"; 
} 
cout << numerator << "/" << denom; 
} 
void fraction::inputFrac() 
{ 
cout<<"Please input the numerator "; 
cin>>numerator; 
cout<< "Please input the denominator "; 
cin>>denom; 
cout<<"Is the fraction positive? (y or n)"; 
cin>>choice; 
} 

fraction fraction::fracMult(fraction b) 
{ 

fraction temp; 
temp.numerator=numerator*b.numerator; 
temp.denom=denom*b.denom; 
return temp; 
} 
fraction fraction::fracAdd(fraction b) 
{ 
fraction temp; 
temp.numerator=numerator + b.numerator; 
temp.denom=denom + b.denom; 

return temp; 
} 
fraction fraction::fracDiv(fraction b) 
{ 
fraction temp; 
temp.numerator = numerator * b.denom; 
temp.denom = denom * b.numerator; 

return temp; 

    } 
fraction fraction::fracSub(fraction b) 
{ 

fraction temp; 
temp.numerator = numerator - b.numerator; 
temp.denom = denom - b.denom; 

return temp; 
    } 

int main(int argc, char** argv) { 

fraction f1, f2, fresult; 

f1.inputFrac(); //input the first fraction 
f2.inputFrac(); //input the second fraction 
cout<<endl; 
f1.printFrac(); 
cout<<endl; 
f2.printFrac(); 
cout<<endl; 

cout << "The result of a * b is: "; 

fresult = f1.fracMult(f2); // calculate a * b 
fresult.printFrac(); // print out the result 
}