error: lvalue required as left operand of assignment

Hier mal der Code:

Die spiel.h:

class gegner : public QLabel {
   private:
        int xpos;
        int ypos;
        int richtung;

   public:
        void setx(int);
        void sety(int);
        int getx();
        int gety();
        void bewege();
        //schuss_gegner * schuss;
        gegner(QWidget * w = 0);  //Destruktor
        ~gegner();  //Konstruktor

};

class spiel : public QLabel {
...
public:
gegner *g;
...
}

Die spiel.cpp

spiel::spiel (QWidget * w = 0):QLabel(w){
...
g = new gegner(this);
..
}

//Zugriffsmethoden Setzen der Eigenschaft(set-Methoden)
void gegner::setx(int x) {
   xpos =x;
}
void gegner::sety(int y) {
   ypos =y;
}

//Zugriffmethode Abfrage der Eigenschaft(get-Methoden)
int gegner::getx() {
    return xpos;
}
int gegner::gety() {
    return ypos;
}

void spiel::timerEvent(QTimerEvent * event){
...
g->pos().x() = rand() % 1 + 0;  //Fehler!!!!!!
...
}

Hoffe auf eure Hilfe

Gruß

Aber ich vermute mal:

vector2d gegner::pos();

int vector2d::x();

x() liefert somit eine Kopie zurück. Diese Kopie ist temporär und an diese kannst du nichts zuweisen.

Grüssli

edit: 15 Sekunden zu langsam...

der code sieht aber so und so etwas chaotisch aus - wieso besitzt spiel einen pointer auf gegner?

bb

btw: Du hast Konstruktor und Destruktor verwechselt(zumindest bei den Kommentaren) ;o)

Ich denke du willst eher das hier:

g->pos().setx( rand() % 1 + 0 );

Aber auch Fehler von mir, dass die Fehlercodezeile falsch ist...
Sie sieht eigendlich so aus:

g->getx() = rand() % 1 + 0;  //Fehler!!!!!!

Hoffe das hilft vllt weiter!!

Die Lösung hat drakon schon gepostet ;o)

Zumindest fast und die Antwort von SeppJ und mir trifft hier auch zu. Man kann einer temporären Variable nichts zuweisen.

Grüssli

Nur bei einer noch nicht ganz...ich poste mal den Code...
leider bringt er schon folgende Fehlermeldung bei der get-Methode...

error: conversion from 'QObject*' to non-scalar type 'QObject' requested

spiel.h

class spiel : public QLabel {
   private:
       int xpos;
       int ypos;
       int richtung;
       int myTimerId;
       schiff * s;
       highscore * h;
       schuss * su;
       gegner *g;
       QObject * myApp;  //das hier!!
       int i;
       QString str;

   public:
      QObject getobject();
      spiel (QWidget * );  //Konstruktor
      ~spiel();  //Destruktor
      QVector<gegner *> vector_gegner;
      QVector<schuss_gegner *> vector_schuesse;
      void  keyPressEvent(QKeyEvent *event);
      void  showEvent(QShowEvent * event);
      void  timerEvent(QTimerEvent * event);
      void  hideEvent(QHideEvent * event );
      void  keyReleaseEvent( QKeyEvent *event);
};

spiel.cpp

QObject spiel::getobject() {
    return myApp;
}

main.cpp

...
s->myApp = (QObject *)&app;
...

Nur leider bekomm ich es bei der main.cpp nicht hin...

Wenn du noch etwas konkreter wirst, können wir dir vielleicht sogar helfen.

die aktuelle spiel.h

class spiel : public QLabel {
   private:
       int xpos;
       int ypos;
       int richtung;
       int myTimerId;
       schiff * s;
       highscore * h;
       schuss * su;
       gegner *g;
       QObject * myApp;
       int i;
       QString str;

   public:
      QObject *getobject();
      spiel (QWidget * );  //Konstruktor
      ~spiel();  //Destruktor
      QVector<gegner *> vector_gegner;  //dynamisches Array gegner
      QVector<schuss_gegner *> vector_schuesse;  //dynamisches Array schuesse
      void  keyPressEvent(QKeyEvent *event);
      void  showEvent(QShowEvent * event);
      void  timerEvent(QTimerEvent * event);
      void  hideEvent(QHideEvent * event );
      void  keyReleaseEvent( QKeyEvent *event);
};

spiel.cpp

QObject* spiel::getobject() {
    return myApp;
}

Der Fehler kommt bei der main.cpp:

s->myApp = (QObject *)&app;

error: 'QObject* spiel::myApp' is private

Fehlermeldung ist klar...nur wie schreib ich es um in der main.cpp,dass es passt?

Gruß

Zum Problem: Die Fehlermeldung sagt doch alles. Entweder machst du myApp public, oder du bietest eine Funktion "spiel::setMyApp(QObject* o)" an, um das Object zu setzen. Was du aber, wie bereits gesagt, gar nicht brauchst, wenn es dir nur um das QApplication geht.

Grüssli

void spiel::setMyApp(QObject* o) {
     myApp = o;
}

und:

s->setMyApp((QObject *)&app);

Warum Qt? gezwungenermaßen