这是不好的做法吗？ C ++

Question

Class1 myclass(someparameter);

int main(int argc, char* argv[])
{
    myclass = Class1(anotherparameter);
}

我在文件范围内有一个变量。

我真的不知道怎么说出我的问题。但基本上我正在复制一个类，这段代码看起来很时髦。这样做有什么后果吗？我应该使用new / delete吗？我能想到的一个潜在问题是，如果类包含指针（但那可以通过创建复制构造函数来解决）

Answer 1

我认为全局变量通常是不受欢迎的。但这并不意味着它们是非法的或“坏的风格”。在这种特殊情况下，我肯定会避免使用全局变量......

Answer 2

假设你的课程可以正确复制，那么我没有看到任何内在错误。但它并不是那么有效，因为myclass实际上被初始化了两次：一次声明它，再次指定另一个实例。

如果这是你的逻辑要求，那没关系。否则，您可以提高代码效率。

Answer 3

你所做的事情没有错，虽然目前尚不清楚这是什么意思。有时这种事情是有条件的 - 例如，如果命令行参数指定变量的替代值。您不需要使用new和delete ...在这种情况下，您要求编译器创建一个临时对象，然后将其复制到全局变量中（使用其operator=），然后临时将销毁。效率通常至少比直接构造具有所需值的变量更差，但在过早或不必要的优化中没有任何意义。

为了进行比较，请考虑：

std::string sep = "\n";

int main(...)
{
    if (...)
        sep = std::string("\r\n");
}

这或多或少都是一样的，尽管这里std::string临时的显式构造是可选的，因为该类碰巧有operator=(const char*)。一切都好。