연산자를 오버로드하는 방법 =

Question

할당 연산자를 멤버 함수로 오버로드하여 문자열을 인수로 사용하여 해당 값을 A, 현재 개체에 할당하려고합니다. 아래의 의견에 오류를 게시했습니다.

누군가 내가 뭘 잘못하고 있다고 말할 수 있습니까? 나는 그것이 매개 변수와 관련이 있으며, 아마도 정의 안에있는 코드라고 생각한다.

내가 올바르게 선언 있는지 확실하지 않습니다,하지만 나는 이런 식으로 선언 :

WORD operator=(const string& other); 

그리고 나는 이런 식으로 정의 : 도움이된다면 여기

WORD WORD::operator=(const string& other) //<---not sure if I did the parameters Correctly 
{ 
(*this) = other; 
return (*this); 
} 

전체 파일입니다 :

#include <iostream> 

using namespace std; 

#pragma once 

class alpha_numeric //node 
{ 
    public: 
char symbol; //data in node 
alpha_numeric *next;//points to next node 
}; 

class WORD 
{ 
    public: 
     WORD() : front(0) {length=0;}; //front of list initially set to Null 
     WORD(const WORD& other); 
     bool IsEmpty(); 
     int Length(); 
     void Insert(WORD bword, int position); 
     WORD operator=(const string& other); 

    private: 
     alpha_numeric *front; //points to the front node of a list 
     int length; 
}; 

WORD WORD::operator=(const string& other) //<---not sure if I did the parameters Correctly 
{ 
     (*this) = other; 
     return (*this); 
} 

Answer 1

확인 2 일 :

먼저 복사 생성자의 정의가 누락되어 컴파일되지 않습니다. 당신의 할당 연산자가 올바른지,

WORD(const WORD& other) 
: length(other.length) 
{ 
    // Construct me ! 
} 

둘째 있지만, 모든 제어 경로에 재귀 : 클래스 (도시 부분적으로 만 구현) 내부에이를보십시오. 예 : 그것은 무기한으로 호출합니다. 당신은 아마 (다시, 부분적인 구현이 표시) 메서드 내 구성원을 할당 할 :

WORD WORD::operator=(const string& other) 
{ 
    // Only copy the length, the rest is to be filled 
    this.length = other.length; 
    return (*this); 
} 

마지막으로 다른 사람이 지적한대로, 당신이 당신의 실행 파일 내부에 동일한 기호 여러 정의가 있습니다. 헤더를 단 한 번만 포함하도록 (#pragma 한 번 처리해야 함) 헤더 파일의 모든 구현 세부 사항을 소스 파일로 이동해야합니다. 예 : WORD WORD :: operator = (const string & other)의 정의를 CPP 파일로 이동하십시오.

Answer 2

오류 메시지는 링커에서 가져온 것입니다. 그것은 동일한 함수의 다중 정의가 발견되었음을 알려줍니다. 이는 여러 소스 파일에 포함 된 헤더 파일에서 함수를 정의했기 때문에 여러 함수 정의로 끝났기 때문입니다.