MouseListener für viele Objekte erstellen

[lorddtm]

Hi, ich programmiere etwas in Java für die Schule. Mein Ziel ist es aus 100 Objekten der Klasse Feld durch Mausklick auf einen jeweiligen Wert zuzugreifen. Ich will aber nicht für alle 100 Objekte jedes Mal einen eigenen MouseListener mit allem drum und dran schreiben. Deshalb hab ich versucht das Problem so zu Lösen: (Felder[] enthält die Objekte FeldA1 - FeldF10, von denen ich die Werte haben will und Maus[] die Mauslistener m1-m100, istbesetzt() gibt den Wert zurück)
[Java] public void ermittleWert()
{
for(i=0;i<99;i++)
{ Maus = new MouseAdapter(){
public void mouseClicked(MouseEvent mEvent) {
System.out.println(Felder.istbesetzt());
}
};
Felder.addMouseListener(Maus);

}

}[/Java]

Das Problem is jetzt, dass egal wo ich hinklicke immer der Wert des letzen Felds zurückgegeben wird, vermutlich weil [Java]Felder.istbesetzt()[/Java] ja erst aufgerufen wird, wenn die Maus geklickt wird, und i dann schon 99 ist. Ich hab leider keine Idee wie ich das verhindern kann, kann mir jemand helfen? ... Schonmal vielen Dank, euer Chris

 

[vanny]

Wenn du´s schick machen möchtest, dann benötigst du eigentlich genau einen MouseListener.
Dieser wir der Componente zugewiesen, auf der die Felder optisch dargestellt werden.
In der mouseClicked(); holst du dir die Position der Maus und kannst damit die Position im Raster ermitteln und somit direkt das passende Feld ansprechen.
Das setzt dann natürlich vorraus, dass du deine Felder irgendwie sinnig ordnest, in deinem Fall zBsp. ein Array ala Feld[][] felder = new Feld[x][y]; oder so.
Dann muss dein MouseListener nur noch felder[mouseX/feldGroesse][mouseY/feldGroesse].getWert(); und gut is.

Und Methodennamen wie istBesetzt(); sollten dem Namen nach eher einen Boolean zurückgeben.

Gruß Vanny

[EDIT]for(i=0;i<99;i++)
das ist auch nicht die schönste Art, die 99 ist dann eine sogenannte Magic Number, die du eher in eine Variable packst, die aussagt, was dieser Wert darstellt. Naja bei meiner Variante sähe das eh anders aus.[/EDIT]

 

[lorddtm]

Danke , hört sich gut an, werd ich so machen. ... istBesetzt() gibt auch einen Boolean zurück ... wenn ich anstatt 99 lieber Felder.lenght, also die Länge des Arrays nehm?
Ach und sei gnädig mit mir, wenns nicht so schön ausschaut, ich mach das zu ersten mal
Gruß, Chris

 

[vanny]

Danke ... wenn ich anstatt 99 lieber Felder.lenght, also die Länge des Arrays nehm?

Haargenau so :toll:

...
Ach und sei gnädig mit mir, wenns nicht so schön ausschaut, ich mach das zu ersten mal
Gruß, Chris

Hat mit gnädig nüx zu tun, ich wollte dir nur gleich n paar nützliche Tipps mitgeben

Gruß Vanny

 

[lorddtm]

Hat mit gnädig nüx zu tun, ich wollte dir nur gleich n paar nützliche Tipps mitgeben

Immer Gern ... Ich habs übrigens hinbekommen, nochmal vielen Dank :toll:

Gruß, Chris

 

[tagedieb]

So ging's auch...

MouseListener ml = new MouseAdapter(){
  public void mouseClicked(MouseEvent mEvent) { 
    System.out.println(((Felder) mEvent).istbesetzt()); 
  } 
};

for(i=0;i<99;i++) { 
  Felder[i].addMouseListener(ml);
}

 

[mla.rue]

Ich würde den Inhalt der einzelnen Textfelder nicht über die Position der Maus abfragen

Object obj = evt.getSource();
//entweder oder
JComponent source = (JComponent)(evt.getSource());

JTextField txtf = (JTextField)source; //oder halt obj statt source
txtf.getText();

 

[vanny]

Ich würde den Inhalt der einzelnen Textfelder nicht über die Position der Maus abfragen

Wo hast du denn was von TextFeldern gelesen? o0

 

[mla.rue]

gute Frage, frag ich mich gerade auch. Schon das zweitemal heute wo ich mal falsch gelesen habe, das WE hallt wohl noch nach

 

[tagedieb]

Ob 'Textfeld' oder 'Felder' ist ja egal. Beiden wird ein MouseListener zugewiesen ;-)

 

[vanny]

Ob 'Textfeld' oder 'Felder' ist ja egal. Beiden wird ein MouseListener zugewiesen ;-)

public class Feld {
	
	private int wert;
	private boolean besetzt;
	
	
	public Feld(int wert, boolean besetzt){
		setWert(wert);
		setBesetzt(besetzt);
	}
	
	public int getWert() {
		return wert;
	}
	public void setWert(int wert) {
		this.wert = wert;
	}
	public boolean isBesetzt() {
		return besetzt;
	}
	public void setBesetzt(boolean besetzt) {
		this.besetzt = besetzt;
	}

}

so egal ist es wohl nicht.

[EDIT]Vielleicht mal noch die Frage an den TO:
Ist Feld bei dir eine JComponent(dann Klar pack einfach jeweils einen MouseListener drauf) oder, wie ich es meinte eine Wrapperklasse für deine Datenlogik?[/EDIT]

 

[tagedieb]

public void ermittleWert()
    {        
       for(i=0;i<99;i++) 
       { Maus[i] = new MouseAdapter(){
               public void mouseClicked(MouseEvent mEvent) {
               System.out.println(Felder[i].istbesetzt());       
              }
            };
         Felder[i].addMouseListener(Maus[i]); 
         
        }
        
    }

Aha und wo kommt diese Klasse her??
Im Originalbeitrag schein die Klassen im Array 'Felder' auf jeden Fall die Methode addMouseListener implementiert zu haben.

 

[vanny]

Ich will jetzt nicht mit dir streiten tagedieb, sondern eigentlich dem TO helfen eine schöne Lösung für sein Anliegen zu finden. Der Grundgedanke meines ersten Posts war eben auch gleich den Weg für eine saubere Trennung zwischen Programmlogik(scheinbar Minesweeper oder so) und der grafischen Darstellung zu bereiten ohne Ihm gleich alles haarklein vorzukauen.
IMHO ist es nicht erstrebenswert, seine JComponents mit irgendwelcher Programmlogik zu füttern. Wie er es letztendlich umsetzt ist freilich dem TO überlassen.

Gruß Vanny

 

[tagedieb]

Ok..

Ich bin nur davon ausgegangen, dass 'Feld(er)' eine 'Component' ist, welche am Bildschirm dargestellt wird und sich selber rendert. Alternativ könnte man den auch durch ein JLabel ersetzen, falls nur ein Icon angezeigt werden soll.

Ich gebe dir natürlich recht, dass man die Daten (Model) vom GUI trennen sollte.

Ursprünglich ging es ja darum bloss mit einem einzigen MouseListener zu arbeiten.

Gruss Tagedieb

 

[lorddtm]

Also, falls es noch jemanden interessiert, die Klasse Feld ist ein JPanel und ich mache ein Schiffeversenken. Also für jedes Kästchen ein eigenens Panel, dass dann Boolean-Werte wie besetzt, getroffen usw annehmen kann. Ich habs so gemacht wie Vanny mirs am Anfang empfohlen hat , funktioniert auch super. Was mit einer Wrapperklasse gemeint ist, erschließt sich mir nicht so ganz. Auf jeden Fall Danke für die vielen Beiträge

 

[vanny]

... Was mit einer Wrapperklasse gemeint ist, erschließt sich mir nicht so ganz....

Eine Wrapperklasse ist nichts weiter, als eine eigene Klasse, die Werte und Funktionen bündelt.
Dein Feld(auch wenn es von JPanel erbt) ist auch eine Wrapperklasse, da sie ja nicht nur für das Zeichnen des JPanels sondern auch für die Logik (istBesetzt) zuständig ist.
(Google sagt da bestimmt was anderes aber so wars von mir gemeint)
Ich wollt dich mit dem Ausdruck nicht verwirren

Gruß Vanny

 

[lorddtm]

Aso, hört sich vernünftig an Danke! .. Nene, passt schon