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也无关紧要：问问自己：“员工是雇员吗？”因此，重要的是要实现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();

}

注意：我编写的代码无法在“ Turbo C ++”上编译。请放弃旧的错误的编译器，如果使用新的复杂的编译器，则使用它。

错误的附加值

1 个答案: