Некорректная работа, но ошибок нет

[vvsmvps]

Здравствуйте. Простейшая задача - сформировать массив строк из файла. Вроде бы всё правильно, но почему-то то работает, то не работает, то вылетает. В чём может быть дело?

char **keyWords = new char*[57];

//+++++++++++++++++++++++++Prototypes++++++++++++++++++++++++++
void readKeyWords();

//+++++++++++++++++++++++++Main++++++++++++++++++++++++++++++++
void main()
{
	clrscr();
	readKeyWords();
	delete [] keyWords;
	getch();
}

//++++++++++++++++++++++++++Functions++++++++++++++++++++++++++
void readKeyWords()
{
	unsigned i=0;
	FILE *fkey;

	fkey = fopen("c:\keywords.txt","r");

	while(!feof(fkey))
	{
		fgets(keyWords[i],20,fkey);
		i++;
	}
	fclose(fkey);
}

[fenryr]

char **keyWords = new char*[57];

Первая ошибка сразу видна - память под массив указателей выделяется, но память на строку по каждому указателю - нет.

keyWords[i]  = new char[21];
fgets(keyWords[i],20,fkey);

[vvsmvps]

Вау, спасибо. А есть ещё ошибки?

P.S. Просветите меня, есть ли разница между вот этим:

char a[12][10];

и вот этим:

char **a;

?

Обязательно ли второе - массив указателей? Можно ли обычные массивы так объявлять?

[An1ka]

В первом случаи это статический двойной массив, а во втором это объявления указателя на указатель.

Сообщение от vvsmvps

Обязательно ли второе - массив указателей?

Не обязательно. Можно статически выделить память.

Сообщение от vvsmvps

Можно ли обычные массивы так объявлять?

Что значит "обычные" массивы, а есть ли "необычные" массивы ?

char **keyWords = new char*[57];

Здесь просто массив указателей. Т.е. 57 строчек будущих.

А потом надо

for (int i=0; i<57; i++) keyWorlds[i] = new char [N];

Выделить память для каждой строчки размером N. А потом уже можешь записывать в эти строки текст.

[vvsmvps]

Я имею в виду, есть ли принципиальная разница в использовании между статическим двумерным массивом и таким: int **a. Как вы говорите "объявление указателя на указатель". Но ведь, как я понимаю, так можно и массив объявить? Или тут разница в распределении памяти?

[An1ka]

Сообщение от vvsmvps

Я имею в виду, есть ли принципиальная разница в использовании между статическим двумерным массивом и таким: int **a. Как вы говорите "объявление указателя на указатель". Но ведь, как я понимаю, так можно и массив объявить? Или тут разница в распределении памяти?

Да. Разница в способе выделения памяти. Либо статически - память автоматически освободится при выходе из области видимости, либо динамически - будет "жить", пока не будет освобождена через delete.
Еще статически выделенные блоки памяти должны быть константного размера (их размер известен еще до компиляции и не меняется в процессе выполнения программы).
А динамически выделенные блоки памяти могут иметь произвольный размер и их размер может определяться в процессе выполнения программы.

[vvsmvps]

Подниму тему.
Во время очистки памяти я пишу

delete [] keyWords;

Очистится ли вся память под массив или же нужно так же в цикле очищать?

[EUGY]

new выделяет память. И на каждый new требуется delete.
В Вашем случае нужно в цикле: delete[] keyWorlds[i];
И итоговый: delete[] keyWords.