c++ - 在 OpenCV 项目中添加 CvSeq 或其他类型的 C/C++ 动态内存存储

Question

问题已回答，请参阅问题末尾的解决方案。仍然欢迎更多评论/回答。

所以我在将所有元素放入一个内存存储时遇到了一些麻烦，我设法做的最好的事情就是每次都覆盖存储并且只取出覆盖它的最终元素，遗憾的是这还不够。

我主要是一名 Java 程序员，所以我可能会使用一些 Java 术语来代替 C 等价术语，并且在引用 C 对象时可能会提到错误的类型，所以我为此提前申请(不要误会我的意思)从指针开始)。

最终我想要做的是获取图像中的所有凸面缺陷并将它们存储到一个连续的空间中，然后我将通过 JNI 将其传回 Java。商店的类型并不重要，在这个阶段我只需要它工作，如果我们继续进行，我可以进行优化，但我认为 Stack、Queue、List、Vector 或类似的东西可以完成这项工作。目前我一直在尝试循环将 CvSeq 对象组添加到一个大型 CvSeq，我将发布我的代码并在发布后讨论它:

 CvSeq *allDefects;
allDefects = cvCreateSeq(0, sizeof (CvSeq), sizeof (CvConvexityDefect), mem_storage4);

CvContourScanner scanner3 = cvStartFindContours(img_bin, mem_storage3);
while ((c2 = cvFindNextContour(scanner3)) != NULL) {
    if (threshold != 0 && cvContourPerimeter(c2) < threshold) {
        cvSubstituteContour(scanner3, NULL);
    } else { // otherwise create the hull
        CvSeq* c_new;
        c_new = cvConvexHull2(c2, mem_storage5, CV_CLOCKWISE, 0);
        CvMemStorage* storage;
        storage = cvCreateMemStorage(0);
        defects = cvConvexityDefects(c2, c_new, storage);
        allDefects = defects;
        //            for (int i = 0;  i < defects->total; i++){
        //            cvSeqPush(allDefects, CV_GET_SEQ_ELEM(CvSeq, defects, i));
        //            }
    }
}

所以我们创建了一个扫描仪，虽然我们可以找到轮廓，但我们这样做并检查它是否大于给定的阈值。假设它是我们围绕它创建一个船体，然后找到它和船体之间的缺陷。这意味着 defects 中可以有多个对象，我想在每次循环时将其添加到 allDefects 但唯一能让它继续运行的方法是制作 allDefects 等于 defects 意味着它在每个循环中都会被覆盖。您可以在我尝试将其像堆栈一样推送的地方看到一些注释代码，但这会因错误而崩溃:

Assertion failed: sizeof(((defects))->first[0]) == sizeof(CvSeqBlock) && ((defects))->elem_size == sizeof(CvSeq), file vtoolsModified.cpp, line 1407

根据文档“cvConvexityDefects() 例程返回一系列 CvConvexityDefect 结构”，因此这是一个充满 CvConvexityDefect(s) 的 CvSeq。如果有任何帮助，cvConvexityDefects 的签名是:

 CVAPI(CvSeq*)  cvConvexityDefects( const CvArr* contour, const CvArr* 
    convexhull, CvMemStorage* storage CV_DEFAULT(NULL));

所以总结一下我要做的就是找到一个轮廓，找到它的外壳，找到它们之间的缺陷，将所有这些缺陷添加到一个大商店，重复直到没有轮廓为止，将所有缺陷的大型存储返回给 Java。这是我正在寻求帮助的粗体字。

任何人都可以帮助或指出可以提供来源的方向吗？ (我已经在这个特定问题上工作了大约 2 周，所以已经找到了很多资源，是的，我觉得很愚蠢，因为我自己没有设法弄清楚什么应该有效地成为一个简单的循环。)

谢谢

编辑 - 根据评论添加了更多详细信息。

编辑 2 - 问题已回答，生成的固定代码如下

看来我对 C 缺乏理解是根本问题。我假设我需要在我应该使用 CvConvexityDefect 的地方使用 CvSeq。修改后的代码如下:

CvSeq *defects;
CvSeq *allDefects;
allDefects = cvCreateSeq(0, sizeof(CvSeq), sizeof(CvConvexityDefect), mem_storage4);

CvContourScanner scanner3 = cvStartFindContours(img_bin, mem_storage3);
while ((c2 = cvFindNextContour(scanner3)) != NULL) {
    if (threshold != 0 && cvContourPerimeter(c2) < threshold) {
        cvSubstituteContour(scanner3, NULL);
    } else { // otherwise create the hull
        CvSeq* c_new;
        c_new = cvConvexHull2(c2, mem_storage5, CV_CLOCKWISE, 0);
        CvMemStorage* storage;
        storage = cvCreateMemStorage(0);
        defects = cvConvexityDefects(c2, c_new, storage);
        //            allDefects = defects;
        if (defects->total < 100) {
         for (int i = 0; i < defects->total; i++) {
          CvConvexityDefect* element = CV_GET_SEQ_ELEM(CvConvexityDefect, defects, i);
           if (element != 0){      
               cvSeqPush(allDefects, element);                    }
            }
        }
    }
}

你可以看到我也在检查以确保返回的缺陷少于 100 个(有时是数百万个会破坏系统)并且还确保 CV_GET_SEQ_ELEM 不返回 null(我是不确定它是否可以返回 null 但我会检查以防万一。

Answer 1

我的猜测是您的循环在这一行中有错误:

CV_GET_SEQ_ELEM(CvSeq, defects, i)

如果你看一下它的定义:

#define CV_GET_SEQ_ELEM( elem_type, seq, index ) CV_SEQ_ELEM( (seq), elem_type, (index) )

你可以看到第一个参数必须是元素类型(cvFindContours 通常生成 CvPoint 的序列)而不是序列的类型(CvSeq)

您可以存储序列的其他方法是使用

    void* cvCvtSeqToArray(const CvSeq* seq,
                          void* elements,
                          CvSlice slice = CV_WHOLE_SEQ
                         );

并将数组存储在一个列表中，这样可以将每个对象的缺陷分开。 根据您的需要，还有更多选项。

编辑2:

好的，所以我刚刚查看了 cvConvexityDefects 并看到它返回了一个结构序列，而不是 CvPoint，那么您的 allDefects 应该是一个定制的 CvSeq，您将像这样修改您的代码:

cvSeqPush(allDefects, CV_GET_SEQ_ELEM(CvConvexityDefect, defects, i));