Vermeiden 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 ?

 

[noodyn]

indem du den Codeblock in eine eigene Methode baust und dann an verschiedenen Stellen einfach nur die Methode mit Parametern aufrufst.

 

[oneOeight]

indem du deine if abfrage verbesserst.
wenn bei beiden bedingungen der gleiche code ausgeführt wird, kannst die ja auch verknüpfen.
if (bedingung1 || (bedingung1 && bedingung2))

 

[rev]

indem du deine if abfrage verbesserst.
wenn bei beiden bedingungen der gleiche code ausgeführt wird, kannst die ja auch verknüpfen.
if (bedingung1 || (bedingung1 && bedingung2))

Und plötzlich merkt man, dass man Bedingung2 auch komplett weglassen kann

 

[aeska]

Nur so am Rande: Muss es nicht heißen if bVar ==1 und nicht if Bool == 1? Weil Bool ist ja nur der Datentyp. Jedenfalls ist das bei c++ so. Dann könnte man auch nur bool bVar; schreiben da eine Bool ja sowieso nur true oder false sein kann.

 

[oneOeight]

Nur so am Rande: Muss es nicht heißen if bVar ==1 und nicht if Bool == 1? Weil Bool ist ja nur der Datentyp. Jedenfalls ist das bei c++ so.

ja, das siehst du schon richtig.
ich denke mal beim pseudo code schreiben für hier hat er nicht aufgepasst.

 

[roedert]

indem du deine if abfrage verbesserst.
wenn bei beiden bedingungen der gleiche code ausgeführt wird, kannst die ja auch verknüpfen.
if (bedingung1 || (bedingung1 && bedingung2))

Da hast du das if/else mit dem bVar==1 aber komplett ignoriert..... kommt davon, wenn man Quelltext nicht vernünftig mit Einrückungen formatiert

Richtig wäre wohl
if ((bVar == 1 && bedingungen1 && bedingungen2) || (bVar == 0 && bedingungen1)) .........

 

[oneOeight]

Da hast du das if/else mit dem bVar==1 aber komplett ignoriert..... kommt davon, wenn man Quelltext nicht vernünftig mit Einrückungen formatiert

ja, hast recht, hab ich übersehen oder direkt wegen Bool == 1 schon ausgeblendet gehabt.

 

[iCode]

Wie wäre es mal mit dem Vermeiden von doppelten Posts: Vermeiden von doppelten Code.

 

[oneOeight]

Wie wäre es mal mit dem Vermeiden von doppelten Posts: Vermeiden von doppelten Code.

dort wird auch noch auf ein anderes forum verlinkt, also eher tripple

 

[madu]

Wie wäre es mal mit dem Vermeiden von doppelten Posts: Vermeiden von doppelten Code.

Doppelt ginge ja noch

 

[aXiEd]

Edit, hat sich erkledigt

 

[iCode]

Klasse! Echt gar keine Manieren mehr die Leute. :d

 

[iCode]

dort wird auch noch auf ein anderes forum verlinkt, also eher tripple

Jetzt schon ein Quadrupel:
http://www.hardwareluxx.de/community/f24/vermeidung-von-doppelten-code-1019273.html

Schön das er verstanden hat, dass man Redundanzen vermeidet.
Und wann fängt er damit an? :d