输出提供整数的大小而不是实际的整数

Question

// 创建名为 Class1 和 Class2 的类，每个类都有一个私有成员。添加成员函数以在每个类上设置一个值（比如 setValue）。再添加一个对两个类都友好的函数 max()，max() 函数应该比较两个类的两个私有成员并显示它们之间的最大值。为每个类创建一个对象并为其设置一个值。显示其中的最大个数

#include <iostream>
using namespace std;

class Class1
{
    int a;

    public:
        int setValue();
    
    friend int max();
};

class Class2
{
    int b;

    public:
        int setValue();

    friend int max();
};

int Class1::setValue()
{
    cout<<"Enter the first number:";
    cin>>a;
}


int Class2::setValue()
{
    cout<<"Enter the second number:";
    cin>>b;
}

int max()
{
    Class1 c1;
    Class2 c2;

    if(c1.a>c2.b)
    {
        cout<<"Greater number: "<<c1.a<<endl;
    }

    else
    {
        cout<<"Greater number: "<<c2.b<<endl;
    }
}

int main()
{
    Class1 obj1;
    Class2 obj2;

    obj1.setValue();
    obj2.setValue();
    cout<<endl;
    max();

    return 0;
}

Answer 1

您应该尝试传递要比较的Class1 和Class2 对象，而不是在max 函数内创建新的本地Class1 和Class2 对象。

试试这样的方法：

#include <iostream>
using namespace std;

class Class1;
class Class2;

class Class1
{
    int a;

    public:
        void setValue();
    
    friend int max(Class1 c1, Class2 c2);
};

class Class2
{
    int b;

    public:
        void setValue();

    friend int max(Class1 c1, Class2 c2);
};

void Class1::setValue()
{
    cout<<"Enter the first number:";
    cin>>a;
}


void Class2::setValue()
{
    cout<<"Enter the second number:";
    cin>>b;
}

int max(Class1 c1, Class2 c2)
{
    int maxValue;
    
    if(c1.a>c2.b)
    {
        cout<<"Greater number: "<<c1.a<<endl;
        maxValue = c1.a;
    }

    else
    {
        cout<<"Greater number: "<<c2.b<<endl;
        maxValue = c2.b;
    }

    return maxValue;
}

int main()
{
    Class1 obj1;
    Class2 obj2;

    obj1.setValue();
    obj2.setValue();
    cout<<endl;
    int maxValue = max(obj1, obj2);

    return 0;
}

Answer 2

我发现您的代码存在一些差异。首先，你想在setValue() 方法中返回什么。您已将返回类型 int 而不是 void 设置为那些肯定会给出错误的 setValue() 方法。找到最大值的友元函数还需要返回一个int，正如您在max() 的原型中声明的那样。我相信你不应该从setValue() 方法返回任何东西。

---

另一个你做错了的事情是：

Class1 c1;
Class2 c2;

在max 方法内部，而不是将对象作为参数传递给max 方法。 您尚未将对象传递给 max() 方法的参数。在max 方法内创建新的class1 和class2 对象将无济于事。同时，max 方法的输出将是未定义的。

---

解决方案：

#include <iostream>
using namespace std;

class Class1
{
    int a;

    public:
        void setValue();
    
    friend int max(Class1 obj1, Class2 obj2);
};

class Class2
{
    int b;

    public:
        void setValue();

    friend int max(Class1 obj1, Class2 obj2);
};

void Class1::setValue()
{
    cout<<"Enter the first number:";
    cin>>a; // this sets the value of private data-member a.
}


void Class2::setValue()
{
    cout<<"Enter the second number:";
    cin>>b; // this sets the value of private data-member b.
}

int max(Class1 obj1, Class2 obj2)
{

    int max;


    if(obj1.a>=obj2.b)
    {
        cout<<"Greater number: "<<obj1.a<<endl;
        max=obj1.a;
    }

    else
    {
        cout<<"Greater number: "<<obj2.b<<endl;
        max=obj2.b;
    }
    return max;
}

int main()
{
    Class1 obj1;
    Class2 obj2;

    obj1.setValue();
    obj2.setValue();
    cout<<endl;
    int maximum=max(obj1, obj2);
    cout<<"Maximum is: "<< maximum<< endl;
    return 0;
}

希望，这会有所帮助。 :-)

Answer 3

您已声明 max 以及两个成员函数 setValue 返回 int。这些函数执行到最后而不返回值。程序的行为未定义。

解决方案：要么返回一个值，要么声明函数返回void。

---

这里：

Class1 c1;
Class2 c2;

您默认初始化两个变量。这些对象的成员将被默认初始化，在 int 的情况下意味着它们具有不确定的值。

在这一行：

if(c1.a>c2.b)

您将不确定的值与另一个不确定的值进行比较。输出更大的值时也是如此。程序的行为未定义。

解决方案：初始化您使用其值的所有对象。在这种情况下，我怀疑您不应该创建新的、未初始化的变量，而是要读取在main 中创建的现有对象。通常，这是通过将值作为参数传递给函数来实现的。