Refactoring meine if-Anweisung Code

Question

Refactoring meine if-Anweisung Code

stimmen

1

Ich habe mit diesem Stück Code wurde durcheinander über eine Stunde versucht es verschiedene Möglichkeiten, neu zu ordnen. Gibt es einen einfacheren Weg, es zu schreiben?

   if x is not Number      ;// if x is string
   {
      if y is not Number      ;// x, y both strings
      {
         Eval(x)
         Eval(y)
         return
      }
      else                    ;// x is string, y is Number
      {
         Eval(x)
         Scale(y)
         return
      }
   }
   else if y is not Number    ;// x is Number, y is string
   {
      Scale(x)
      Eval(y)
      return
   }
   else                       ;// both are numbers
   {
      Scale(x)
      Scale(y)
      return   
   }

refactoring if-statement pseudocode

Veröffentlicht am 26/08/2010 um 08:59
quelle vom benutzer mikew

In anderen Sprachen...

2 antworten

stimmen

2

// First handle x
if x is Number
{
    Scale(x)
}
else
{
    Eval(x)
}

// Then handle y
if y is Number
{
    Scale(y)
}
else
{
    Eval(y)
}

return

Beantwortet am 26/08/2010 um 09:04
quelle vom benutzer Anders Abel

Chris Schmich · Accepted Answer · 2010-08-26 09:04

Es sieht aus wie Sie wollen EvalStrings und ScaleZahlen. Statt vier explizite Fälle von (die acht mit drei Variablen werden würde), wobei jeder Fall für handhaben xund yunabhängig:

if x is Number
    Scale(x)
else
    Eval(x)

if y is Number
    Scale(y)
else
    Eval(y)

Oder, noch besser, können Sie Push Eval/ Scalein ein Dienstprogramm - Methode:

ScaleOrEval(z):
    if z is Number
        Scale(z)
    else
        Eval(z)

... und dann verwenden, ...

ScaleOrEval(x)
ScaleOrEval(y)

Wenn Sie gute Methodennamen auswählen, dann ein Dienstprogramm Verfahren zu schaffen macht den Code lesbar und helfen Sie, Copy-and-Paste Wiederholungen zu vermeiden.