C++ 使用 cppcheck 建议的显式

Question

使用强制转换构造函数不好吗？否则为什么代码质量检查器（在我的例子中是 cppcheck）会不断建议在单参数构造函数之前添加显式？

如果我想怎么办

class MyClass {  
 A(int) {}
};

A a = 1;

如果我按照“建议”写

class MyClass {  
 explicit A(int) {}
};

A a = 1;

会引发错误，但如果我使用第一个，我会收到一条警告，提示我必须记录才能通过代码审查。

Answer 1

C++ Core Guidelines

C.46: By default, declare single-argument constructors explicit

Reason

To avoid unintended conversions.

Example, bad
class String {
public:
    String(int);   // BAD
    // ...
};

String s = 10;   // surprise: string of size 10
Exception

If you really want an implicit conversion from the constructor argument type to the class type, don't use explicit:
class Complex {
public:
    Complex(double d);   // OK: we want a conversion from d to {d, 0}
    // ...
};

Complex z = 10.7;   // unsurprising conversion
See also: Discussion of implicit conversions

Answer 2

这种隐式的class类型转换很容易被无意间使用。使用此转换构造函数，每个接受 MyClass 作为参数的函数或成员函数也将接受 int。因此，传递给此类函数的每个 int 都将转换为临时 MyClass，在函数完成后将被丢弃。可能不是你想要的。

C++ 使用 cppcheck 建议的显式

C++ use of explicit suggested by cppcheck

c++

explicit

explicit-constructor

cppcheck