Methode mouseClicked() aufrufen

[Inventar]

Hallo,

ich möchte gerne die Methode mouseClicked() aufrufen, wenn eine Bedingung erfüllt ist innerhalb einer Methode. Nur, wie kriege ich das realisiert? mouseClicked realisiert in einer eigenen Methode funktioniert so weit einwandfrei.

public void run()
     while (up){
          if (Bedingung == true){
             public void mouseClicked(mouseEvent e){
                  ..................
            }
         }
      }
}

 

[wayne]

ich glaube, der grund, warum bisher niemand auf deine post geantwortet hat, ist der, daß keiner blickt, was du wissen willst. vielleicht erklärst du es etwas ausführlicher. der code, den du hier reingepostet hast, ist ja weder lauffähig noch sinnvoll, denn die methode mouseClicked(...) ist ja bekanntlich bestandteil eines MouseListeners. und letzterer ist in deinem code nicht enthalten. davon abgesehen, wenn du die methode mouseClicked(...) in deiner schleife aufrufen willst, warum machst du's denn dann nicht einfach. in dieser klasse brauchst du dafür nichts weiter als eine referenz auf den MouseListener von dem du die methode aufrufen willst und schon funktioniert das. oder gehts dir darum, daß du net weißt, wie du dafür ein MouseEvent generieren sollst? dann wär's aber ziehmlich hilfreich für uns, wenn wir wüsten, in welchem zusammenhang du das brauchst, weil an sich kannst du ja auch mouseClicked(null) aufrufen, wenn du das MouseEvent nicht verarbeitest.

beste grüße

wayne

 

[Guest]

Hi,

die Informationen sind wirklich was dürftig, so habe ich die entsprechende klasse mal neu generiert, um die Problematik aufzuzeigen.
Das Progreamm bricht ab mit:

Exception in thread "AWT-EventQueue-0" java.lang.StackOverflowError
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)
	at java.awt.AWTEventMulticaster.mousePressed(AWTEventMulticaster.java:262)

Ich denke mal, weil zu viele MouseListener registriert werden. Das liegt wohl an der while-Schleife des Threads. Nur brauche ich dort diesen Thread, wie auch mouseClicked() in diesem Thread.. Mir will keine elegantere Lösung einfallen, wie ich das hinkriegen soll.. Hier die Klasse...

public class TestMouseClass implements MouseMotionListener,
		MouseListener, Runnable {

	private ArrayList<Integer> testMouseArray = new ArrayList<Integer>();
	private Rectangle2D geo1, geo2;

	
	
	public Shape[] testGeo = {
			geo1 = new Rectangle2D.Double(10,10,10,10),
			geo2 = new Rectangle2D.Double(60,60,50,50),
			};

	public TestMouseClass(){
		
		//erste Initialisierung der ArrayList(), was defintiv sein muss
		for (int i = 0; i < testGeo.length; i++) {
			testMouseArray.add(new Integer(7));
			addMouseListener(this);
		}
		repaint();
	}

	
	public void paint(Graphics g) {
		Graphics2D g2D = (Graphics2D) g;
		for (int i = 0; i < 2; i++) {
			g2D.setColor(Color.BLACK);
			g2D.fill(testGeo[i]);
		}
	}

	protected void paintComponent(Graphics g) {
		super.paintComponent(g);
		paint(g);
	}

	
	
	public void run(){
		Object andereKlasse;
		while(andereKlasse.applicationIsUp == true){
			if (andereKlasse.status == 1){
				
				//hier will ich ein Rechteck erst klicken können, wenn eine Vorrausetzung erfuellt ist
				//es wird aber wieder ein MouseListener regestriert
				addMouseListener(new MouseAdapter(){
					public void mouseClicked(MouseEvent e) {
						int x = e.getX();
						int y = e.getY();
						int variablePflicht = 0;
						for (int i = 0; i < 2; i++) {
							if (testGeo[i].contains(x, y)) {
								variablePflicht = i;
								wiederAndereKlasse.ausfuehren(variablePflicht);
							}
						}
					    
						repaint();
					}
				});
			}
		}
	}


	public void mouseDragged(MouseEvent e) {
		// TODO Auto-generated method stub
		
	}


	public void mouseMoved(MouseEvent e) {
		// TODO Auto-generated method stub
		
	}


	public void mouseClicked(MouseEvent e) {
		// TODO Auto-generated method stub
		
	}


	public void mouseEntered(MouseEvent e) {
		// TODO Auto-generated method stub
		
	}


	public void mouseExited(MouseEvent e) {
		// TODO Auto-generated method stub
		
	}


	public void mousePressed(MouseEvent e) {
		// TODO Auto-generated method stub
		
	}


	public void mouseReleased(MouseEvent e) {
		// TODO Auto-generated method stub
		
	}
}

 

[SlaterB]

du hast eine for-Schleife am Anfang und dann noch die while-Schleife mit unnötigen MouseListenener,
muss alles weg, 2 insgesamt dürften maximal reichen,

falls du genauere Tipps haben willst, dann erkläe genauer, was das Problem ist,
und welche Strategie du bisher mit den vielen Listenern verfolgst

 

[Gast]

Hi,

das habe ich ja in meinem Posting versucht. Aber gut.. Vorneweg, der MouseListener in der for - Schleife ist auch schon weg gelassen worden, aber nichtsdestotrotz hängt sich das Programm auf, was wohl an der Registrierung der MouseListener in der while - Schliefe hängt. Also wäre die Registrierung in der for - Schleife die elegantere Lsg. Nun muss ich aber auf MouseClicks in der while - Schleife reagieren können. Für mich gibt es da erstmal nur die Lsg die MouseListener neu zu registrieren.

 

[SlaterB]

du kannst gerne darauf beharren, keiner hindert dich daran

 

[Gast]

Aber es funktioniert so nicht. Deswegen erfrage ich eine alternative Lsg.

 

[wayne]

was du mit diesem code tust ist, solange applicationIsUp == true und der status==1 immer wieder neue MouseListener einzutragen. diese reagieren auf ein mouseClicked, egal ob der status==1 oder applicationIsUp==true. ich vermute aber, das du den code in der mouseClicked nur dann ausführen willst, wenn die bedingungen zuschlagen und zwar nur ein einziges mal.

das würde dann so funktionieren, daß du erstens mal das erben von Runnable rausschmeißt, denn das brauchst du nicht. und den code aus der mouseClicked in der aktuellen methode run() schreibst du in den konstruktor deiner klasse, wie folgt:

addMouseListener(new MouseAdapter(){ 
               public void mouseClicked(MouseEvent e) { 
                  if( andereKlasse.applicationIsUp() && andereKlasse.getStatus() == 1 ){
                      int x = e.getX(); 
                      int y = e.getY(); 
                      int variablePflicht = 0; 
                      for (int i = 0; i < 2; i++) { 
                         if (testGeo[i].contains(x, y)) { 
                            variablePflicht = i; 
                            wiederAndereKlasse.ausfuehren(variablePflicht); 
                         } 
                      }                    
                      repaint(); 
                  }
               } 
            });

des weiteren würde ich mir mal deinen konstruktor nochmal ansehen. aktuell fügst du pro rechteck einen MouseListener hinzu und zwar immer den selben nämlich this. das ist, gelinde gesagt, schwachsinn. füge this ein einziges mal hinzu. dafür schreibst du die mouseClicked(...) oder mousePressed(...) oder welche auch immer um, dahingehen, daß du per aMouseEvent.getPosition() bzw. getX() oder getY() und aShape.contains(...) (also eine for-schleife über dein testGeo-machen und immer schon von jedem element vergleichen ob contains(...) == true und falls ja dann was auch immer ausführen und anschließend break, die for-schleife verlassen.

viel erfolg

wayne

 

[wayne]

sorry, mir ging vorhin die zeit aus. was ich im unteren absatz meine ist, daß du im prinzip das in der mousePressed(...) oder mouseClicked(...) machen solltest, was du bereits in deinem MouseAdapter tust. aber genau aus diesem grund stellt sich für mich die frage, warum du dann dein klasse als MouseListener deklarierst, die dazu gehörenden methoden leer lässt und statt dessen den code der dort hinein gehört in einen extra MouseAdapter auslagerst. für mich sieht das nach ziemlichen unsinn aus. entweder du machst einen MouseAdapter oder deine klasse ist ein MouseListener. ich würde dir letzteres vorschlagen, damit du in den methoden auch schön auf alle variablen zugreifen kannst. sprich vergiss den vorschlag den code des MouseAdapters in den konstruktor der klasse zu verlegen, entferne den MouseAdapter und schmeiße den code seiner mouseClicked(...) methode in die mouseClicked(...) methode deiner klasse hier herein. und deine klasse registrierst du bei dem JPanel auf die du deine klasse zeichnen lässt. hier in deiner klasse selbst brauchst du überhaupt nix zu registrieren, den deine klasse ist ja kein Component und somit auch auf keiner oberfläche selbstständig darstellbar, woraus folgt, daß sie selbst auch nicht in der lage ist, MouseEvents auszuwerfen, die du empfangen und wieder verarbeiten könntest. davon abgesehen habe ich meine zweifel ob deine paintComponent(Graphics) methode überhaupt jemals von irgendwem aufgerufen wird. würde deine klasse von Component erben so hätte ich diesen zweifel nicht, jedoch wäre es unsinnig, da paint(Graphics) zuerst aufgerufen werden würde und dieses nomalerweise dann unter anderem paintComponent(...) aufruft. sprich, wenn du von Component erben möchtest, dann verlege den code von paint(Graphics) in paintComponent(Graphics), denn in paint(...) sollte man eigentlich nix zeichen (funktioniert zwar trotzdem, ist aber unsauber). ach ja, bevor ich's vergesse. wenn du von Component erben möchtest, dann vergiss nicht, daß deine Component natürlich auch irgendwie mitteilen muss, wie groß sie sein möchte. will sagen, vergiss nicht die getPreferredSize()-methode zu überschreiben.

viel erfolg

wayne

 

[Iventar]

Danke Dir für die Unterstützung, und deine Tipps. Ich werde von meinem Konstrukt absehen und die Methode MouseClicked() wieder als "eigene" Methode auslagern in der Klasse. Das war wirklich ein relativ bescheidener Gedankengang. Ich werde jetzt mal die Tipps wenn möglich einbauen, und mich ggf nochmal hier melden.

Danke & Gruss