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

Question

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

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

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

最终我要做的是获取图像中的所有凸面缺陷并将它们存储到一个连续空间中，然后我将通过 JNI 将其传回 Java。商店的类型并不重要，在这个阶段我只需要它工作，如果我们继续进行，我可以进行优化，但我认为堆栈、队列、列表、向量或类似的可以完成这项工作。目前我一直在尝试循环添加一组 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 意味着每个循环都会覆盖它。您可以看到一些注释代码，我尝试将其像堆栈一样推送，但这会因错误而崩溃：

断言失败：sizeof(((defects))->first[0]) == sizeof(CvSeqBlock) && ((defects))->elem_size == sizeof(CvSeq)，文件 vtoolsModified.cpp，第 1407 行

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

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

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

任何人都可以帮助或指出可以的来源吗？ （我已经研究这个特定问题大约 2 周了，所以已经找到了很多资源，是的，我觉得自己没有设法弄清楚什么应该是一个有效的简单循环真的很愚蠢。）

谢谢

编辑 - 由于 cmets 添加了更多细节。

编辑 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
                         );

并将数组存储在一个列表中，这样就可以将每个对象的缺陷分开。 还有更多选择，具体取决于您的需要。

EDIT2：

好的，所以我刚刚查看了 cvConvexityDefects，发现它返回的是结构序列，而不是 CvPoint，那么您的 allDefects 应该是定制的 CvSeq，您将修改您的代码，如下所示：

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