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

Question

我有一个结构CGPoint的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)。

现在，这里的该用新的替换点阵列的功能：

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

这是一个有点长镜头，但也许有在*focal->focalPoints[i]与操作者优先的问题。你有没有尝试根据你想要实现的内容添加括号？

Answer 2

我相信这个问题带有地方GCPoint *fPoints分配为&fPoints计算结果为一个地址......这不再是一次函数退出有效。

（其所点被分配与malloc精细的数据。）

Answer 3

除了我在评论中提出的建议，使用链表/ NSMutableArray的，我的另一项建议是，你使用realloc()而是不断利用malloc()，复制的手，然后free()荷兰国际集团旧的分配。

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