Programm soll auf die Eingabe vom Mouselistener warten.

[BlinderOpa]

Hi...
ich möchte das altbekannte Übungsprogramm Schiffeversenken mit Java programmieren.
die Consolenversion ist schon komplett lauffähig und der Computer stellt sich sogar nicht mal soo dum an...^^

Jetzt bin ich dabei, das ganze grafisch aufzubauen.
Die anzeige selber funktioniert und ich hab für das Panel der Feld-Anzeige auch einen Mouselistener...
Im Moment soll der Mouslistener erstmal ermitteln, in welches Feld ich schießen möchte.

mein Problem:

das Programm rennt mir einfach über die Stelle hinweg, in der ich die Eingabe über die Maus machen möchte...
in der Console wartet der einfach bis die Eingabe über den Buffered Reader kommt.

ich hab da zwar eine Lösung, die saugt aber 100% der CPU da es eine Endlosschleife ist, die solange schleift, bis ich geclickt hab... und das mit der maximalen Geschwindigkeit...^^

         boolean clicked= Grafik.getclicked();
         while (clicked==false)
	{clicked= Grafik.getclicked();}

oder ich mach das über eine Eingabe in der Console und jedesmal wenn ich ein Feld auswähle muss ich Fenster wechseln und Eingabe drücken...-.-

			BufferedReader userIn = new BufferedReader(new InputStreamReader(System.in) );
			System.out.println ("weiter? ");
			String input = userIn.readLine();

[/u]

 

[Gast]

Das Problem liegt bei deinem Programmdesign,
schreibe das programm einfach so um, dass es erst beim klicken eines schiffes den Algorithmus startet also -> prüft ob sich dort ein ziel befindet -> falls nein: zeige wasseranimation etc... -> falls doch: zeige brennende animation und speichere status in vector der schiffe...

ggf kannst du während diese animation und prüfung abläuft eine flag setzen, die weiteres klicken verbietet..

 

[BlinderOpa]

Sorry... hab zu spät das Game-Unterforum gesehen...

 

[Leroy42]

Was du da machst, hat ja auch gar nichts mit einem MouseListener zu tun.

Der MouseListener wird automatisch aufgerufen, wenn die
Maustaste gedrückt wurde und informiert dann die Anwendung.

Hast du dir die Kapitel über Action/Mouse-Listener überhaupt
schon einmal durchgelesen?

 

[BlinderOpa]

hm... werd ich mal überdenken.. danke...
ist nur blöd, da ich bisher alles so in Module verschachtelt habe...
und wollte zu dem bisherigen Consolenspiel eigentlich nur die Grafik durch andere Module hinzufügen...^^
aber ich werde das mal ein wenig umdenken und so...

hab gehofft es gibt ein Befehl dafür, auf die Eingabe zu warten...
oder dass man den Buffered Reader etwas anders auslegen könnte...
so dass der anstatt der Tastatureingabe auf die Eingabe vom Mousemodul wartet...

 

[Leroy42]

so dass der anstatt der Tastatureingabe auf die Eingabe vom Mousemodul wartet...

... der MouseListener (deines Maus-Moduls) muss deine
eigentliche Anwendung informieren, daß etwas durch
den user geschehen ist.

 

[BlinderOpa]

hm.... im Prinzip soll das Mouseevent die Koordinaten ausrechnen und neuX und neuY damit füttern...
neuX und neuY soll dann weitergeleitet werden über das Mainprogramm (in dem bei mir alles zusammenläuft...) zur Spielfeldclasse bzw zur Zielcomputerclasse die die Trefferverwertung usw. macht...
also soll das Programm abwarten bis ich mit der Eingabe fertig bin... oder bis ich n boolean auf True gesetzt habe oder ähnliches...

 

[Leroy42]

public void mouseClicked(MouseEvent e) {
  int x = e.getX() / widthOfField;
  int y = e.getY() / heigthOfField;
  mainModule.doSomethingWith(x, y);
}

 

[BlinderOpa]

genau... nur die Main rauscht einfach über
mainModule.doSomethingWith(x, y);
drüber...-.-
so sieht mein Mouseevent aus... raus kommen schöne die Koordinaten meiner Tabelle..

	public void mouseClicked(MouseEvent m)
	{
	
		if( m.getSource() instanceof SpielfeldPanelModule) {
			SpielfeldPanelModule sfm = (SpielfeldPanelModule)m.getSource();

			neuX= m.getX();
			neuY = m.getY();
			neuX -= sfm.getInsets().left;
			neuX -= 20;
			neuY -= sfm.getInsets().top;
			neuY -= 20;
			neuX = (int)Math.floor(neuX / 20d)+1;
			neuY = (int)Math.floor(neuY / 20d)+1;
			
			click=true;
		}	
		//Objekt ist jetzt richtig. aber der wartet nicht auf den input...-.-
		PvE.setneuX(neuX); //hier wird das ganze an die Main "PvE" geschickt...
		PvE.setneuY(neuY);
		System.out.println(" Die Maus ist bei ( "+neuX+" | "+neuY+" )");
		}

und so siehts in der Main aus...

		Player1 schießt...
			System.out.println ("MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM");
			System.out.println ("PLAYER 1 ist dran.");
			System.out.println ("WWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWW");
			p1feld.printConsoleArray(zielP1.zielarray);
			Grafik.Hauptfenster(p1feld.getFeld(), zielP1.zielarray) ;


			BufferedReader userIn = new BufferedReader(new InputStreamReader(System.in) );
			System.out.println ("weiter? ");
			String input = userIn.readLine();

			//Funktioniert... zieht aber Leistung ohne Ende durch das dauergefrage "clicked==true?" CPU Auslastung==100%...
//			boolean clicked= Grafik.getclicked();
//			while (clicked==false)
//			{clicked= Grafik.getclicked();}
//			daher wird das auskommentiert...

			neuX=Grafik.getneuX();
			neuY=Grafik.getneuY();
			
			zielP1.setneuX(neuX);
			zielP1.setneuY(neuY);

			System.out.println ("SCHUSS auf Y "+neuY+"! X "+neuX);

//			Schuss auswerten...
			treffer=false;
			treffer=p2feld.setMyFeldEnemyShot(neuY, neuX);

			if (treffer == true)
			{
				System.out.println ("TREFFER!!");
				zaehlerplayer2=zaehlerplayer2-1;
				System.out.println ("Ein Schiff-Feldchen weniger...:) "+zaehlerplayer2+" bleiben noch uebrig.");
			}
			else
			{
				System.out.println ("Daneben... Schade...");
			}
			zielP1.trefferAuswertung(treffer);

wie man sieht ist die hälfte noch auf die Console geschrieben... aber das ändere ich Schritt für Schritt...[/quote]

 

[Marco13]

Hm. Das macht man ggf. über den wait/notify-Mechanismus. Sinngemäß wäre das dann sowas

class HauptPanel
{
    private int clickX, clickY;
 
    void spiele()
    {
         while (!spielZuende)
         {
              System.out.println("Warte auf Eingabe...");
              synchronized(this)
              {
                   try
                   {
                        wait();
                   }
                   catch (InterruptedException e) {}
              }
              System.out.println("Eingabe ist erfolgt. Es geht weiter");
              schießeAuf(clickX, clickY);
         }
    }


....
    public void mouseClicked(MouseEvent e)
    {
         clickX = berechneX();
         clickY = berechneY();
         
         System.out.println("Eingabe ist erfolgt. Sage dem main-loop, dass es weitergehen kann");
         synchronized(this)
         {
              notify();
         }
    }

Innerhalb eines "synchronized(hauptPanel)"-Blocks wird 'wait' aufgerufen. Dieses wait wartet (ohne Prozessorbelastung!) so lange, bis in einem anderen "synchronized(hauptPanel)"-Block dann 'notify' aufgerufen wird.

 

[BlinderOpa]

geht das auch klassen übergreifend?
das klingt so ziemlich danach was ich bräuchte...^^
ich bastle das mal so als versuch in einer Klasse...

 

[Marco13]

Das geht auch Klassenübergreifend. Das entscheidende ist, dass in BEIDEN 'synchronized(object)'-Blöcken wirklich auf ein und dasselbe Objekt gesynchronized wird!

 

[BlinderOpa]

obwohl halt.. lesen forthewin...
innerhalb eines blocks....überlesen...^^

 

[BlinderOpa]

GEIL... funktioniert..
jetzt noch ein wenig verschönern...
danke...
das Problem beim Programmieren meistens ist, dass man einfach die Befehle nicht alle kennt... bzw nicht alle kennen kann...

 

[Leroy42]

GEIL... funktioniert..

Dann ist ja gut!

Dennoch frage ich wozu das ganze Geraffel mit
wait/notify eigentlich notwendig ist.

Mit einer anderen Modulstruktur wäre das nicht
nötig gewesen.

Aber egal, ... es läuft

 

[Marco13]

Warum "Geraffel"? Ich könnte mir vorstellen, dass da ein Thread lauft, der abwechselnd von zwei Spielern Eingaben erwartet. Und dieses Erwarten wird eben durch das 'wait' erreicht...? Aber vielleicht habe ich BlinderOpa's Absichten falsch verstanden...

 

[phyrexianer]

ist es nicht geschickter einfach durch den Mouslistener eine Methode aufzurufen ??? und der einfach die Koordinaten übergeben ??? also das was er in der Main hat einfach als eine Methode Schreiben ??? da braucht man keine Schleifen kein Sync usw. Die Methode wird dann aufgerufen wenn sie auch gebraucht wird. Denn wenn ich es richtig gelesen habe ist sogar ein "OnePlayer" Game !?!
Selbst bei zwei Spielern gäbe es da kein Prob wenn man zusätzlich noch den Spieler übergibt welche gedrückt hat.... zusätzlich zu den Koordinaten!

 

[Leroy42]

...

Genau das meinte ich mit andere Modulstruktur

 

[Leroy42]

dass da ein Thread lauft, der abwechselnd von zwei Spielern Eingaben erwartet.

Genau das sehe ich schon als falsches Design an (siehe phyrexianer).
Ich würde das Spiel eher so konzipieren, das das entsprechende
Modul sowohl von einer Kommandozeilen-GUI als auch via
richtiger GUI genutzt werden kann.

 

[BlinderOpa]

im Prinzip so, dass die GUI die "Main" Methode aufruft...
da ich momentan aber am lernen bin, hatte ich alles Modulartig zusammengebaut... also nach dem Motto:
main ruft grafik auf und bekommt X/Y zurück... dann ruft die Main das Spielfeld auf und frägt nach ob treffer oder nicht... usw usw... und im Prinzip wollte ich alles einfach ersetzbar machen... erst hatte ich alles über die Console laufen (lernstatus) und jetzt gehts eben an die GUI...

naja.. Danke für die Hilfe

thread closed...