依靠隐式参数转换是否被认为是危险的？

C ++具有一项功能（我无法弄清楚它的正确名称），如果参数类型不是期望的参数类型，它将自动调用参数类型的匹配构造函数。

一个非常基本的示例就是调用一个std::string带const char*参数的函数。编译器将自动生成代码以调用适当的std::string构造函数。

我想知道，它是否像我认为的那样对可读性不利？

这是一个例子：

class Texture {
public:
    Texture(const std::string& imageFile);
};

class Renderer {
public:
    void Draw(const Texture& texture);
};

Renderer renderer;
std::string path = "foo.png";
renderer.Draw(path);

这样好吗 还是太过分了？如果我不应该这样做，我可以以某种方式让Clang或GCC发出警告吗？

这称为转换构造函数（或有时称为隐式构造函数或隐式转换）。

我不知道会在发生这种情况时发出警告的编译时开关，但是很容易防止。只需使用explicit关键字。

class Texture {
public:
    explicit Texture(const std::string& imageFile);
};

至于转换构造函数是否是一个好主意：这取决于。

隐式转换有意义的情况：

• 该类足够便宜，以至于您不需要关心它是否是隐式构造的。
• 有些类在概念上与它们的参数相似（例如，std::string反映了与其const char *隐式转换的概念相同的概念），因此隐式转换是有意义的。
• 如果禁用了隐式转换，则某些类将变得更加令人讨厌。（考虑到每次要传递字符串文字时，都必须显式调用std :: string。Boost的各个部分相似。）

隐式转换的意义较弱的情况：

• 构造非常昂贵（例如您的Texture示例，该示例需要加载和解析图形文件）。
• 从概念上讲，类与它们的参数非常不同。例如，考虑一个以其大小作为参数的类似数组的容器：

    类FlagList
    {
        FlagList（int initial_size）; 
    };

    无效的SetFlags（const FlagList＆flag_list）;

    int main（）{
        //现在可以编译，即使它并不明显
        //它在做什么。
        SetFlags（42）;
    }

• 施工可能会产生有害的副作用。例如，AnsiString类应该不隐式地从一个构建体UnicodeString中，由于Unicode到ANSI转换可能会丢失信息。

进一步阅读：

• 有关显式构造函数的C ++常见问题解答
• 在谷歌C ++风格指南说要几乎总是使用显式构造函数。
• 这个StackOverflow问题有更多利弊。

这更多的是评论，而不是答案，但是太大了，无法发表评论。

有趣的是，g++不允许我这样做：

#include <iostream>
#include <string>

class Texture {
        public:
                Texture(const std::string& imageFile)
                {
                        std::cout << "Texture()" << std::endl;
                }
};

class Renderer {
        public:
                void Draw(const Texture& texture)
                {
                        std::cout << "Renderer.Draw()" << std::endl;
                }
};

int main(int argc, char* argv[])
{
        Renderer renderer;
        renderer.Draw("foo.png");

        return 0;
}

产生以下内容：

$ g++ -o Conversion.exe Conversion.cpp 
Conversion.cpp: In function ‘int main(int, char**)’:
Conversion.cpp:23:25: error: no matching function for call to ‘Renderer::Draw(const char [8])’
Conversion.cpp:14:8: note: candidate is: void Renderer::Draw(const Texture&)

但是，如果我将行更改为：

   renderer.Draw(std::string("foo.png"));

它将执行该转换。

这称为隐式类型转换。总的来说，这是一件好事，因为它可以防止不必要的重复。例如，您自动获得的std::string版本，Draw而无需为此编写任何其他代码。它还可以帮助您遵循开闭原则，因为它使您Renderer无需修改即可扩展其功能Renderer。

另一方面，它也不是没有缺点。一方面，它可能使弄清楚一个论点从何而来。在其他情况下，有时可能会产生意外的结果。这就是explicit关键字的目的。如果将其放在Texture构造函数上，它将禁用使用该构造函数进行隐式类型转换。我不知道一种全局警告隐式类型转换的方法，但这并不意味着不存在一种方法，只是gcc拥有大量的选项。