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

所以我在将所有元素都放入一个内存存储区时遇到了麻烦,我设法做到的最好是每次都覆盖存储区,而只能找出覆盖它的最终元素,这可悲的是还不够。

我主要是Java程序员,所以我可能会使用一些Java术语来代替C等效项,并且在引用C对象时可能会提到错误的类型,因此我会为此预先提出建议(并且不要让我开始使用指针) )。

最终,我想要做的是捕获图像中的所有凸性缺陷,并将它们存储在一个连续的空间中,然后我将通过JNI将其传递回Java中。商店的类型并不重要,在这个阶段,我只需要它可以工作,如果可以继续进行,我可以进行优化,但是我认为堆栈,队列,列表, 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));
        //            }
    }
}

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



转到文档“cvConvexityDefects()例程将返回CvConvexityDefect结构的序列”,以便它是一个充满CvConvexityDefect(s)的CvSeq。如果有帮助,cvConvexityDefects的签名是:
 CVAPI(CvSeq*)  cvConvexityDefects( const CvArr* contour, const CvArr*
    convexhull, CvMemStorage* storage CV_DEFAULT(NULL));

因此,总结一下我要做的就是找到一个轮廓,找到它的 shell ,找到它们之间的缺陷,将所有这些缺陷添加到一个大商店中,重复直到没有轮廓为止,返回所有缺陷的大商店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,但是我ll检查以防万一。

最佳答案

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

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

关于c++ - 在OpenCV项目中以C/C++添加到CvSeq或其他类型的动态内存存储中,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/8172642/

10-12 14:52