연산자 오버로딩 C++ 조언

Question

나는 연산자 오버로딩의 주위에 내 머리를 얻기 위해 고군분투하고있다. 이 경우에는 + 연산자와 제가 시도한 예제를 통해 도움을 얻을 수 있습니다.

나는 '클래스 매트릭스'나는이 두 행렬을 추가하는 방법이 문제를 해결하는 방법에 대한 확신이 오전의 "invaild 사용이 함께 객체라는 오류는 무엇입니까?

Matrix Matrix::operator+(const Matrix& rhs){ 
return Matrix(Matrix + rhs.Matrix()); 
} 


    Matrix::Matrix(int MM, int NN){ 
        M = MM; 
        N = NN; 
        data = new double[M * N]; 
        for (int i =0; i < M; i++) 
        { 
         for (int j = 0; j < N; j++) 
         { 
          data[i* N+j] = (double) 1000 + i*N+j; 
        //  cout << data[i*N+j] <<"\t"; 
         } 
         //cout <<"\n"; 
        } 

     cout << "Matrix Constructor... (code to be implemented here!!!)\n";} 

감사

Answer 1

JPM의 대답은보기 위하여 매우 중요하다. 이 문제를 해결하면 내 것을 볼 수 있습니다.

본질적 연산자 과부하는 다른 기능 다르지 않다.

그래서의 경우 :

당신이 정말로이 말하는 무슨 일을하는지

Matrix Matrix::operator+(const Matrix& rhs) 

: 현재 행렬에 우를 추가하고, 새로운 행렬을 반환합니다. 오버로드로 인해 현재 매트릭스가 변경되지 않아야합니다. 자신을 도움말 및 상수를 사용

Matrix Matrix::operator+(const Matrix& rhs) const 

그와 같은 매트릭스 또한 먼저 행렬은 크기가 동일한 경우에 당신이 그들을 모두 "세포"를 통해 함께 다음 루프를 추가하고 추가 할 수 있도록 확인하고 작성해야 그것의 매트릭스. 이를 위해, 내가 두 번째 생성자가 필요합니다 같은데요, 뭔가 같은 :

Matrix::Matrix(int MM, int NN, double values[]) 
{ 
    M = MM;//TODO: change to width 
    N = NN;//TODO: change to height 
    data = new double[M*N]; 
    for(int i < 0; i < M; i++) 
     for(int j < 0; j < N; j++) 
      data[i * N+j] = values[i * N+j]; 
} 

Answer 2

방법은
1. 매우 불법처럼
2. 우 는 생성자를 호출Matrix
3. 입니다, Matrix은 식별자가 아닙니다
4. 식별자를 정리하면 *this + rhs은 this->operator+(rhs)과 같습니다. 여기에서 한 것은 무한 재귀를 만드는 것임을 분명히해야합니다.

Answer 3

return Matrix(Matrix + rhs.Matrix()); 
       ^^^^^^  ^^^^^^ 

당신은 표현이 있어야한다 유형 이름 (Matrix)를 사용하는 - 즉, 컴파일러는 무엇 에 대해 불평. 또한 무효 인 기존 객체에 대해 생성자를 호출하려고합니다. 또한 자신을 호출하는 방식으로 operator+을 구현하려고 시도하는 것 같습니다. 어떻게 든 컴파일하면 무한 재귀로 인해 스택 오버플로가 발생합니다. 이것은 정확한 복사 생성자가 당신을 필요로 않습니다

Matrix & Matrix::operator+=(Matrix const & rhs) { 
    // Perform addition here 
    for (int i = 0; i < N*M; ++i) { 
     data[i] += rhs.data[i]; 
    } 
    return *this; 
} 

// This can be a non-member function 
// Pass "lhs" by value to get a copy, then modify and return that. 
Matrix operator+(Matrix lhs, Matrix const & lhs) { 
    return lhs += rhs; 
} 

// Or if you really want a member function for some reason 
Matrix Matrix::operator+(Matix const & rhs) const { 
    return Matrix(*this) += rhs; 
} 

:

아마도 추가를 구현하는 가장 쉬운 방법은 기존에 하나의 매트릭스를 추가 operator+=을 구현하고 그 측면에서 operator+을 구현하는 것입니다 - 당신이 소멸자에서 할당을 해제 아마 생성자에서 메모리를 직접 할당하고있어 이후, 당신 은 그렇지 않으면 당신은 이중 삭제 후를 얻을 수 있습니다 올바르게합니다 (Rule of Three 당) 새로운 메모리를 할당하기 위해 복사 생성자를 구현해야합니다 행렬 복사.