Здравствуйте.

Суть моего вопроса такова. Мне необходимо создать двумерный массив, состоящий из строк. Потом мне нужно в отдельной функции по необходимости добавлять туда значения. Делаю это так.



Так вот. После того, как я очищаю буфер, значение MyArray[x][y] превращается в мусор. Как бы мне сохранить значение переменной MyArray при выходе из функции?

Сообщение отредактировал Freeze Anti - Feb 10 2009, 11:35

Все правильно, это же указатели!

strcpy(MyArray[x][y], buffer)

Выделить память где-нибудь вне MyFunc, или просто объявить

Убрать статик.

компилирую в Dev-C++ (компилятор - gcc)

strcpy вызывает ошибку доступа к памяти во время исполнения программы

static char MyArray[255][8] - это тип char**, а я все же объявляю char***,

кстати. еще попробовал создать дополнительную переменную char buf[51]
после этого попытался приравнять buf = buffer компилятор у меня выдал incompatible types in assignment

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

разобрался

я перед strcpy не выделял память для MyArray

вот так работает



всем спасибо за оказанную помощь

Сообщение отредактировал Freeze Anti - Feb 10 2009, 12:47

Осталось выяснить, зачем вообще нужны пляски с calloc, и memcpy, если статически выделенная память уже имеется (если речь действительно идет о массиве строк, а не УКАЗАТЕЛЕЙ на них). В чем заключается "работа" написанного выше я не понял

нельзя вызывать free(buffer).

У вас есть двухмерный аррей пойнтеров на стринги, сами стринги вы правильно аллокируете - calloc,
освобождать будете в другом месте.
При освобождении проверяйте на NULL - если не NULL - free

кстати, можно пользовать strdup() вместо calloc() i strcpy()

"У вас есть двухмерный аррей пойнтеров на стринги, сами стринги вы правильно аллокируете - " -
Задорнов отдыхает

Совет - задать этот вопрос в програмистком форуме (на vingrad например). Из 4х советов, озвученных здесь - 3 неправильные (или провокационные, уж не знаю).
Ваше последнее решение тоже неверное.



Сообщение отредактировал XVR - Feb 11 2009, 07:51

Совет не верный - правильный совет - прежде всего сформулировать вопрос хоть как-то, поскольку
Мало коррелирует с тем, что автор пытался написать.

"двухмерный аррей пойнтеров на стринги" читать как "двухмерный массив указателей на строки".
А в остальном согласен с мнением A. Fig Lee.

Значение MyArray[x][y] вовсе не превращается в мусор, там находится указатель на память, выделенную вызовом calloc(1, 51).
А вот после free(buffer) эта память освобождается и при последующих вызовах calloc() может быть выделена уже под другую строку. Просто не нужно освобождать память под строку, до тех пор пока она нужна. А как перестала быть нужной - можно и освободить:
free( MyArray[x][y] );
MyArray[x][y] = NULL;

мы и сюда ходим.

Точно, тогда понятно откуда пришел единственный правильный ответ
С мнением zltigo полностью согласен - не понятно, чего аффтор хотел. Понятно только то, что получил он не то, что хотел (что бы это не было )

XVR не прав - неправильных ответов здесь гораздо больше.


MyArray - это не массив строк, а массив указателей, он может указывать на любую строку, но сам он ее не содержит. Строкой MyArray[x][y] = buffer; ты указываешь на buffer, но следом этот буфер удаляешь. Поэтому указатель в MyArray после этого указывает в пустоту.
Потом, тебе не мешало бы понять что такое строка в Си, тогда тебе станет ясно, что массивов строк, а тем более двумерных, вообще говоря не бывает. Строка это одномерный массив char-ов заполненный символами и оканчивающийся нулем. Поскольку строки могут быть разной длины, то массив из них не создашь. Поэтому обычно имеют дело с указателями на строки - создается переменная типа char* или массив как у тебя MyArray и отдельно создаются строки, выделяя для каджой из них память нужной длины. После этого нужному элементу массива присваивается значение адреса выделенной строки и с ней работают. Удалять же ее нужно только по окончании работы, а не как у тебя.



Кстати, это одна из причин почему надо ликвидировать раздел "Для начинающих" - его посещают только начинающие, поэтому дать правильный ответ некому.


Фигасе форум. Программисты не смогли даже для себя нормальный форум создать, о чем там с ними можно говорить?
А про rsdn.ru вы когда-нибудь слышали?