如何避免关于未初始化变量的整洁警告

Question

我正在寻找使用clang-tidy 避免cppcoreguidelines-init-variables 警告的最佳方法。

std::istringstream in(line);
float x, y, z, rotx, roty, rotz;
in >> x >> y >> z >> rotx >> roty >> rotz;

-> 变量“x”未初始化，变量“y”未初始化，...

double min_intensity, max_intensity;    
cv::Point point_min, point_max;
cv::minMaxLoc(input, &min_intensity, &max_intensity, &point_min, &point_max, elliptic_mask_around_center);

-> 变量 'min_intensity' 未初始化，...

我可以为x、y、...、min_intensity 和max_intensity 指定一个值，但我不明白为什么要这样做。

避免此警告的最佳方法是什么？

float x = 0.F; // initialize to a value, but 6 lines, and it makes no sense because x is never 0
float x = NAN; // works but 6 lines, and needs <cmath> inclusion
float x, y, z, rotx, roty, rotz; // NOLINT [not very practical, and means the warning is a false positive]

感谢您的帮助。

Answer 1

一般而言，您应该始终初始化变量，因为从未初始化的变量中读取是未定义的行为。

当从流中提取失败时，从 C++11 开始分配 0。但是，只有当流不处于失败状态时才会出现这种情况。因此，当读取 x 失败时，x 的值将是 0，但其他的值将是不确定的。

您应该经常检查提取是否成功：

if (in >> x >> y >> z >> rotx >> roty >> rotz) { 
     // use x,y,z,rotx,roty,rotz
}

如果你想使用部分输入，即使在某些时候提取失败，你应该单独阅读它们：

if (in >> x) {
     // use x
}
if (in >> y) {
     // use y
}
// ...

如果你这样做，初始化严格来说是没有必要的。但是，从流中读取比初始化变量要昂贵得多，因此为了保持一致（参见例如here），我建议无论如何都要初始化它们。不要担心更多的代码行，一个常见的建议是无论如何每行声明一个变量（参见例如here）：

std::istringstream in(line);
float x = 0.0;
float y = 0.0;
float rotx = 0.0;
float roty = 0.0;
float rotz = 0.0;
if (in >> x >> y >> z >> rotx >> roty >> rotz) {
     // extraction succeeded
} else {
     // extraction failed
}