c++ 类，这可行，但有人可以改进它，使其符合标准/编码实践

Question

#include <iostream>

using namespace std; 




class Calculator
{
        public: 
          int number1; 
          int number2 ; 

        public: 

        void  setCalcNumbers(int input1 , int input2 )
          {

          number1 = input1; 
          number2 = input2; 
          }
          int addNumber()
          {
              return number1 + number2;
          }
          int subtractNumber()
          {
              return number1 - number2;
          }
          int divideNumber()
          {
              return number1 / number2;
          }
          int multiplyNumber()
          {
              return number1 * number2;
          }

};

int main()
{
    int numberInput1 = 0; 
    cout << "Enter number 1: "; 
    cin >> numberInput1; 


    int numberInput2 = 0;
    cout << "Enter number 2: "; 
    cin >> numberInput2;

    Calculator t; 
    t.setCalcNumbers(numberInput1, numberInput2);

    char userOperationChoice; 
    cout << "which operation would you like to perform? "
         << " , enter M for Multiplication, D for Division, A for addition or S for Subtraction:" << endl;

    cin >> userOperationChoice; 
    char a,d,m,s;
    switch (userOperationChoice) 
    {

    case 'a'  : 
         t.addNumber();
         cout << "the total is: " << t.addNumber() << endl; 
         break; 

    case 's':
         t.subtractNumber(); 
         cout << "the total is: " << t.subtractNumber() << endl; 
         break ; 
    case 'd': 
         t.divideNumber();
         cout << "the total is: " << t.divideNumber() << endl; 
         break; 
    case 'm':
         t.multiplyNumber();
         cout << "the total is: " << t.multiplyNumber() << endl; 
         break; 

    }        



    system("pause"); 
    return 0 ; 
}

Answer 1

不确定这是不是作业，所以我会给出额外的简短版本。 特别是，我会去 function template路径。

template <typename T> T  add(T n1,T n2){                                        
        return n1 + n2; 
}

并使用它:

 int s = add<int> (1,3); 

考虑到您询问的是类，我会删除 setNumbers 函数，并将每个函数设为静态，并接收要操作的数字作为参数。此外，我会将其设为 template class .

为什么类型很重要？好吧，现在您的计算器只能对整数进行运算。那么如果用户输入 float ，比如 1.5 和 3.2 会发生什么？当您使用 int 类型时，所选操作将介于数字 1 和 3 之间，因为它们被截断了。还不错吧？好吧，考虑一下这个案例。用户输入数字3和0.5，选择分割。会发生什么 ？是的，黑洞发生器:除以零。因此，模板类/函数 是合适的方式。如果您只是在寻找整数计算器，我建议将该类重命名为 integerCalculator 并对用户输入进行一些验证(以避免 fatal error ，例如更优雅地除以零)。

例如

模板类

 template <typename T> calculator {
      static T add (T n1 ,T n2){
        return n1 + n2;
      }
    };

整数计算器

class integerCalculator{
  public:
          //other functions

          int divideNumber(){
                 if (number2 == 0) //handle error
                 return number1/number2;
          }
};

关于您的版本的更多信息:

number1 和 number2 不应公开，或者如果是，为什么需要 setNumber() 函数？降低他们的知名度(private 似乎很合适)。

class Calculator{
        int n1;
        int n2;

   public:  void setNumbers(int n1,int n2){...}
   //add, divide, multiply, subtract functions
}

a、d、m、s 变量似乎未使用。如果它们不是解决方案的一部分...