当前位置: 代码迷 >> 综合 >> warning: when initialized here [-Wreorder]|
  详细解决方案

warning: when initialized here [-Wreorder]|

热度:10   发布时间:2023-12-08 12:30:23.0

        有下面一些代码:

class userMsg {private:uint8_t m_senfitLen = 0;//for filt un-normal datauint8_t m_S_peakorTroughLen = 0;//for sensor wave peaks or troughs arruint8_t unifiedPeakLen = 0;uint8_t unifiedTroughLen = 0;public:SensorQueue s1Que, s2Que, s3Que, s4Que;userMsg(uint8_t _senLen, uint8_t _senfitLen) : s1Que(_senLen), s2Que(_senLen), s3Que(_senLen), s4Que(_senLen), m_senfitLen(_senfitLen) {};~userMsg(){};};

        目前这样看起来是没有问题的。

        但是当我们尝试编译它,会出现以下警告:

        但是当我们运行它,还是可以正常运行。我们知道,代码出现警告是可以运行的。但是警告往往意味着潜在的问题或者diamante的不规范性。而这些就是在一开始就需要避免的。

        那么这是怎么回事呢?

        我们首先来看一下warning定位的地点:

userMsg(uint8_t _senLen, uint8_t _senfitLen) : s1Que(_senLen), s2Que(_senLen), s3Que(_senLen), s4Que(_senLen), m_senfitLen(_senfitLen) {};

        这是我们的构造函数。有两个参数:_senLen,_senFitLen,然后我们先用_senLen初始化了几个对象,然后用_senFitLen初始化了本类中的一个私有变量。这样使用是没错的。那么问题出在哪里呢?

        我们来看一下声明的位置:

        私有变量在前,然后是在public中声明了4个SensorQueue类的对象。然后我们定义userMsg类时,先初始化的是在后面的public中的类对象,然后才是私有的变量。那么问题是不是出在这里?我们做个尝试,代码修改如下:

userMsg(uint8_t _senfitLen, uint8_t _senLen) : m_senfitLen(_senfitLen), s1Que(_senLen), s2Que(_senLen), s3Que(_senLen), s4Que(_senLen) {};

         再次编译,warning消失了。

        我们重新思考,既然是因为声明顺序与使用顺序导致的问题,那么我们将private改到public下面去是不是原来的构造函数编译也没问题了?试一下:

class userMsg {public:SensorQueue s1Que, s2Que, s3Que, s4Que;userMsg(uint8_t _senLen, uint8_t _senfitLen) : s1Que(_senLen), s2Que(_senLen), s3Que(_senLen), s4Que(_senLen), m_senfitLen(_senfitLen) {};~userMsg(){};uint8_t s1tend = 0;private:uint8_t m_senfitLen = 0;//for filt un-normal datauint8_t m_S_peakorTroughLen = 0;//for sensor wave peaks or troughs arruint8_t unifiedPeakLen = 0;uint8_t unifiedTroughLen = 0;};

         如图,猜测是正确的。

        所以我们可以得到结论:

                在C++代码中,变量声明与构造函数初始化时候的位置最好保持一致。

  相关解决方案