konvertieren Char zu arithmetischen Operator

Question

Ich habe ein Zeichen stehen für einen Operator, gibt es vier Operatoren (+ - * /) insgesamt.

Wie ich tue:

int Compute(char c, int a, int b) 
{ 
    if(c == '+') 
     return a+b; 
    else if(c == '-')  
     return a-b; 
    else if(c == '*') 
     return a*b; 
    else (c == '/') 
     return a/b; 
} 

Gibt es einen bequemen Weg, dies zu tun?

Answer 1

Sie eine switch-Anweisung verwenden:

int Compute(char c, int a, int b) 
{ 
    switch (c) { 
    case '+': return a+b; 
    case '-': return a-b; 
    case '*': return a*b; 
    case '/': return a/b; 
    default: throw std::runtime_error("No such operator"); 
    } 
} 

Answer 2

Zunächst wird die Syntax

else (a == '/') 
    return a/b; 

falsch ist, und sollte

else if (a == '/') 
    return a/b; 

Zweitens sein, Ihre a Parameter nur vier diskrete Werte annehmen kann, so gute Praxis ist eine Aufzählung zu verwenden, z.B.

enum Operation 
{ 
    kAddition, 
    kSubtraction, 
    kMultiplication, 
    kDivision 
}; 

int Compute(Operation a, int a, int b) 
{ 
    if (a == kAddition) 
     return a+b; 
    else if (a == kSubtraction)  
     return a-b; 
    else if (a == kMultiplication) 
     return a*b; 
    else if (a == kDivision) 
     return a/b; 
} 

, die sicherstellt, dass der Benutzer von Compute nur eine dieser vier Werte für die Operation (a) Parameter verwenden wird.

Ich habe wahrscheinlich keine Best Practices in meinem Beispiel verwendet, daher empfehle ich Ihnen, lesen Sie this answer für weitere Details.

Schließlich könnten Sie den Code übersichtlicher machen, indem ein Schalter-Anweisung:

enum Operation 
{ 
    kAddition, 
    kSubtraction, 
    kMultiplication, 
    kDivision 
}; 

int Compute(Operation a, int a, int b) 
{ 
    switch (a) 
    { 
     case kAddition: 
      return a+b; 
     case kSubtraction: 
      return a-b; 
     case kMultiplication: 
      return a*b; 
     case kDivision: 
      return a/b; 
    } 
} 

Answer 3

int ComputeByChar(char a, int c, int b) 
{ 
    switch(a) 
    { 
    case '+': return c+b; 
    case '-': return c-b; 
    case '/': return c/b; 
    case '*': return c*b; 
    default: cout<< "Invalid"; 
     break; 
    } 
    return 0; 
}