在分數方案中使用結構有問題

Question

這是我的程序。它應該佔用2個分數，並在建模結構時將它們加在一起。我遇到了一些麻煩。我收到了一些錯誤消息。複製這段代碼，看看你能否爲我正確調試它並解釋你做了什麼。我很困難！非常感謝你！

#include <cstdlib> 
#include <iostream> 
#include <math.h> 

/* 
Name: Fraction 
Author: Grant Birkinbine 
Date: 13/10/14 17:33 
Description: Takes two fractions and outputs them in different ways 
*/ 

using namespace std; 

//Struct 

struct fraction{ 
    int numer, denom ; 
}; 

//Function to print fractions 

void displayfractionadd (fraction a) { 
cout << "Addition: " << add.numer << "/" << add.denom << endl; 

} 

//Function to get fraction info 

fraction newfraction() { 
    fraction a1; 
    cout << "Enter numerator: " ; 
    cin >> a1.numer ; 
    cout << "Enter denomerator: " ; 
    cin >> a1.denom ; 
    return a1; 
} 

//Function to add fractions 

fraction addition(fraction a1, fraction a2) { 
fraction add; 
add.denom = (a1.denom * a2.denom) ; 
add.numer = a1.numer * a2.denom + a1.denom * a2.numer; 
return add; 
} 

//Main  

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

cout << "Fraction 1 = " ; 
cout << endl; 
fraction fraction1 = newfraction(); 
fraction fraction2 = newfraction(); 
cout << endl; 
cout << "Solutions: " ; 
fraction newadd = addition(fraction a1, fraction a2) ; 
cout << endl; 
displayfractionadd (fraction a); 

system("PAUSE"); 
return EXIT_SUCCESS; 
} 

Answer 1

這些線路不調用函數：

fraction newadd = addition(fraction a1, fraction a2) ; 
//... 
displayfractionadd (fraction a); 

這些行聲明的功能，而不是給他們打電話。要調用一個函數，只要提供的參數：

fraction newadd = addition(a1, a2); 
//... 
displayfractionadd (a); 

其次，a，a1和a2沒有在任何聲明。你的意思是fraction1fraction2和newadd？

fraction newadd = addition(fraction1, fraction2); 
//... 
displayfractionadd (newadd); 

最後，除非你有這樣做的一個具體的理由，通過reference或const reference通過fraction，而不是價值。例如：

fraction addition(const fraction& a1, const fraction& a2) 
{ 
    //... body of function here 
} 

由於兩個a1和a2不在addition函數內改變，所述參數應該是常數引用來指示（和執行），該a1和a2不改變。另外，按引用傳遞不會引發無關的副本。