ZZZnake - Dynamisch geht nix

01/22/2013 14:56 Mostey#1

Moin. Bin seit etwa einer Woche am Lernen von Java und bin es bisher relativ langsam und entspannt angegangen. Heute wollte ich mal nach den ersten 2 Kapiteln des "Java ist auch eine Insel" Buches mal das kleine "ZZZnake" Programm umschreiben - es "dynamisch" machen.

Das hei�t das ich die Positionen der Figuren durch eine Zufallszahl festsetzen m�chte, die nach jedem generieren anders ist. Die Positionen werden durch die 2 Random Methoden "randomX" und "randomY" generiert. Mein bisheriger Code pr�ft, ob es bei den Positionen irgendwo eine Kollision gibt und soll die Positionen dann mal testweise ausgeben damit ich sehe ob da alles passt.

Das tut er aber leider nicht, das Programm bricht irgendwo ab.

Hier mal der Ansatz:

Spoiler

mfg

01/22/2013 15:03 Der Anbieter#2

Gibt es beim Kompilieren keine Fehlermeldung?

01/22/2013 15:05 Mostey#3

Quote:

Originally Posted by Der Anbieter

Gibt es beim Kompilieren keine Fehlermeldung?

Nein, w�sste auch nicht wo es da eine geben sollte.

Er bricht halt bei der Positionsgenerierung irgendwo ab und das Programm schlie�t sich.

01/22/2013 15:33 Sedrika#4

Ich hab das ganze mal etwas umgeschrieben. Getestet habe ich es nicht, da ich keinen Java Compiler zurzeit habe.

Code:

import java.text.SimpleDateFormat;
import java.util.Date;
/*
 *  Spiel: Ein Spieler (S) muss 3 Geldstuecke ($) einsammeln ohne von der Schlange (!) gebissen zu werden
 */
public class ZZZZnake {

	public static void main(String[] args) {
		SimpleDateFormat sdf = new SimpleDateFormat("HH:mm:ss");
		var nDate = sdf.format(new Date());
		JOptionPane.showMessageDialog(null, "ZZZZZZnake!");
		int confirm = JOptionPane.showConfirmDialog(null, "Spiel starten? | Systemzeit: " + nDate);
		// Confirmrequest
		if (confirm == 0) // Ja = Spielablauf
		{
			positions();
		} 
		else // 1 oder 2 = Nein = Abbruch
		{
			System.exit( 0 );
		}
	}

	public static void positions() {
		Point playerpos = new Point();
		Point schlangenpos = new Point();
		Point geldpos1 = new Point();
		Point geldpos2 = new Point();
		Point geldpos3 = new Point();
		int minXY = 0;
		int maxX = 40;
		int maxY = 10;

		playerpos.setLocation(GetRandomPos( minXY, maxX ), GetRandomPos( minXY, maxY ));
		schlangenpos.setLocation(GetRandomPos( minXY, maxX ), GetRandomPos( minXY, maxY ));
		geldpos1.setLocation(GetRandomPos( minXY, maxX ), GetRandomPos( minXY, maxY ));
		geldpos2.setLocation(GetRandomPos( minXY, maxX ), GetRandomPos( minXY, maxY ));
		geldpos3.setLocation(GetRandomPos( minXY, maxX ), GetRandomPos( minXY, maxY ));
				
		while( geldpos1.getLocation() == playerpos.getLocation() ||
			   geldpos2.getLocation() == playerpos.getLocation() ||
			   geldpos3.getLocation() == playerpos.getLocation() ||
			   playerpos.getLocation() == schlangenpos.getLocation() )
		{
			geldpos1.setLocation(GetRandomPos( minXY, maxX ), GetRandomPos( minXY, maxY ));
			geldpos2.setLocation(GetRandomPos( minXY, maxX ), GetRandomPos( minXY, maxY ));
			geldpos3.setLocation(GetRandomPos( minXY, maxX ), GetRandomPos( minXY, maxY ));
			playerpos.setLocation(GetRandomPos( minXY, maxX ), GetRandomPos( minXY, maxY ));
			System.out.println( "X: " + playerpos.getX() + " Y:" + palyerpos.getY() );
			System.out.println( "X: " + schlangenpos.getX() + " Y:" + schlangenpos.getY() );
			System.out.println( "X: " + geldpos1.getX() + " Y:" + geldpos1.getY() );
			System.out.println( "X: " + geldpos2.getX() + " Y:" + geldpos2.getY() );
			System.out.println( "X: " + geldpos3.getX() + " Y:" + geldpos3.getY() );
		}

	}

	public static int GetRandomPos(int low, int high) {
		return (int) (Math.random() * (high - low) + low);
	}
}

01/22/2013 15:40 Mostey#5

Selbes Ergebnis, die Dialoge funktionieren, alles danach allerdings nicht mehr. Finde aber auch dass das genau das selbe ist was ich ja schon geschrieben hatte, halt nur mit anderen L�sungsans�tzen.

01/22/2013 15:44 Sedrika#6

Ich sehe gerade das hier xD

Code:

if (confirm == 0) // Ja = Spielablauf

�nder es mal in

Code:

if (confirm) // Ja = Spielablauf

0 steht f�r false.

Abbruch ist immer 0.

01/22/2013 15:54 Mostey#7

Wir verwenden hier keinen Boolean, daher geht das ohne den Wert in der if Abfrage sowieso nicht. 0 ist in Java nicht immer false. Zumindest nicht bei der Funktion "JOptionPane".

Stimmt schon, habs eben auch nochmal getestet. Der z�hlt von rechts nach links: Abbruch - Nein - Ja

€: Manchmal k�nnt ich mir so derma�en eine reinhauen... echt ey.

Code:

while (geldpos1.equals(playerpos) || geldpos2.equals(playerpos)
				|| geldpos3.equals(playerpos) || playerpos.equals(schlangenpos)) {
			geldpos1.setLocation(randomX(), randomY());
			geldpos2.setLocation(randomX(), randomY());
			geldpos3.setLocation(randomX(), randomY());
			playerpos.setLocation(randomX(), randomY());
			System.out.println(playerpos);
			System.out.println(schlangenpos);
			System.out.println(geldpos1);
			System.out.println(geldpos2);
			System.out.println(geldpos3);

		}

Ich hab ne while Schleife die �berpr�ft ob die Koordinaten alle verschieden sind und nicht aufliegen. Die Ausgabe kommt ja nur wenn sie mindestens einmal irgendwo aufeinander gelegen haben.

Sorry.

01/22/2013 16:00 Sedrika#8

Dann nimm doch mal die Abfrage raus und teste mal so, um zu sehen, ob es an der Abfrage liegt.

Ich verstehe auch nicht, wieso du nicht so abfragst:

Code:

int confirm = JOptionPane.showConfirmDialog(null, "Spiel starten? | Systemzeit: " + nDate, "TITEL", JOptionPane.YES_NO_CANCEL_OPTION);

und dann mit

Code:

if (confirm == JOptionPane.YES_OPTION)
		{
			positions();
		}
		else // if (result == JOptionPane.NO_OPTION)
		{
			System.exit(0);
		}

den R�ckgabewert ermittelst.

01/23/2013 14:40 Mostey#9

So, nach diesem dummen Fehler meinerseits hab ich mich nochmal an das Minigame gemacht und nun Spielfeld, Figuren und Steuerung hinzugef�gt.

Allerdings hab ich so das Gef�hl dass das sehr un�bersichtlich f�r andere wird, daher poste ich mal den Source von der Klasse mit den ganzen Methoden und ihr k�nnt ja mal dr�berschauen. �brigens hab ich sehr viele Probleme mit dem R�ckgabewert. Ich w�rde gern f�r jede Aufgabe eine seperate Methode machen und das alles in der Main Methode zusammenfassen, allerdings geht das oftmals nicht, da ich bei einigen Funktionen eine Eingabe ben�tige um die Methode auszuf�hren, da ich das selbst so festgelegt habe. Naja, seht ihr ja dann schon selbst.

Zur Verst�ndlichkeit ist einiges kommentiert.

Spoiler

Code:

/*
 *  Spiel: Ein Spieler (S) muss 3 Geldstuecke ($) einsammeln ohne von der Schlange (!) gebissen zu werden
 */
public class ZZZZnake {

	public static void main(String[] args) {
		GregorianCalendar gc = new GregorianCalendar();
		int h = gc.get(Calendar.HOUR) + 12;
		int m = gc.get(Calendar.MINUTE);
		int s = gc.get(Calendar.SECOND);
		JOptionPane.showMessageDialog(null, "ZZZZZZnake!");
		int confirm = JOptionPane.showConfirmDialog(null,
				"Spiel starten? | Systemzeit: " + h + ":" + m + ":" + s);
		; // Confirmrequest
		if (confirm == 0) // Ja = Spielablauf
		{
			spielfeld(positions());

		} else // 1 oder 2 = Nein = Abbruch
		{
			System.exit(0);
		}

	}

	public static Point[] positions() {
		Point playerpos = new Point();
		Point schlangenpos = new Point();
		Point geldpos1 = new Point();
		Point geldpos2 = new Point();
		Point geldpos3 = new Point();
		Point[] positionen = new Point[5];

		playerpos.setLocation(randomX(), randomY());
		schlangenpos.setLocation(randomX(), randomY());
		geldpos1.setLocation(randomX(), randomY());
		geldpos2.setLocation(randomX(), randomY());
		geldpos3.setLocation(randomX(), randomY());

		while (geldpos1.equals(playerpos) || geldpos2.equals(playerpos) // Dynamisch
																		// und
																		// optional,
																		// falls
																		// Koordinatenkonflikte
																		// auftauchen.
				|| geldpos3.equals(playerpos) || playerpos.equals(schlangenpos)) {
			geldpos1.setLocation(randomX(), randomY());
			geldpos2.setLocation(randomX(), randomY());
			geldpos3.setLocation(randomX(), randomY());
			playerpos.setLocation(randomX(), randomY());

		}

		// Alle Koordinaten in einem Array untergebracht damit man diese in
		// anderen Methoden verwenden kann
		positionen[0] = playerpos;
		positionen[1] = schlangenpos;
		positionen[2] = geldpos1;
		positionen[3] = geldpos2;
		positionen[4] = geldpos3;

		return positionen;
	}

	public static int randomX() {
		return (int) (Math.random() * 10); // Zufallszahl bis 10

	}

	public static int randomY() {
		return (int) (Math.random() * 40); // Zufallszahl bis 40
	}

	public static void spielfeld(Point[] positionen) {

		// Spielfeld: size = 40 * 10 ;
		boolean beendet = false; // Noch ohne Funktion, keine Ahnung wie ich von
									// einer anderen Methode den R�ckgabewert in
									// diese Methode noch zusaetzlich neben der
									// Point Variable uebergeben soll damit das
									// Spiel beim Einsammeln der 3 Goldstuecke
									// stoppt.

		while (beendet == false) {
			for (int a = 0; a < 10; a++) {
				for (int b = 0; b < 40; b++) {
					Point xy = new Point(a, b);
					if (positionen[0].equals(xy)) // pos0 = playerpos
					{
						System.out.print("S");
					} else if (positionen[1].equals(xy)) { // pos1 = schlange
						System.out.print("!");
					} else if (positionen[2].equals(xy) // geld1, 2, 3
							|| positionen[3].equals(xy)
							|| positionen[4].equals(xy)) {

						System.out.print("$");
					} else {
						System.out.print("*");
					}
				}

				System.out.println();
			}
			steuerung(positionen);
		}

	}

	public static Point[] steuerung(Point[] positionen) {
		// Keys: Hoch - w / Runter - s / Rechts - d / Links - a
		// Pausetaste: P
		boolean pausiert = false;

		switch (new Scanner(System.in).next().charAt(0)) {
		case 'w':
			positionen[0].x = Math.max(0, positionen[0].x - 1); // Spielerschritt
			break;
		case 's':
			positionen[0].x = Math.min(9, positionen[0].x + 1);
			break;
		case 'a':
			positionen[0].y = Math.max(0, positionen[0].y - 1);
			break;
		case 'd':
			positionen[0].y = Math.min(39, positionen[0].y + 1);
			break;
		case 'p': // Hier komm ich �berhaupt nicht weiter, der Dialog
					// funktioniert aber der Scanner ist weiterhin aktiviert,
					// wie soll ich den auch noch deaktivieren? (Er ist in einer
					// anderen Methode)
			pausiert = true;
			if (pausiert == true) {
				int pause = JOptionPane.showConfirmDialog(null,
						"Spiel pausiert, fortfahren?", "ZZZnake!",
						JOptionPane.YES_NO_OPTION);
				if (pause == 1) {
					int sicher = JOptionPane.showConfirmDialog(null,
							"Moechtest du das Spiel wirklich beenden?",
							"ZZZnake!", JOptionPane.YES_NO_OPTION);
					if (sicher == 0) {
						System.exit(0);
					} else {
						pausiert = false;
					}
				} else {
					pausiert = false;
				}
			}
			break;

		}

		if (positionen[0].x < positionen[1].x) { // Automatisierte
													// Schlangenbewegung
			positionen[1].x--;
		} else if (positionen[0].x > positionen[1].x) {
			positionen[1].x++;
		} else if (positionen[0].y < positionen[1].y) {
			positionen[1].y--;
		} else if (positionen[0].y > positionen[1].y) {
			positionen[1].y++;
		}

		if (positionen[0].equals(positionen[1])) {
			System.out.print(positionen[0] + "" + positionen[1]); // Wenn
																	// Schlange
																	// = Player
			JOptionPane.showMessageDialog(null,
					"Die Schlange hat dich gefressen und du hast verloren!",
					"ZZZnake", JOptionPane.CLOSED_OPTION);
			System.exit(0);
		}

		return positionen; // weitergabe von den aktualisierten positionen f�r
							// Generierung vom Spielfeld
	}

}