If_Bedingung in statischer Methode beim zweiten Mal nicht durchlaufen

[wutzefreck]

Guten Tag,

ich hab hier mal einen stark verkürzten Codeteil reingestellt. Ich muss mich ausnahmsweise ein wenig mit Java beschäftigen und drehe mich völlig im Kreis.
Eigentlich wollte ich, dass die "if(glass.isEmpty)"-Bedingung beim ersten Durchlauf true ergibt und beim zweiten Mal false. Dafür habe ich mir diese glassChecked Methode gebaut und mich dann so lange selbst verwirrt bis dies hier übrig geblieben ist. Natürlich soll glassChecked() nicht immer true sein...
Ich komme hier grad einfach nicht vorran. Kann mir jemand helfen, dass gewünschte Ergebnis zu erzielen?

static Glass createGlass(......) {
		......
               glass = ...
		if (glass.isEmpty() && !glassChecked()) {
			...
			glassChecked();
		} else {
			....
		}
		return createSomethingElse
	}
	
	static boolean glassChecked() {
			return true;
	}

 

[Keyone94]

glass = ...
if (glass.isEmpty() && !glassChecked()) {
    ...
   glassChecked();
} ...

ich vermute einfach mal glassChecked(); gibt einen boolean wieder, ein zweiter Aufruf davon dürfte einfach gar nichts bewirken.
aber eins nach dem andern:
was macht glassChecked?
wo änderst du den Wert, der bei glass.isEmpty() wiedergegeben wird (also in welcher Methode)?

 

[wutzefreck]

Das sind alles größere Klassen aus einem sehr großen Projekt. Kann nur abgewandelte Bruchstücke hier posten.
Glass.isempty ist fix und funktioniert auch, da braucht nichts geändert zu werden. Ich wollte mir mit der Methode glassChecked selbst ne Methode erstellen um mir nen Boolean zu liefern mit dem ich die of Bedingung abändern kann. Die hatte vorher dementsprechend auch das glassChecked nicht drin.
Ich brauche etwas um die of Bedingung beim zweiten mal im "Else" landen zu lassen.

 

[Keyone94]

glassChecked gibt ja einen boolean wieder
wird der Wert dieses booleans irgendwo mal auf true gesetzt?
du bräuchtest einen zweite Methode die den Wert auf true setzt denke ich

 

[wutzefreck]

Achso? Damit ich dann eine Methode für die Bedingung nutze und die andere um es auf True zu setzen?

 

[Keyone94]

ja genau anders wird das nicht klappen,
wenn du in glassChecked den wert auf true setzt, bevor er wieder gegeben wird, ist er natürlich bei der ersten Wiedergabe auch schon true

 

[wutzefreck]

Klingt logisch. Danke. Ich Versuch es mal.

 

[wutzefreck]

In etwa so?

private static boolean checkGlass(boolean glassChecked) {
		return glassChecked;
			
	}
	
	private static boolean glassAlreadyChecked() {
		if (checkGlass(true)) {
			return true;
		}
		else {
			return false;
		}
		
	}

 

[dzim]

Mit dem Aufruf wäre die Methode eigentlich sinnlos.

private boolean mGlassChecked = false;

private void setGlassChecked(boolean value) {
    mGlassChecked = value;
}

und dann noch so was hier.

if (glass.isEmpty() && !mGlassChecked) {
    // [...]
    setGlassChecked(true);
} else {
    // [...]
}

"Static" verwende nur, wenn dieser Wert für alle Objekte dieser Klasse so sein sollen. Ansonsten genügt ein normaler privater Wert.

 

[Thallius]

Vielleicht eine blöde Frage aber mit Deinem Wissen hast du dieses riesen Projekt ja sicher nicht selber programmiert. Wieso fragst du nicht die Leute und läßt Dir direkt vor Ort helfen und was beibringen? Das ist doch viel effektiver. Mal ganz davon abgesehen, dass ich als Programmiere dieses Paketes Schüttelfrost kriegen würde wenn ein Anfänger daran rumfummelt und lauter blödsinn verzapft.

Gruß

Claus

 

[wutzefreck]

Ist für eine Bachelorarbeit (keine Sorge, am Produktivcode wird nix geändert).

Ich habe noch immer das Problem, dass die Ursprungsmethode static bleiben muss und er somit meckert, dass er keine static references auf non-static fields machen kann.

 

[Natac]

Dann mach das Feld doch auch static?

private static boolean glassChecked;

private static boolean isGlassChecked(){return glassChecked;}

private static void setGlassChecked(boolean checked){glassChecked = checked;}

 

[Keyone94]

Ist für eine Bachelorarbeit

hoffentlich nicht für deine, und hoffentlich nicht für Informatik :noe:

Ich habe noch immer das Problem, dass die Ursprungsmethode static bleiben muss und er somit meckert, dass er keine static references auf non-static fields machen kann.

warum muss die Methode denn statisch sein? für mich klingt der Name (der Methode) nicht nach statischem Inhalt

und falls das nicht dein Code ist, wäre es nicht besser derjenige, dem der Code gehört, macht das selber?

 

[wutzefreck]

So. Gehirnstaub entfernt und Fehler gefunden. Sorry. Habe komplett falsch gedacht. Habt Nachsicht: bin frischer Papa und schlafe momentan wenig ;-)

Problem mit internen Mitteln gelöst.

Danke trotzdem

 

[dzim]

Stimme dir da absolut zu @Thallius...