C++ 不能直接调用构造函数''答案

【问题标题】：C++ Cannot call constructor ' ' directlyC++ 不能直接调用构造函数''
【发布时间】：2012-03-04 10:44:25
【问题描述】：

我正在编写一些 OpenCV 代码并在 Windows 上的 VS 2008 中开发它。我正在尝试使用 g++ 在 Linux 上运行代码，但对于 ImageProcessor 和我创建的所有其他类，我收到错误“无法直接调用构造函数 'ImageProcessor::ImageProcessor'”。我试图找到一种间接调用构造函数的方法，但无济于事。任何建议都会很棒。代码在 Windows 上编译并运行良好。

if (x == 1){
    cout <<"MODE SELECTED: IMAGE TESTING \n";
    ImageProcessor* IP = new ImageProcessor;
    LaneDetector* LD = new LaneDetector;
    LaneInfo* LI1 = new LaneInfo;
    LaneInfo* LI2 = new LaneInfo;
    LaneVector* LV = new LaneVector;
    cvNamedWindow("Window",CV_WINDOW_AUTOSIZE);

    IplImage* temp = 0;
    IplImage* img0 = 0;
    img0 = cvLoadImage(PICTURE_INPUT);
    CvRect r = cvRect(0,((img0->height)/3),img0->width,((img0->height)/3)+20);
    cout <<"IMG0 LOADED \n";

    while(1){
        IP->ImageProcessor::ImageProcessor(img0, r);
        temp = IP->ImageProcessor::get_processed_image();
        LD->LaneDetector::LaneDetector(temp,r);
        LD->LaneDetector::find_edges();
        LI1 = LD->LaneDetector::find_lanes(5);
        LI2 = LD->LaneDetector::find_lanes(25);
        LV->LaneVector::LaneVector(LI1,LI2);
        LV->LaneVector::print_lane_angle_info();

        if( (cvWaitKey(20) & 255) == 27 ) break;
        cvShowImage("Window", temp);
        hold(1);
    }
}

【问题讨论】：

请提供ImageProcessor的声明
你需要告诉我们错误发生在哪里，就像 Ed 说的，ImageProcessor 是什么样的？ while 循环中的第一行对我来说看起来很奇怪。
C++ 的工作方式与此不同。从一本好书开始。
¤ 在 C++ 中，变量不必是new-ed，除非你想要一个任意的生命周期。只需将T 类型的变量v 声明为T v;。对于您的图像处理器对象，声明该变量在循环中，如ImageProcessor ip( img0, r );。这就是所说的问题。但除此之外，您应该避免使用全部大写的变量名。将其保留为宏名称。干杯，为自己准备一本好的 C++ 初学者书籍！，

标签： c++ class constructor g++

【解决方案1】：

这段代码太糟糕了。

为什么要限定每个成员函数？

不，你不能在已经创建的对象上调用构造函数。初始化对象时应提供任何构造函数参数（您的代码使用 new 执行此操作，这也不是好的 C++ 编码风格）。如果在构造后很久才提供这些参数，请将“构造器”更改为具有适当名称的普通成员函数。

您的代码也存在大量内存泄漏。看起来您正在使用 C++ 语法编写 Java 代码。这可不是什么好事。

【讨论】：

-1 "你不能在已经创建的对象上调用构造函数。"是不正确的，答案并没有建设性地帮助 OP 解决问题。无论我多么同意这里表达的其他观点，我都必须投反对票。对不起。
为什么不将此答案移至 comment 部分——毕竟只是评论？
@Alf：我的回答是正确的。您可以创建一个新对象重用相同的内存，但那是一个新对象。而“把构造函数改成普通的成员函数”就是解决问题的办法。
我已经告诉过你，你的评论是不正确的。有像我这样的人告诉你这应该足以让你去寻找如何做你说做不到的事情。碰巧的是，您不必搜索更长的时间，只需完整阅读此页面即可。干杯&hth.，
我不确定您要究竟问什么。但是您使用malloc 进行分配。然后您必须使用free 释放。要获得delete 表达式的一般效果，您必须首先显式调用对象上的析构函数，例如p->T::~T()（您必须查找它，我不记得是否需要或禁止限定，这很漂亮随意的）。请注意，放置delete 运算符仅在一种情况下调用，即构造函数从new 表达式中抛出。这就是我认为您要问的实际情况。干杯&hth，

【解决方案2】：

这段代码很奇怪，每次通过循环都在现有对象之上重建IP？

不确定语法

IP->ImageProcessor::ImageProcessor(img0, r);

曾经有效。也许在非常古老的 C++ 中。这样做的正常方法是

new (IP) ImageProcessor(img0, r);

不是说这是个好主意，但我认为它会做同样的事情。

【讨论】：

该代码没有重构任何东西。这在 C++ 中是完全非法的。您的代码只是在解决一个根本不存在的问题。
请不要建议放置new。我认为合适的东西是循环内的局部非指针变量。
好吧，我不会说我是建议它，只是如果 VS 接受IP->constructor，我猜这就是它正在做的事情。但我永远不会为整件事辩护；这很奇怪，我无法解释为什么有人会写这样的东西。但是，然后，我没有 VS 的背景，我确实看到 VS sn-ps 一直看起来很奇怪（虽然不是很奇怪。）
OP 的语法永远无效。 :-)