1

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

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

我主要是一名 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));
        //            }
    }
}

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

断言失败：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 周，所以已经找到了很多资源，是的，我觉得自己真的很愚蠢，因为我没有设法弄清楚什么应该是一个有效的简单循环。）

谢谢

编辑 - 由于评论而添加了更多细节。

编辑 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，但我会检查以防万一。

4

1 回答 1

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));
于 2011-11-18T07:24:58.350 回答