aritmetik işlecine dönüşümü dönüştürme

Question

Operatör için bir char standım var, toplamda dört operatör (+ - * /) var.

yaptım nasıl: Bunu yapmak için daha kolay bir yoludur

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

var mı?

Answer 1

Bir switch deyimi kullanabilirsiniz:

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

Öncelikle

,

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

yanlıştır sözdizimi ve

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

İkincisi, sizin a parametre yalnızca 4 ayrı değerler alabilir olmalıdır, bu nedenle iyi bir uygulama, örneğin bir numaralandırma kullanmaktır Compute kullanıcı, sadece operasyon (a) parametre için bu dört değerden birine kullanımı garanti eder

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

.

Muhtemelen örneğimdeki en iyi uygulamaları kullanmadım, bu yüzden daha fazla ayrıntı için this answer numaralı telefonu okumanızı tavsiye ediyorum.

Son olarak bir switch deyimi kullanarak kod daha özlü yapabilir:

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