Неверные добавочные значения

Question

В этой программе я принимаю некоторые значения и просто добавляю их.Программа принимает значения, но показывает неправильное добавление.

#include<iostream.h>
#include<conio.h>
class Employee
{
    char name[30];
    int id;
    public:
    void Getdata()
    {
        cout<<"Enter Name: ";
        cin>>name;
        cout<<"Enter ID: ";
        cin>>id;
    }
    Employee()
    {
    }
    void Putdata()
    {
        cout<<name<<endl;
        cout<<id<<endl;
    }
};
class Salary : public Employee
{
    int sal;
        int basic, hra, da, cla;
    public:
    void set()
    {
        cout<<"Enter Basic Pay: ";
        cin>>basic;
        cout<<"Enter HRA: ";
        cin>>hra;
        cout<<"Enter Da: ";
        cin>>da;
        cout<<"Enter CLA: ";
        cin>>cla;
    }
    Salary() : Employee()
    {
        sal = (basic+hra+da+cla) ;
    }
    void show()
    {
        cout<<"Salary: "<<sal;
    }
};
void main()
{
    clrscr();
    Salary s;
    s.Getdata();
    s.set();
    s.Putdata();
    s.show();
    getch();
}

Я ожидал, что зарплата равна 13000, но вместо этого получаю зарплату: 11172. В результате.

Answer 1

Вы вычисляете значение sal из неинициализированных элементов в конструкторе по умолчанию: sal = (basic + hra + da + cla);. Таким образом, ваша программа выполняет неопределенное поведение.

Это потому, что конструктор работает перед любым другим членом. Чтобы решить эту проблему, вы должны сначала получить значения, а затем рассчитать следующее:

Salary() : Employee(){
//  cout << basic << ", " << hra << ", " << da << ", " <<  cla << endl; // this line proves that hra, basic... are not initialized.
    set(); // assign values before calculate.
    sal = (basic + hra + da + cla);
}

• Я рекомендую удалить эту строку из конструктора по умолчанию: sal = (basic + hra + da + cla); и поместить ее в set. так set() будет выглядеть так:

  void set(){
      cout << "Enter Basic Pay: ";
      cin >> basic;
      cout << "Enter HRA: ";
      cin >> hra;
      cout << "Enter Da: ";
      cin >> da;
      cout << "Enter CLA: ";
      cin >> cla;
      sal = (basic + hra + da + cla); // after getting values it is a good place here to process the sal.
  }
  

• Еще одна вещь, которую следует учитывать, - main должна возвращать целое число, а не void Возвращение void неверно.

• Также не используйте символьные строки, но используйте std::string, которое стоит использовать:

  class Employee{
      std::string name;
      int id;
      // ...
  };
  

• Кроме того, получение Salary из Employee не имеет значения: Спросите себя: "Является ли salray служащим?" Таким образом, уместным является достижение отношения Has-a вместо Is-a, потому что у сотрудника есть зарплата, но он не является наемником.

  class Salary {
      public:
          Salary() :
              sal{}, basic{}, hra{}, da{}, cla{} {
          }
          void set() {
              std::cout << "Enter Basic Pay: ";
              std::cin >> basic;
              std::cout << "Enter HRA: ";
              std::cin >> hra;
              std::cout << "Enter Da: ";
              std::cin >> da;
              std::cout << "Enter CLA: ";
              std::cin >> cla;
              sal = basic + hra + da + cla;
          }
          void show() {
              std::cout << "Salary: " << sal << std::endl;
          }
      private:
          int sal, basic, hra, da, cla;
  };
  
  class Employee {
      public:
          Employee() :
              name{}, id{}, sal{} {
          }
          void Getdata() {
              std::cout << "Enter Name: ";
              std::getline(std::cin, name);
              std::cout << "Enter ID: ";
              std::cin >> id;
              sal.set();
          }
          void Putdata() {
              std::cout << name << std::endl;
              std::cout << id << std::endl;
              sal.show();
          }
      private:
          std::string name;
          int id;
          Salary sal;
  };
  
  int main(){
      Employee emp{};
      emp.Getdata();
      emp.Putdata();
  
  }
  

• NB. Код, который я написал, не компилируется в "Turbo C ++". Пожалуйста, откажитесь от этого старого и ошибочного компилятора и используйте его, если он новый.