Vermeiden von doppelten Code

Bullz · 29. April 2014, 07:09

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

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

Quellcode

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 ?

gritsch · 29. April 2014, 07:22

1. schlechter variablenname
2. musst du bVar überprüfen und nicht Bool
3. boolsche variablen nicht mit 1 oder 0 vergleichen sondern einfach nur if (boolvar) oder if (!boolvar)
4. ist das obj-c? dann verwende BOOL
5. jetzt zur eigentlichen frage: if ((bVar && bedingungen1 && bedingungen2) || (!bVar && bedingungen1) { DEINE 10 zeilen code }

MarkusM · 29. April 2014, 08:38

Wenn ich mir den Source Code so anschaue, dann glaube ich dir, dass dir zu dem Gedanke dahinter die Worte fehlen.

Lösungen gibt es mehrere. gritsch hat schon eine aufgezeigt.

Du könntest deinen Anweisungsblock in eine Hilfsmethode packen und diese dann aufrufen. Die Logik kann auch noch umgeformt werden. Die Ausführung deines Anweisungsblock hängt direkt von bedingung1 ab. Ist bedingung1 = false dann ist der Wert von bVar und bedingung2 irrelevant.

Da ist mehr als ein Bock drin.

macmoonshine · 29. April 2014, 09:03

Bullz schrieb:

Wie bekomme ich das schöner hin ohne den gleichen Code 2 mal verwenden zu müssen ?

Unabhängig von Deinem Code und den genannten Lösungen solltest Du Dich vielleicht mal mit Refactoring beschäftigen. Das geht genau auf solche Problemstellungen ein und das Buch von Fowler ist sehr lesenswert.

Michael · 29. April 2014, 09:57

Hach, noch so ein Crossposter.

mattik · 29. April 2014, 10:20

gritsch schrieb:

if ((bVar && bedingungen1 && bedingungen2) || (!bVar && bedingungen1)

Oder kurz: if (bedingungen1 && (bedingungen2 || !bVar))

gritsch · 29. April 2014, 10:25

mattik schrieb:

gritsch schrieb:

if ((bVar && bedingungen1 && bedingungen2) || (!bVar && bedingungen1)

Oder kurz: if (bedingungen1 && (bedingungen2 || !bVar))

jep klar, ich wollts nur so klar wie mäglich halten ;-)

NSObject · 29. April 2014, 10:43

Michael schrieb:

Hach, noch so ein Crossposter.

Da gibt es noch mehr: macuser.de/forum/thema/701630-Vermeiden-von-doppelten-Code

Michael · 29. April 2014, 11:10

Und in keinem der vier "belästigten" Foren auch nur eine Reaktion des Threaderstellers.

NSObject · 29. April 2014, 11:35

Michael schrieb:

Und in keinem der vier "belästigten" Foren auch nur eine Reaktion des Threaderstellers.

Vier?

macmoonshine · 29. April 2014, 11:38

Michael schrieb:

Und in keinem der vier "belästigten" Foren auch nur eine Reaktion des Threaderstellers.

Was für ein Schuft.

Michael · 29. April 2014, 12:22

NSObject schrieb:

Michael schrieb:

Und in keinem der vier "belästigten" Foren auch nur eine Reaktion des Threaderstellers.

Vier?

Mindestens.

macmoonshine · 29. April 2014, 12:38

Der OP hat also doppelten Code als Attitüde.

AppleDeveloper · 29. April 2014, 12:47

5 mal: codekicker.de/fragen/Vermeiden-doppelten-Code

Osxer · 29. April 2014, 18:31

Mir wäre der Aufwand zu groß, bei so einer Frage 5 mal zu posten...
Und die besten Antworten gibt es doch eh hier...

Thallius · 29. April 2014, 19:03

Hihi, -4 lasst uns einen neuen Negativ-Rekord setzen

Gruß

Claus

gritsch · 29. April 2014, 19:05

Thallius schrieb:

Hihi, -4 lasst uns einen neuen Negativ-Rekord setzen

Gruß

Claus

erledigt

John1994 · 29. April 2014, 23:00

Osxer schrieb:

Mir wäre der Aufwand zu groß, bei so einer Frage 5 mal zu posten...
Und die besten Antworten gibt es doch eh hier...

Naja Copy&Paste
Bleiben nur noch die ganzen Anmeldungen

nussratte · 29. April 2014, 23:55

Wenn man sicher gehen muss das irgendeiner die Hausaufgaben zu morgen fertig macht , dann kann man auch mal Massen copy Paste machen

macmoonshine · 30. April 2014, 08:26

Vielleicht ist es ja auch ein Kunstprojekt, das auf die normative Redundanz in der postdemokratischen Gesellschaft hinweisen soll. Oder der OP hat gar kein Problem mit seinem Quälcode, sondern möchte von uns eigentlich wissen, wie er die vielen Posts zu einem zusammenfassen kann.

Quellcode

Ähnliche Themen