Error: initial value of reference to non-const sein muss ein Wert

class A
{

public:

    int v;
    A * p;
    A& operator*(const A& a)
    {
        return this->v*a.v//here is a red line under this say error initial value of reference to non-const must be an value
    }
    ~A()
    {
        this;
    }
};

int main()
{
    A a;
    a.p = new A;
    delete a.p;
    return 0;

    system("pause");
    return 0;
}

überladen von operator * kann ich nicht verwenden, das zur Darstellung des Objekts an sich. Warum dies passiert ist.

  • Es ist ein binärer operator (multiplizieren) - nicht um einen pointer zu dereferenzieren.
  • Oh. ja. du hast Recht.
  • Ich danke Ihnen sehr.
InformationsquelleAutor Dongyu Li | 2015-10-07
  1. 4

    Sicher sein, es sagt, dass es sein muss eine lvalue. Sie versuchen die Rückgabe einer Referenz auf ein temporäres. Das ist schlechtes karma.

    Außer, es ist überhaupt nicht das, was Sie wollen. Die Multiplikation operator sollte auf jeden Fall ein Wert zurückgegeben, nicht eine Referenz.

    Nicht sicher, was Ihr Konstruktor aussieht, aber angenommen, es dauert eine Ganzzahl:

    A operator * (A const& other) const
{
    return A{ v * other.v};
};

    Edit:

    Und eigentlich sollte man noch einen Schritt weiter gehen:

    struct A
{
    A& operator *= (A const& other) { v *= other.v; return *this; }
    A(int i) : v(i) {}
private:
    int v;
}

A operator * (A lh, A const& rh)
{
   A res{std::move(lh)};
   res *= rh;
   return res;
}
    • Oder einfach nur return lh *= rh;. Keine std::move oder noch eine weitere lokale variable benötigt wird.
    • Hast du vergessen, eine return *this; im operator*=?
    • nicht so viel Materie in diesem Fall, aber die temporäre tatsächlich dient einem wichtigen Zweck, wenn die Objekte größer sind. Es kann der compiler durchführen die übergabe von Parametern und Rückgabewert Optimierungen. Wenn Sie den Rückgabewert parameter direkt, dass kann nicht passieren. Siehe " Soll-Geschwindigkeit? Pass by value'. Schwer zu finden, jetzt, dass Abrahams' Seite verschwunden ist, aber es ist immer noch da draußen.
    • web.archive.org/web/20140205194657/http://cpp-next.com/archive/...
    InformationsquelleAutor Edward Strange
  2. 3

    this->v*a.v ergibt eine int. Ein int kann nicht konvertiert werden, ein A&.

    Verwenden

    A operator*(const A& a) //Return a value, not a reference.
{
   A res;
   res.v = this->v*a.v;
   return res;
}

    Sollten Sie die member-Funktion a const member-Funktion zu, da es nicht ändern Sie das Objekt.

    A operator*(const A& a) const
{
   A res;
   res.v = this->v*a.v;
   return res;
}
    InformationsquelleAutor R Sahu
  3. 0

    Ergebnis this->v * a.v ist ein rvalue, eine temporäre Unbenannte Wert. Als ein temporäres, es kann nicht binden an eine non-const Referenz. Nur lvalues binden können, um non-const Referenzen. Das ist es, was die Fehlermeldung "initial value of reference to non-const sein muss, die ein l - Wert" bezieht.

    Jedoch, die Sie nicht wollen, um wieder eine const-Referenz entweder, Sie wollen wieder den Wert von Wert:

    A operator*(const A& a) { … }
 ^ remove &

    Hinweis: dies wird nicht fix deinen code komplett, wie Sie versuchen, um eine int wo Sie erklärt, um eine A, und int ist nicht implizit konvertierbar A in Ihrer aktuellen code.

    InformationsquelleAutor emlai
Schreibe einen Kommentar