有下面一些代码:
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++代码中,变量声明与构造函数初始化时候的位置最好保持一致。