malwieder so ne Übungsaufgabe. Ich komme leider nicht weiter. Die stellen an dehnen ich Probleme habe hab ich mit // versehen. Wäre dankbar für eure Hilfe.

#include <iostream>
using namespace std;

class Vektor
{
    private:
        const int dim;                             // soll laut Übung const sein
        int *komponenten;

    public:
        ~Vektor(); 
        Vektor(const Vektor& obj);   
        Vektor& operator+=(const Vektor& other);
        Vektor operator+(const Vektor& other);    
};

Vektor::Vektor(const Vektor& obj) : dim(obj.dim)  // so klappt es   
{  
    // dim = obj.dim;                             // so nicht warum?
    komponenten = new int;
    *komponenten = *(obj.komponenten);
}

Vektor::~Vektor()
{
    delete komponenten;
    komponenten = 0;
}

Vektor::Vektor& operator+=(const Vektor& other)
{
    // Komponentenweise Addition der Operanden falls Dimension gleich sonst
    // Ausgabe einer Fehlermeldung
    // ich bekomme die Implementierung nicht hin :-(
}

Vektor::Vektor operator+(const Vektor& other)
{
   // wie bei operator+=
}

zu problem 1)

dim ist als constant declariert, das bedeutet dass dim nach der initialisierung nicht mehr verändert werden darf. Warum man sie initialisiern kann liegt auf der hand; was nützt einem eine konstante deren wert man nicht festlegen kann.

Normalerweise würde man sowas schreiben:

const int konstante=3;

Das geht bei klassen und structuren nicht, deswegen gibt es etwas dass sich initialisierungsliste nennt. Nur ctors haben initialisierungslisten und in dieser liste kann man zum beispiel konstanten oder andere member initialisieren.

foo::foo() : konstante(3){}

Wie ich bereits sagte darf man eine konstante nach der initialisierung nicht mehr verändern. Die initialisierung erfolgt in der initialisierungsliste oder auch nicht wenn mans einfach weglässt. In diesem fall initialisiert der compiler die variable. Im body des ctors ist das alles schon passiert und deswegen eine zuweisung illegal weil dass ja die bereits initialisierte konstante verändern würde.

zu 2)

Du addierst einfach wie du es bei der vectoren aus der mathematik gewohnt bist und weisst die neuen werte deinem member zu und gibst eine referenz auf das eigene objekt zurück. (stichwort this-zeiger)

zu 3)

Im prinzip wie 2 nur dass du werte nicht zuweisst sondern das ergebnis der addition der componenten einem temporären object vom typ vector zuweisst und dieses zurück gibst als funktionsergebnis.

Vektor::Vektor& operator+=(const Vektor& other)
{
    if(this.dim == other.dim)
    {
       for(int i=0; i<dim; i++)
       {
           *komponenten = *komponenten + *(other.komoponenten);
           komponenten++;
       }
    }

    return *this;
}

Wäre das so OK?

Zu dem was Optimizer bereits gesagt hat, kommt noch hinzu, dass this ein Zeiger ist. Du musst deshalb '(*this).dim' oder 'this->dim' schreiben. Oder this am besten ganz weglassen.

Wernus schrieb:

Wie würde das mit + aussehen?

Die binären Non-Assignment Operatoren kann man sehr einfach über die Assignment Operatoren implementieren. Das verhindert Code Redundanz und ist deshalb leichter wartbar und weniger fehleranfällig. Sofern nicht direkt vorgeschrieben, wäre es besser, wenn du op+ ausserhalb der Klasse implementierst.

Vektor operator +(const Vektor& lhs, const Vektor& rhs)
{
    Vektor tmp(lhs);
    tmp += rhs;
    return tmp;
}

bei mir ist jetzt irgendwie die totalle Verwirrung ausgebrochen. Bin auch noch eine totaller Neuling. Naja jedenfalls war in der Aufgabenstellung nur folgendes vorgegeben:

class Vector
{
    private:
        const int dim;
        int *komponenten;
    public:
        ...
}

und folgende Deklarationen

Vektor& operator+=(const Vektor& other);
Vektor operator+(const Vektor& other);

Nun geht es nur darum die beiden überladenen Operatoren korrekt zu definieren.
Die überladenen Operatoren sollen eine Komponentenweise Additon der Operanden durchführen. Sind die Dimensionen der Operanden verschieden soll mit einer Fehlermeldung abgebrochen werden.

Könnt ihr mir vielleicht die komplette Implementierung der beiden Funktionen angeben. Ich bekomme es einfach nicht hin