return-Wert verwenden?

[tigull]

Ich stehe mal wieder an bei meinem Konsolen-Minesweeper.

Ich habe mehrere Klassen. Von Bedeutung sind jetzt Raster und Zelle. Ich habe in der Klasse Raster ein 2D-Array namens feld, das mit Zellen gefüllt ist. Im Objekt Zelle habe ich Datenfelder wie

boolean sichtbar

oder

int anzahlminen

. Ursprünglich hatte ich diese auf public und konnte sie so aus anderen Klassen mit Punktnotation abrufen. Aber nun habe ich erfahren, dass das für die Bewertung schlecht ist, wenn man diese Datenfelder auf public hat, sicherheitstechnisch ...
Deshalb möchte ich nun den anzahlminen-Wert mit einer Methode abrufen. Diese ist in der Klasse Zelle und sieht folgendermassen aus:

public int gibAnzahlminen() {
		return anzahlminen;
	}

In der Klasse Raster möchte ich diesen return-Wert nun weiterverwenden. Aber wie stelle ich das an?
Ich habe es mal so versucht, aber da scheint was falsch zu sein.

int returnwert = (feld[wertx][werty].gibAnzahlminen);

Könnt ihr mir helfen?

 

[Spacerat]

Ich habe es mal so versucht, aber da scheint was falsch zu sein.

int returnwert = (feld[wertx][werty].gibAnzahlminen);

Irgendwie fehlen da doch Klammern beim Moethodenaufruf oder sind die nur falsch gesetzt? [c]int returnwert = feld[wertx][werty].gibAnzahlminen();[/c]

 

[The_S]

Was scheint denn daran falsch zu sein (außer die Syntax)?

 

[tigull]

Danke, das war der Fehler. Aber das nächste Problem bahnt sich bereits an.
Ich muss in meinem Minesweeper-Spielfeld, wenn ich auf ein leeres Feld klicke, alle leeren Felder im Umfeld sichtbar machen (sichtbar ist ein boolean-Wert, den jedes Objekt in meinem Array hat) sowie das erste Feld mit Zahl und dann nicht mehr weiter. Aber mit x/y-Koordinate erhöhen und verkleinern ist das irgendwie nicht möglich. Hat jemand eine Idee?

 

[The_S]

Natürlich ist das möglich. Du musst vom "Klick" aus einfach in alle Richtungen suchen. Könnte man bspw. rekursiv machen.

 

[tigull]

Bis jetzt hab ich es so, dass rund um das betroffene 0-Feld die Felder aufgedeckt werden. Und dann müsste eigentlich bei jedem Feld, das 0 war, diese Methode wieder aufgerufen werden ... hmmm ich muss das nochmal ein bisschen durchdenken.

 

[bone2]

Bis jetzt hab ich es so, dass rund um das betroffene 0-Feld die Felder aufgedeckt werden. Und dann müsste eigentlich bei jedem Feld, das 0 war, diese Methode wieder aufgerufen werden ... hmmm ich muss das nochmal ein bisschen durchdenken.

Du hast gerade Rekursion gefunden. Rekursion ? Wikipedia

 

[tigull]

:toll: Ich weiss, danke!

Hatte dann leider einen StackOverflowError. Muss das dann mal mit dem Debugger anschauen.

 

[maki]

StackOverflowError im zusammenhang mit Rekursion kann IME 2 Ursachen haben:
1. Endlosrekursion
2. Einen zu kleinen Stack (Windows 32 BIt System haben dieses Problem), nach ca. 4000-5000 rekursiven aufrufen hast du dann den StackOverflowError

 

[Landei]

Du musst dir irgendwie merken, welche Felder du schon "besucht" hast, und diese ignorieren.

 

[tigull]

Du musst dir irgendwie merken, welche Felder du schon "besucht" hast, und diese ignorieren.

Aber das Feld ist höchstens 10x10=100 Felder gross, das kann ja fast nicht der Fehler sein, oder?

 

[The_S]

Doch, wenn du alle Felder wieder besuchst und wieder und wieder und wieder dann ist genau das der Fehler.

 

[tigull]

Du musst dir irgendwie merken, welche Felder du schon "besucht" hast, und diese ignorieren.

Du hast Recht. Ich hab mir jetzt noch eine "besucht"-Variable eingebaut. Und es scheint zu funktionieren. Vielen Dank euch allen🙂