Блог

Я пишу приложение для iPhone, которое использует связанный список для отслеживания набора точек. Я снова и снова успешно использовал связанные списки в других программах на C, но я никогда не использовал связанный список на iPhone, и моя функция ‘freeList’ дает сбой.

 pointNode * temp;

pointTrain = lPoints->next;
while (pointTrain) {
    temp = pointTrain->next;
    free(pointTrain);
    pointTrain = temp;
}
  

Вот как я освобождаю свой список, ‘lPoints’ является заголовком и всегда выделяется во время выполнения. Этот код выполняется до добавления каких-либо дополнительных узлов, но, похоже, он все еще выполняется, даже если «pointTrain» равен NULL, что должно заставить его пропустить цикл while. К сожалению, цикл все еще выполняется, и программа завершает работу при освобождении (pointTrain)

Есть ли что-то, чего я просто не вижу?

Точки добавляются подобным образом в цикле:

 pointTrain->x = onPx / trueWidth;
pointTrain->y = onPx % trueWidth;
pointTrain->next = (pointNode*)malloc(sizeof(pointNode));
pointTrain = pointTrain->next;
  

Один мудрый человек однажды сказал: «Я бы предпочел управлять лошадью в том направлении, в котором она движется».

Не создавайте свою собственную структуру, когда есть совершенно хороший NSMutableArray класс, который вы могли бы использовать. Линейная итерация массива дешева и проста, а класс поставляется со своими собственными настройками управления памятью, и все, что вам нужно сделать, это ИСПОЛЬЗОВАТЬ их.

Это то, что я часто вижу у разработчиков C и C , которые переходят на Objective-C. Следующий проект на моей тарелке — отменить запуск проекта, на который один ОЧЕНЬ высокопоставленный специалист по C потратил последние шесть месяцев. Глядя на код, становится ясно, что он потратил почти все это время на споры с фреймворком. На данный момент он практически создал свой собственный фреймворк.

Вы явно инициализируете рядом с NULL или nil ?

Если нет, то, скорее всего, для него уже установлено некоторое ненулевое значение, и поэтому while (pointTrain) все равно будет установлено значение true.