Compiler-Fehler Not a statement Error

[NickJo]

Moin, ich habe einen Maus listener erstellt und eine "Leinwand", der dieser listener zugewiesen werden soll:

public void mausDetektor() {
        mausdetektor = new Mausereignisse();
        fenster.addmausdetektor;
    }

Der Compiler unterstreicht mir immer den Punkt zwischen "fenster" und "addmausdetektor" und sagt mir "Error: not a Statement". Kann mir da jemand helfen?

 

[temi]

Vermutlich heißt die Methode add() und der Parameter ist "mausdetektor". Es fehlen also Klammern.

Oder die Methode heißt addmausdetektor(), dann fehlen auch Klammern.

 

[KonradN]

Einfach einmal etwas ausführlicher:

Wenn man sich fenster.addmausdetektor ansieht, dann ist das kein Java Statement. Java Statements gibt es einige ... Zuweisungen, Methodenaufrufe, die Statements von Java wie if, switch, ...

Ein Konstrukt wie fenster.addmausdetektor könnte ein Feld bezeichnen. Aber ein Feld alleine ist kein Statement. Ein Statement macht imme etwas. Wenn es so ein Feld gäbe, dann müsste man damit etwas machen. ALso man könnte dann eine Zuweisung machen oder so....

Aber wie @temi schon richtig geschrieben hat: Vermutlich willst Du einfach nur eine Methode add aufrufen. Also fenster.add(....) - das wäre ein Statement, da ein Methodenaufrufr ein Statement ist.
Die ... müssen natürlich durch die Parameter ersetzt werden. Das ist vermutlich der mausdetektor, so dass Du ein fenster.add(mausdetektor); haben möchtest.

 

[NickJo]

Einfach einmal etwas ausführlicher:

Wenn man sich fenster.addmausdetektor ansieht, dann ist das kein Java Statement. Java Statements gibt es einige ... Zuweisungen, Methodenaufrufe, die Statements von Java wie if, switch, ...

Ein Konstrukt wie fenster.addmausdetektor könnte ein Feld bezeichnen. Aber ein Feld alleine ist kein Statement. Ein Statement macht imme etwas. Wenn es so ein Feld gäbe, dann müsste man damit etwas machen. ALso man könnte dann eine Zuweisung machen oder so....

Aber wie @temi schon richtig geschrieben hat: Vermutlich willst Du einfach nur eine Methode add aufrufen. Also fenster.add(....) - das wäre ein Statement, da ein Methodenaufrufr ein Statement ist.
Die ... müssen natürlich durch die Parameter ersetzt werden. Das ist vermutlich der mausdetektor, so dass Du ein fenster.add(mausdetektor); haben möchtest.

Schon mal danke, die Antwort hilft mir aber leider nur bedignt. Vielleicht hätte ich mein Problem ausformulieren sollen:
Ich habe als Aufgabe, einen Raumplaner zu erstellen mit verschiebbaren Möbelstücken. Ich habe bisher eine Leinwand die funktioniert und einen Mauslistener, der funktioniert.
Das Problem was ich jetzt habe ist, dass ich es nicht hinkriege, dass der Mauslistener der Leinwand zugefügt wird. Ich hatte im Unterricht glaube ich irgendwas in die Richtung aufgeschnappt, dass man das so machen kann, wie ich das oben gemacht habe (also einfach add vor den Namen der Methode, fürs Verständnis: "Mausereignisse" ist die Klasse mit dem Mauslistener und "fenster" ist ein JFrame ). Wenn das nicht geht, hier
eine neue Frage:
Wie kann ich dafür sorgen, dass die Klasse, die Mausereignisse registriert in dem gleichen Fenster arbeitet, wie die Klasse Leinwand? Oder gibt es eine Möglichkeit wo das gar nicht erst so sein muss?

 

[KonradN]

Ok, wenn Du also mit Mausereignisse eine Klasse hast, die MausListener implementiert und fenster ein JFrame ist, dann gibt es die Methode addMouseListener.

Siehe dazu die Dokumentation unter: https://docs.oracle.com/en/java/javase/17/docs/api/java.desktop/javax/swing/JFrame.html
addMouseListener ist geerbt von Component, daher findest Du es da in der Auflistung. Ein KLick drauf bringt Dich zu:

Component (Java SE 17 & JDK 17)

declaration: module: java.desktop, package: java.awt, class: Component

docs.oracle.com

Da würde der Code dann so aussehen:

public void mausDetektor() {
        mausdetektor = new Mausereignisse();
        fenster.addMouseListener(mausdetektor);
    }

Also nicht ganz wie das, wa sgeraten wurde aber vom Prinzip her das gleiche: Die methode heisst assMouseListener und will als Parameter ein MouseListener. Die Methode wird also richtig geschrieben und in Klammern kommt dann das, was du übergeben willst.

 

[NickJo]

Ahhh, jetzt habe ich aber den Fehler "Undeclared variable" beim "mausdetektor". Ich dachte die wird damit definiert als Mausereignisse?

Ok, wenn Du also mit Mausereignisse eine Klasse hast, die MausListener implementiert und fenster ein JFrame ist, dann gibt es die Methode addMouseListener.

Siehe dazu die Dokumentation unter: https://docs.oracle.com/en/java/javase/17/docs/api/java.desktop/javax/swing/JFrame.html
addMouseListener ist geerbt von Component, daher findest Du es da in der Auflistung. Ein KLick drauf bringt Dich zu:

Component (Java SE 17 & JDK 17)

declaration: module: java.desktop, package: java.awt, class: Component

docs.oracle.com

Da würde der Code dann so aussehen:

public void mausDetektor() {
        mausdetektor = new Mausereignisse();
        fenster.addMouseListener(mausdetektor);
    }

Also nicht ganz wie das, wa sgeraten wurde aber vom Prinzip her das gleiche: Die methode heisst assMouseListener und will als Parameter ein MouseListener. Die Methode wird also richtig geschrieben und in Klammern kommt dann das, was du übergeben willst.

 

[temi]

Fehler "Undeclared variable" beim "mausdetektor"

Es fehlt die Typangabe vor der Variablen "mausdetektor", wie es die Fehlermeldung ja auch sagt: "nicht deklarierte Variable".

EDIT: Alternativ, würde auch Folgendes funktionieren:

fenster.addMouseListener(new Mausereignisse());

Die mit new erzeugte Instanz der Klasse Mausereignisse wird hier direkt als Parameter verwendet, ohne sie vorher in einer zusätzlichen Variablen zwischenzuspeichern.

 

[KonradN]

Bei Variablen gibt es zwei wichtige Dinge:
a) Deklaration: Das definiert eine Variable. Dabei wird ein Typ mit angegeben. Also etwas wie Mausereignisse mausdetektor.
b) Zuweisung zu / Initialisierung einer Variable - das wäre dann etwas wie mausdetektor = new Mausereignisse();

Das kann man dann auch kombinieren zu Mausereignisse mausdetektor = new Mausereignisse();