显式构造函数c ++中的特殊运算符[重复]

Question

我是 C++ 新手，我正在练习自己的类和对象。我制作了一个包含运算符重载的程序，它有一个“显式构造函数”。我已经尝试了很多不同的值并且它工作正常，但是我的显式构造函数与我老师的回答有点不同。 老师的回答：

class Ventilator {
int maxrot;
int rot;
int vert;
bool horiz;
public:
explicit Ventilator(int maxrot=500, int rot=0, int vert=0, int horiz=false): maxrot(abs(maxrot)),
   rot(rot<0?0:rot>abs(maxrot)?abs(maxrot):rot), vert(vert<-15||vert>15?0:vert),
   horiz(horiz)  { }

我的是：

class Ventillator{
 int rotation;
 int vertikal;
 int maxrot;
int horiz;
public:
explicit Ventillator(int rotation = 10, int maxrot=100, int vertikal = 11, bool horiz = true):
    rotation(abs(rotation)), maxrot(abs(maxrot)), vertikal(abs(vertikal)), horiz(horiz){

    if (this->rotation > this->maxrot) this->rotation=this->maxrot;
    if (this->rotation < 0) this->rotation = 0;
    if (this->vertikal <= 15 && this->vertikal >=-15) this->vertikal=vertikal;
    else
        this->vertikal = 0;
    }

这两种方式不完全一样吗？他不就是为了让代码更紧凑而使用特殊的运算符吗？

Answer 1

不，有一些区别，只是意识形态上的。 你的解决方案更好，因为它更具可读性——你可以教你的老师如何编写漂亮的代码:)

另外，你可以这样重写代码：

#include <algorithm>
class Ventillator
{
    int rotation;
    int vertikal;
    int maxrot;
    int horiz;
    inline int place_in_range(int num, int max, int min)
    {
        return ( (std::max) ( ( (std::min)(max, num) ), min));
    }
    inline bool is_in_range(int num, int max, int min)
    {
        return num <= max && min >= num;
    }

public:
    explicit Ventillator(int rotation = 10, int maxrot=100, int vertikal = 11, bool horiz = true):
        rotation(place_in_range(abs(rotation), maxrot, 0)), maxrot(abs(maxrot)), vertikal( (is_in_range(vertikal, 15, -15) ? abs(vertical) : 0 )), horiz(horiz)
    {
    }
};

Answer 2

这两个例子在这种情况下达到了相同的效果。仅使用初始化列表（我认为）通常更优雅，因为

• 表示成员可以是 const
• 成员立即初始化为正确的值。如果另一个成员初始化程序或构造函数主体可以抛出，这可能很重要。

但是，我认为带有两个三元运算符 (?:) 的表达式可能难以阅读 - 我可能会编写一个辅助函数来返回值以初始化 rotation。