Mouse methods

[Nico1999]

public void mouseEntered(MouseEvent e) {
    highlightPossibleMoves();

    }
private void highlightPossibleMoves(){
    for (int i = 0; i < brettGröße; i++) {
        for (int j = 0; j < brettGröße; j++) {
            for (int k = 0; k < possibleMoves(moveanzahl, brett).size(); k++) {
                if (e.getSource() == brett[i][j]) {
                    if (possibleMoves(moveanzahl, brett).get(k).location().x() == i
                            && possibleMoves(moveanzahl, brett).get(k).location().y() == j) {
                        int goalx = possibleMoves(moveanzahl, brett).get(k).goal().x();
                        int goaly = possibleMoves(moveanzahl, brett).get(k).goal().y();
                        brett[goalx][goaly].setBackground(Color.green);
                    }
                    }
                }
            }
        }

}

hi ich versuche gerade eine Gui für ein DamenSpiel zu machen und jetzt möchte ich das wenn wenn die mouseEntered, dass die möglichen Moves von der Stelle aus makiert werden(funktiert), beim mouseClickedEvent möchte ich jetzt, dass dauerhaft die felder markiert werden und wenn man auf ein markiertes Feld draufklickt der Move ausgeführt wird und wenn nicht es wieder die Felder entmarkiert werden
wenn ich allerdings jetzt das hier machee

public void mouseClicked(MouseEvent e){
highlightPossibleMoves();//gleiche Methode wie oben
}

passiert gar nichts

 

[Oneixee5]

In mouseEntered/mouseClicked setzt du e (ja es sind deine Variablennamen ...) nicht. Vermutlich wäre es besser du übergibst sowas als Parameter, statt alles irgendwie in der Klasse zu halten.

 

[Nico1999]

In mouseEntered/mouseClicked setzt du e (ja es sind deine Variablennamen ...) nicht. Vermutlich wäre es besser du übergibst sowas als Parameter, statt alles irgendwie in der Klasse zu halten.

wie meinst du das ?
muss ich ne neue Klasse für irgentwas machen?
und ja ich weiß meine Namen sind scheiße, aber ich finde schon deutlich besser als am Anfang

 

[mihe7]

Nein. Es geht um Zeile 9. Wo kommt das e her?

Wenn der Code oben so kompiliert, musst Du e als Instanzvariable in der Klasse deklariert haben. Nur sieht man nicht, wo die gesetzt wird.

Besser wäre es, e nicht als Instanzvariable zu deklarieren sondern an die Methode highlightPossibleMoves() zu übergeben (oder ggf. gleich das Ergebnis von e.getSource() an die Methode zu übergeben) - als Parameter.

 

[Nico1999]

Nein. Es geht um Zeile 9. Wo kommt das e her?

Wenn der Code oben so kompiliert, musst Du e als Instanzvariable in der Klasse deklariert haben. Nur sieht man nicht, wo die gesetzt wird.

Besser wäre es, e nicht als Instanzvariable zu deklarieren sondern an die Methode highlightPossibleMoves() zu übergeben (oder ggf. gleich das Ergebnis von e.getSource() an die Methode zu übergeben) - als Parameter.

sry dass das jetzt so undeutlich wurde ich habe das einfach in methode reinkopiert, ob besser zu verstehen zu geben, dass es das selbe ist
also quasi

public void MouseEntered(MouseEvent e){
    for (int i = 0; i < brettGröße; i++) {
        for (int j = 0; j < brettGröße; j++) {
            for (int k = 0; k < possibleMoves(moveanzahl, brett).size(); k++) {
                if (e.getSource() == brett[i][j]) {
                    if (possibleMoves(moveanzahl, brett).get(k).location().x() == i
                            && possibleMoves(moveanzahl, brett).get(k).location().y() == j) {
                        int goalx = possibleMoves(moveanzahl, brett).get(k).goal().x();
                        int goaly = possibleMoves(moveanzahl, brett).get(k).goal().y();
                        brett[goalx][goaly].setBackground(Color.green);
                    }
                    }
                }
            }
        }
}

sry das ich damit nur missverständnisse erzeugt habe .
wenn ich das selbe was jetzt in mouse entered steht was funktiert in mouse clicked eingebe funktiert es nicht mehr

 

[Oneixee5]

Ich könnte mir vorstellen, dass: if (e.getSource() == brett[i][j]) in dem Fall nicht mehr trifft. Möglicherweise ist e.getSource() die Figur und nicht das Feld auf dem Brett. Du kannst das mit dem Debugger oder Logger überprüfen.

 

[Nico1999]

Ich könnte mir vorstellen, dass: if (e.getSource() == brett[i][j]) in dem Fall nicht mehr trifft. Möglicherweise ist e.getSource() die Figur und nicht das Feld auf dem Brett. Du kannst das mit dem Debugger oder Logger überprüfen.

müsste des dann bei der andere methode nicht auch nicht funktieren ?

 

[mihe7]

Das MouseEntered ist ein Tippfehler und heißt eigentlich mouseEntered?

 

[Nico1999]

ja die methode funktiert nur wenn ich den gleich inhalt bei mouse clicked rein schreibe passiert nichts
nochmal die frage
ich schreibe den gleichen Code wie in mouseEntered rein wo es funktiert
in mouseClicked rein da funktiert es nicht warum ?

 

[mihe7]

Das kann ich Dir anhand des Schnipsels auch nicht sagen. Wie @Oneixee5 geschrieben hat: geh mal mit dem Debugger schrittweise durch oder gib Logmeldungen aus.

 

[Neumi5694]

Click ist auch nicht immer der passende Event, meistens reagiert man auf MouseReleased. Click ist in Java nämlich äußerst sensibel, wenn sich zwischen ButtonDown und ButtonUp da der Mauszeiger auch nur einen Pixel bewegt oder hält man den Button zu lange gedrückt, dann ist es vorbei mit dem Klick.

Es kann auch sein, dass sich bei dir zwei Events gegenseitig bekriegen, deshalb ist der Vorschlag mit dem Log schon der beste bisher.
Schau nach, ob deine Methode überhaupt aktiviert wird.