Vermeidung von doppelten Code

[Bullz]

Hallo, Source Code sagt ja bekanntlich mehr wie 1000 Wörter.

Habe eine Variable bVar die entweder an dieser Stelle true / false sein kann.

Bool bVar = true | false
if (Bool == 1) {
if (bedingungen1 && bedingungen2)
//Anweisungsblock 10 Zeilen lang
} else {
if (bedingungen1) // ist die exakt gleiche wie die bedingungen1 von oben.
//Gleicher Anweisungsblock wie oben 10 Zeilen lang
}

So habe ich es implementiert und es funktioniert. Habe aber 2 mal den gleichen Code. Wie bekomme ich das schöner hin ohne den gleichen Code 2 mal verwenden zu müssen ?

 

[prometheus7]

If bedingung 1 oder 2...
Logisches oder sollte es eigentlich in jeder programmiersprache geben.

Soll die anweisung ausgeführt werden wenn bedingung 1 oder 2 wahr ist?

 

[little_skunk]

Erstmal nochmal dein Orginal etwas besser formatiert:

Bool bVar = true | false
if (Bool == 1) 
{
	if (bedingungen1 && bedingungen2)
		//Anweisungsblock 10 Zeilen lang
} 
else {
	if (bedingungen1) // ist die exakt gleiche wie die bedingungen1 von oben.
		//Gleicher Anweisungsblock wie oben 10 Zeilen lang
}

Damit wird dann auch ersichtlich was das für ein else Zweig ist. Daraus kann man schonmal folgendes machen.

Bool bVar = true | false
if ((Bool == 1 && bedingungen1 && bedingungen2) || (Bool != 1 && bedingungen1)
		//Anweisungsblock 10 Zeilen lang

Das ist aber noch nicht alles. Besser wäre folgendes.

Bool bVar = true | false
if (bedingungen1 && ((Bool == 1 && bedingungen2) || (Bool != 1))
		//Anweisungsblock 10 Zeilen lang

PS: Da ich nicht weiß was bVar sein soll, habe ich Bool und dessen Prüfungen einfach aus deinem Beispiel übernommen. Sollte es eine Boolean Variable sein, solltest du die Prüfung etwas umschreiben.

 

[Blacktron]

Je nachdem was der Code macht könnte es auch sinnvoll sein ihn in eine eigene Methode zu verpacken. Dann kann man ihn evtl. auch an anderen Stellen des Projekts "recyclen".