在结构中传递指向指针时的 EXC_BAD_ACCESS？

Question

我在一个结构中有一个 CGPoints 的 c 数组。添加另一个 CGPoint 时，我需要替换此数组。我发誓我做对了，它似乎工作了好几次，但最终我会得到一个EXC_BAD_ACCESS。我错过了什么？

这是结构，我已将其截断以删除许多不相关的项目。

typedef struct{
    CGPoint **focalPoints;
    NSUInteger focalPointCount;
    CGRect boundingRect;
}FocalPoints;

我是这样初始化它的：

CGPoint *fPoints = (CGPoint *)malloc(sizeof(CGPoint));
FocalPoints focalInfo = {&fPoints, 0, rect};

请注意，focalInfo 是通过引用传递给另一个函数的，例如：anotherFunction(&focalInfo)。

下面是用新数组替换 Points 数组的函数：

void AddFocalPoint (CGPoint focalPoint, FocalPoints *focal){
    if (focalPoint.x == CGFLOAT_MAX) return;
    if (!CGRectContainsPoint(focal->boundingRect, focalPoint)) return;
    int origCount = focal->focalPointCount;
    int newCount = origCount + 1;
    CGPoint *newPoints = (CGPoint *) malloc((newCount) * sizeof(CGPoint));
    for (int i = 0; i < newCount; i++)
        newPoints[i] = (i < origCount) ? *focal->focalPoints[i] : focalPoint; //error occurs here
    free(*focal->focalPoints);
    *focal->focalPoints = newPoints;
    focal->focalPointCount = newCount;
}

上述代码第 8 行出现 EXC_BAD_ACCESS 错误：newPoints[i] = (i < origCount) ? *focal->focalPoints[i] : focalPoint;。那我到底做错了什么？

Answer 1

我相信问题在于分配为&fPoints 的GCPoint *fPoints 计算为该地址的地址...一旦函数退出，该地址将不再有效。

（它指向的数据被malloc很好地分配了。）

Answer 2

这有点远，但*focal->focalPoints[i] 中的操作员优先级可能存在问题。您是否尝试根据您要达到的目的添加括号？

Answer 3

除了我在评论中提出的使用链表/NSMutableArray 的建议之外，我的另一个建议是您使用realloc() 而不是经常使用malloc()，手动复制，然后@987654326 @ing 旧分配。

void * realloc(void *ptr, size_t size);
realloc() 函数尝试将ptr 指向的分配大小更改为size，并返回ptr。如果没有足够的空间来扩大ptr 指向的内存分配，realloc() 会创建一个新分配，复制ptr 指向的尽可能多的旧数据以适应新分配，释放旧数据分配，并返回一个指向已分配内存的指针。

这几乎就是您正在做的事情，但是您可以让库为您处理它。

（我也可以谦虚地建议在你的函数中使用“焦点”这个词来命名变量吗？）（另外，我不太清楚为什么你的结构中的focalPoints 是一个指向指针的指针. 你只需要一个结构数组——单个指针就可以了。）

考虑以下（有点广泛的）重写；希望对您有所帮助。

typedef struct{
    CGPoint *points;    // Single pointer
    NSUInteger count;
    CGRect boundingRect;
} FocalPoints;

// Renamed to match Apple's style, like e.g. CGRectIntersectsRect()
void FocalPointsAddPoint (FocalPoints *, CGPoint);

void FocalPointsAddPoint (FocalPoints *f, CGPoint thePoint){
    if (thePoint.x == CGFLOAT_MAX) return;
    if (!CGRectContainsPoint(f->boundingRect, thePoint)) return;
    NSUInteger origCount = f->count;    // |count| is typed as NSUInteger; |origCount|
    NSUInteger newCount = origCount + 1;    // and |newCount| should be consistent
    // Greatly simplified by using realloc()
    f->points = (CGPoint *) realloc(f->points, newCount * sizeof(CGPoint));
    (f->points)[newCount-1] = thePoint;
    f->count = newCount;
}

int main(int argc, const char * argv[])
{

    @autoreleasepool {
        // Just for testing; any point should be inside this rect
        CGRect maxRect = CGRectMake(0, 0, CGFLOAT_MAX, CGFLOAT_MAX);
        // Can initialize |points| to NULL; both realloc() and free() know what to do
        FocalPoints fp = (FocalPoints){NULL, 0, maxRect};
        int i;
        for( i = 0; i < 10; i++ ){
            FocalPointsAddPoint(&fp, CGPointMake(arc4random() % 100, arc4random() % 100));
            NSLog(@"%@", NSStringFromPoint(fp.points[i]));
        }

    }
    return 0;
}