Странная ошибка C при освобождении памяти?
Я все еще пытаюсь заставить мой генетический алгоритм работать (вчера у меня была проблема с выделением памяти и, следовательно, ужасная ошибка при попытке освободить его), но сегодня у меня есть этот кусок кода
while (iCurrentGen <= data->m_iMaxGenerations)
{
arrfSelectedChromosomes = selection(&data[0], szChromosomes);
iSelectedLen = order_descending_grid(arrfSelectedChromosomes);
szAuxGen = crossover(&data[0], arrfSelectedChromosomes, szChromosomes, iSelectedLen);
//szChromosomes is what I need to free after that call returns
free_generation(&data[0], szChromosomes);//Code Explotion
szChromosomes = szAuxGen;
szAuxGen = NULL;
++iCurrentGen;
}
Я проверял его содержание до free_generation()
вызов функции и это было что-то вроде рисунка ниже:
(Переменная первых четырех значений)
Но внутри free_generation()
функция, та же самая переменная теряет некоторые из своих значений (особенно когда i
принимает значение 2, i
являющийся индексом цикла for), как показано ниже:
(Значения переменных внутри функции)
Я публикую код free_generation ниже:
void free_generation(struct INPUT_DATA* d, char** szChromosomes)
{
int i;
for (i = 0; i < d->m_iPopulationSize; ++i)
{
free(szChromosomes[i]);
}
free(szChromosomes);
szChromosomes = NULL;
}
Определение szChromosomes
как следует:
char** szChromosomes = (char**)malloc(d->m_iPopulationSize * sizeof(char*));
srand(time(NULL));
for (i = 0; i < d->m_iPopulationSize; ++i)
{
szChromosomes[i] = (char*)malloc((d->m_iBitsPChromosome + 1) * sizeof(char));
for (j = 0; j < d->m_iBitsPChromosome; ++j)
{
szChromosomes[i][j] = rand_1_0(0.0, 1.0) == 1? '1' : '0';
}
szChromosomes[i][j] = '\0';
}
Мне нужно уточнить, что потеря этого значения происходит только после второй итерации цикла while, размещенного сверху. Я имею в виду, что при первом запуске все работает отлично, но после этой итерации вторая ведет себя так, как описано выше.
РЕДАКТИРОВАТЬ:
Я забыл включить приращение переменной контроля цикла (спасибо за указание на это! И нет, это не глобальный xD). Я включаю части кода кроссовера:
char** crossover(struct INPUT_DATA* d, float** arrfSelectedChromosomes, char** arrszChromosomes, int iChromosomesInGrid)
{
int i;
int iTIndex = 0, iRPos = 0;
char* szFirstChromosome = NULL;
char* szSecondChromosome = NULL;
char* szFirstNewChrom = (char*)malloc((d->m_iBitsPChromosome + 1) * sizeof(char));
char* szSecondNewChrom = (char*)malloc((d->m_iBitsPChromosome + 1) * sizeof(char));
char** arrszNewPop = (char**)malloc(d->m_iPopulationSize * sizeof(char*));
int iSplitPoint = (int)(d->m_iBitsPChromosome / 4);
float fCrossOverProb = CROSSOVER_PROBABILITY;
srand(time(NULL));
for (i = 0; i < d->m_iPopulationSize; i += 2)
{
iRPos = rand() % iChromosomesInGrid;
iTIndex = (int)arrfSelectedChromosomes[iRPos][0];
szFirstChromosome = arrszChromosomes[iTIndex];
iRPos = rand() % iChromosomesInGrid;
iTIndex = (int)arrfSelectedChromosomes[iRPos][0];
szSecondChromosome = arrszChromosomes[iTIndex];
if (is_same_chromosome(szFirstChromosome, szSecondChromosome))
{
i -= 2;
continue;
}
if (fCrossOverProb < CROSSOVER_PROBABILITY)
{
//if probability is lower than the defined prob. we keep both chromosomes
strcpy(szFirstNewChrom, szFirstChromosome);
strcpy(szSecondNewChrom, szSecondChromosome);
}
else
{
strcpy(szFirstNewChrom, szFirstChromosome);
szFirstNewChrom[iSplitPoint] = '\0';
strcat(szFirstNewChrom, &szSecondChromosome[iSplitPoint]);
//Para crear el segundo hijo se realiza una operacion similar
strcpy(szSecondNewChrom, szSecondChromosome);
szSecondNewChrom[iSplitPoint] = '\0';
strcat(szSecondNewChrom, &szFirstChromosome[iSplitPoint]);
}
arrszNewPop[i] = szFirstNewChrom;
arrszNewPop[i + 1] = szSecondNewChrom;
}
return arrszNewPop;
}
1 ответ
Поскольку запросы функций для пометки комментария как ответа остаются отклоненными, я копирую вышеупомянутое решение здесь.
нм ты мужчина! большое спасибо.. Это была проблема дублированных указателей, и это было связано с неправильным распределением памяти в функции кроссовера. Я выделял память для szFirstNewChrom и szSecondNewChrom в начале функции, но эта память использовалась для 30 различных строк. Из-за этой ужасной ошибки функция free_generation продолжала отказывать, поскольку пыталась освободить ранее освобожденный указатель. Спасибо вам всем! - Хорхе Сеспедес