StackOverflow Error, objekte öfters erzeugen

[christian123]

Hallo

Ich wollte mir gerade ein 4-Gewinnt Spiel schreiben, damit ich etwas besser in Programmieren werde. Jetzt habe ich schon 7 Klassen erstellt, mit einem haufen Methoden, aber ich bekomme jetzt leider einen StackOverflowError

Das Problem ist jetzt anscheinend, dass ich in mehreren Klassen ein Objekt brauche, deshalb habe ich z.B. in 2 Klassen ein Random Objekt eingefügt. Oder in mehreren Klassen brauche ich z.B. eine bestimmte klasse, also erstelle ich in den benötigten klassen dieses Objekt.
Also ich dürfte momentann du viele verbindungen haben.

Was sollte ich jetzt machen? Muss ich das ganze neu schreiben und soll ich auf das achten?

 

[Tobias]

Wenn du mit deinen 7 Klassen zu viele Verbindungen hast, dürfte Java bei Programmen mit 100 oder 1000 oder 10000 Klassen ja gar nicht funktionieren. Auch wenn deren Design super wäre...

Was du beschreibst, klingt eher nach einer Endlosschleife...

mpG
Tobias

 

[Tobias]

Der Stackoverflow-Error gibt dir auch an, wo er auftritt. Poste mal den relevanten Codeteil...

mpG
Tobias

 

[SlaterB]

ein StackOverflow kommt meist durch eineneinzelne Schleife im Programm,
das kann alles möglich sein:
ein gar nicht benötigter Bug,
eine leicht zu änderne Programmeigenschaft,
oder eine essentielle Programmeigenschaft

wenn es ganz ganz ganz dumm gelaufen ist, dann musst du wirklich was grundlegendes ändern,
normalerweise ist das aber schnell behoben,

hast du keinen StackTrace, der dir die Reihenfolge der Operationen gibt?
ohne Code kann man sonst wenig sagen..

-------

deine Klassen-Verknüpfungen haben normalerweise wenig damit zu tun, aber auch dort kann es zu Schleifen kommen, zum Beispiel bei den Konstruktoren:
Klasse A erzeugt Exemparvariable von Klasse B,
B wiederum von A usw.,
auch bei der Initialisierung von einzelnen statischen Variablen denkbar,

Umgehung dafür (unsauber):
erstmal leere Objekte mit null-Referenzen erzeugen und dann überall init() ausführen

 

[christian123]

Hallo

Ja, das ganze habe ich mir auch gedacht, dass man da mehrere Objekte erzeugen kann...

public Spielbrett() {
stein = new steine();
parser = new Parser();
computer = new Computer();

Die 3 sachen werden leider immer wiederholt, viel mehr wird in der Hauptklasse "Spielbrett" nicht abgearbeitet.

Und diese Klasse Computer ist gar nicht verdächtig. (Alle Methoden gehen noch nicht).

import java.util.Random;
public class Computer {
	Random zufall;
	steine steinn;
	ViergewinntCheck viergewinn;

	public Computer() {
    zufall=new Random();
    steinn=new steine();
    viergewinn=new ViergewinntCheck();
	}

	public int generiereZahl()
	{
		if (viergewinn.gibtdrei() ==false)
		{
		return  zufall.nextInt(Spielbrett.BREITE);
        }
        else
        {
         return berechne3er();
        } 
	}
	
	public int zahl()
	{
		int zufall=generiereZahl();
		int freiezeile=stein.freiezeile(zufall);
		
			if (!(stein.frei(freiezeile, zufall)))
			{
			zahl();
			}
			
	return zufall;		
	}
	
	public int berechne3er()
	{
	return 0;    
	}    
	
}

 

[Tobias]

Was möchtest du uns mit diesem Post sagen?

Wo tritt der StackOverflowError auf? Kopiere mal die Ausgabe in der Konsole hier rein und gib uns die Klasse, in der Fehler auftritt.

mpG
Tobias

 

[Marco13]

Vielleicht wollte er mit diesem Post sagen: "Der Fehler liegt in der Methode 'zahl()'" :wink:

Die Methode ruft sich selbst auf, und kann darum (leicht) einen StackOverflow verursachen. Vielleicht gibt
stein.frei(...)
im Moment noch IMMER 'false' zurück? Dann würde es ihn da schon raushauen...