convertito char in operatore aritmetico

Question

Ho un supporto char per un operatore, ci sono quattro operatori (+ - * /) in totale.

come ho fatto:

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; 
} 

C'è un modo più conveniente per fare questo?

Answer 1

è possibile utilizzare un'istruzione switch:

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

In primo luogo, la sintassi

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

è sbagliato, e dovrebbe essere

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

secondo luogo, il parametro a può avvenire solo 4 valori discreti, così buona pratica è quello di utilizzare un'enumerazione, ad esempio

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; 
} 

che assicura che l'utente del Compute utilizza solo uno di questi quattro valori per l'operazione (a) parametro.

Probabilmente non ho utilizzato le migliori pratiche nel mio esempio, quindi vi consiglio di leggere this answer per maggiori dettagli.

Infine si potrebbe rendere il codice più conciso utilizzando un'istruzione switch:

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; 
}