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