c++ Compiler meckert 'multiple definition of `AlgoKS::operator+..."

[system]

Disclaimer: Dieser Thread wurde aus dem alten Forum importiert. Daher werden eventuell nicht alle Formatierungen richtig angezeigt. Der ursprüngliche Thread beginnt im zweiten Post dieses Threads.

[J_Suessmuth]

c++ Compiler meckert 'multiple definition of `AlgoKS::operator+…"
Ich bekam eine Mail mit folgendem Problem:

Hallo,

wenn ich den Operator+ als nicht-member-funktion im Header implementiere (siehe Code), bekomme ich von meinem Compiler den Fehler

/cygdrive/D/Lehre/algo3/SS09/exrechner/R00/A01/Problem/Complex.hpp:29: multiple
definition of `AlgoKS::operator+(AlgoKS::Complex&, AlgoKS::Complex&)'
./Complex.o:/cygdrive/D/Lehre/algo3/SS09/exrechner/R00/A01/Problem/Complex.hpp:2
9: first defined here
collect2: ld returned 1 exit status
make: *** [complex] Error 1

Hier der Quelltext, welcher das Problem auslöst:

#ifndef COMPLEX_HPP
#define COMPLEX_HPP

namespace AlgoKS {

  class Complex {

	friend Complex operator+(Complex& a, Complex& b);

      public:
	Complex(float r, float i);
	~Complex();
	void inc();
	void setReal(float r);
	void setImag(float r);
	float getReal();
	float getImag();
	void add(const Complex c);
	void print() const;

	void operator+=(const Complex& c) {
		add(c);
	}
      private:
	float m_fReal;
	float m_fImag;
    };

    Complex operator+(Complex& a, Complex& b) {
    	Complex ret(a.getReal() + b.getReal(), a.getImag() + b.getImag());
	return ret;
    }
}



#endif

Das Problem ist, dass der g++ die Dateien ‚main.cpp‘ und ‚Complex.cpp‘ unabhängig von einander compiliert.
d.h. beim Compilieren von ‚main.cpp‘ erstellt er Maschinencode für die Funktion ‚operator+‘
aber auch beim Compilieren von ‚complex.cpp‘ erstellt er Maschinencode für dieselbe Funktion.
Will er nun die beiden .o files (main.o und compex.o) zu einem executable zusammenfügen (also linken)
stellt er fest, dass er zweimal Maschinencode für die Funktion hat und meckert.

Es gibt nun zwei mögliche Lösungen für dieses Problem:

• Entweder man hällt sich strikt an die Trennung von Deklaration und Implementierung in Header und .cpp file, d.h. man verschiebt die Implementierung in die Datei complex.cpp und lässt nur eine Deklaration im Header
• Oder man ‚inlined‘ die Funktion im Header. Damit wird kein Maschinencode für die Funktion erstellt sondern die Funktion wird immer direkt an der Stelle in den Code (Und somit auch in den Maschinencode) eingefügt, wo der operator+ benötigt wird.

Also, schreib deinen operator+ (und auch operator<<) am besten wie folgt:

    inline Complex operator+(Complex& a, Complex& b) {
    	Complex ret(a.getReal() + b.getReal(), a.getImag() + b.getImag());
		return ret;
    }

Gruss, Jochen