refactor min hvis setningen kode

Question

refactor min hvis setningen kode

stemmer

1

Jeg har vært rote dette bit av koden med i over en time å prøve å ordne det forskjellige måter. Er det noen enklere måte å skrive det?

   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

Publisert på 26/08/2010 klokken 07:59
kilden bruker mikew

På andre språk...

2 svar

stemmer

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

Svarte 26/08/2010 kl. 08:04
kilden bruker Anders Abel

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

Det ser ut som du ønsker å Evalstrenger og Scaletall. I stedet for å ha fire eksplisitte tilfeller (som skulle bli åtte med tre variabler), håndtere hvert tilfelle for xog yuavhengig:

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

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

Eller enda bedre, kan du presse Eval/ Scaleinn i et verktøy metode:

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

... og deretter bruke den ...

ScaleOrEval(x)
ScaleOrEval(y)

Hvis du velger gode metodenavn, og deretter lage et verktøy metoden gjør koden mer lesbar og hjelper deg å unngå kopier og lim repetisjon.