Question

我一直在学习一些cpp练习来学习cpp，而且我遇到了一个问题。

我创建了一个名为＆＃34; FixedPoint2＆＃34;实现一个固定的点数到2位小数。我已经包含了头文件，其中包含以下所有功能。

我正在努力的是，当我尝试测试相等运算符时，我总是得到一个错误的答案。换句话说，会发生以下情况：

cout << (FixedPoint2(1.0)==FixedPoint2(1.0)) << endl; //returns true as expected
cout << (FixedPoint2(1.2)==FixedPoint2(1.2)) << endl; //returns false
cout << FixedPoint2(1.2) << "\t" << FixedPoint2(1.2) << endl; returns 1.2 1.2

所以你明白了。我还用if语句测试了它，以确保我的重载调用不是问题。例如：

if (FixedPoint2(4.5)==FixedPoint2(4.5)) 
    cout << "Don't post to stackoverflow"; //This doesn't print

我的直觉告诉我一些事情与我忽略的一些隐式类型转换，或者双重内容中的一些混乱事物有关。但我不认为其中任何一个就是它。

using namespace std;


class FixedPoint2
{
    private:
        int16_t m_digi; //chosen because I want the range
        int8_t m_deci; //chosen to optimise memory 
    public:
        FixedPoint2(int16_t digi = 0, int8_t deci = 0):m_digi{digi}, m_deci{deci}
        {
            assert(!(deci>127 || deci<-127)); //to prevent overflows
            if(deci<-100 || deci>100) //just in case some moron (me) does some weird decimal calculations 
            {
                m_digi+=(static_cast<int16_t>(deci)/100);
                m_deci+=(-100);
            }
        }
    FixedPoint2(double mydouble) 
    {
        if (mydouble>=0) //The if-else floors the absolute value of the integer base
        {
            m_digi=static_cast<int16_t>(floor(mydouble));
        }
        else
        {
            m_digi=static_cast<int16_t>(floor(mydouble)+1);
        }
        m_deci=static_cast<int8_t>(round(100*(mydouble-m_digi))); //This rounds off the decimal to two digits

    };

    operator double();

    friend ostream& operator<<(ostream &out, const FixedPoint2 &f1);
    friend istream& operator>>(istream &in, FixedPoint2 &f1);
    friend FixedPoint2 operator+(const FixedPoint2 &f1, const FixedPoint2 &f2);
};

FixedPoint2::operator double()
{
    double base= static_cast<double>(m_digi);
    double decimal= static_cast<double>(m_deci);
    return base+0.01*decimal;
}

ostream& operator<<(ostream &out, const FixedPoint2 &f1)
{
    FixedPoint2 a=f1;
    out << double(a); //this is an easy work around to handling the period placement for the fixed point number
    return out;
}


istream& operator>>(istream &in, FixedPoint2 &f1)
{
    double placeholder;
    in>>placeholder;
    f1=FixedPoint2(placeholder);
    return in;
}

FixedPoint2 operator+(const FixedPoint2 &f1, const FixedPoint2 &f2)
{
    return FixedPoint2(f1.m_digi+f2.m_digi, f1.m_deci+f2.m_deci);
}

Answer 1

编译器不会自动生成operator==，因为赔率确实很好，它会比正确的情况更频繁地弄错。让我们举一个简单的例子：一个动态字符串。编译器会生成逐字符比较的代码，对吧？什么时候停止？现在，编译器需要更多地了解程序员对其字符串的意图，并且编译器不需要心灵感应接口的额外复杂性。

最好拥有一致的规则，不要强制明确定义要比较的内容，而不是由于人们假设他们得到了他们想要的东西而导致的垃圾代码。这里有关于该主题的更长时间的讨论：Why don't C++ compilers define operator== and operator!=?

编译器寻找满足比较的方法。它没有找到直接operator==，但确实发现operator double和double可用于比较。除非有时他们不能：Is floating point math broken?

那说我无法重现OP的结果。我希望完全相同的公式在完全相同的输入上执行以获得完全相同的结果，即使结果是1.199999 ...而不是1.2

虽然我无法重现，但OP仍然可以更好地实现固定点operator==，因为没有固定点数的不精确性。固定点将是相等或不相等，没有ifs ands或buts，所以＆＃34;没有什么可以出错！＆＃34;另外，这个操作符应该是无关紧要的。像return (rhs.m_digi == lhs.m_digi) && (rhs.m_deci == lhs.m_deci);

这样的东西

我班级中的相等运算符（==）不起作用

1 个答案: