使用公共函數調用私有函數

Question

在我的任務中，我需要從一個公共函數（set（int，int））調用一個私有函數（簡化）。當我嘗試編譯時，我收到一個錯誤，說重新定義集合。我不知道我做錯了什麼，或錯誤的含義。很感謝任何形式的幫助。這裏是我有：

//Fraction.h 

struct Fraction{ 
    private: 
      int numerator; 
      int denonminator; 
      void simplify(); 

    public: 
      void set(int n, int d){void simplify();}; 
      void display() const; 
}; 

//Fraction.cpp 

#include <iostream> 
using namespace std; 
#include "Fraction.h" 

void Fraction::set(int n, int d){ 
    numerator = n; 
    denonminator = d; 
    simplify(); 
} 

void Fraction::simplify(){ 
    int i; 
    for (i = denonminator * numerator; i > 1; i--){ 
      if(denonminator % i == 0 && numerator % i == 0){ 
        numerator = numerator/i; 
        denonminator = denonminator/i; 
       } 
     } 
} 

void Fraction::display() const{ 
     cout << numerator << "/" << denonminator; 
} 

Answer 1

您所定義設置兩次..更好申報一次

struct Fraction{ 
    private: 
      int numerator; 
      int denonminator; 
      void simplify(); 

    public: 
      // Dont use this-- void set(int n, int d){void simplify();}; //Defined here once 
      void set(int n, int d); //Just declare it 
      void display() const; 
}; 

//Fraction.cpp 

#include <iostream> 
using namespace std; 
#include "Fraction.h" 

void Fraction::set(int n, int d){//Defined it again 
    numerator = n; 
    denonminator = d; 
    simplify(); 
} 

void Fraction::simplify(){ 
    int i; 
    for (i = denonminator * numerator; i > 1; i--){ 
      if(denonminator % i == 0 && numerator % i == 0){ 
        numerator = numerator/i; 
        denonminator = denonminator/i; 
       } 
     } 
} 

Answer 2

Fraction::set實現兩次：你已經實現了它在頭 - 在.cpp文件中再次實現它。

你必須決定一個實現。

在看什麼都做的實現，你應該改變標題閱讀

//... 
void set(int n, int d); 
//... 

Answer 3

您定義set兩次：

struct Fraction{ 
    void set(int n, int d){void simplify();}; // <== #1 
}; 

And：

void Fraction::set(int n, int d) { // <== #2 
    numerator = n; 
    denonminator = d; 
    simplify(); 
} 

的第一個定義並沒有你想要的要多 - 這不是叫simplify，聲明它的命名simplify的本地函數，它沒有參數和返回void - 這無關與Fraction::simplify。所以只需更換它：

void set(int n, int d);