Удалено модератором

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

С интересом бы прочитал этот коментарий! То, что взгляды не совпадают, явление нормальное, да и шутить тоже не грех, причем, на любую тему!

Все замолчали на эту тему, а, по-моему, она не закрыта! Добавлю еще ряд подвопросов.
class PState{}
PState * ps[1200]; // статически выделенная память
for(i = 0; i <1200;i++)ps[i] = new PState ();
/*заполняем предыдущий статический массив динамически*/
/*
какой-то дурацкий код...
*/
delete [ ] ps;
/* похоже, что этот оператор освободит КАЖДЫЙ динамически выделенный элемент массива*/

// Теперь то же, но по-другому

PState **pps = new PState *[1200];
/*кстати, не понятно почему
PState *pps[ ] = new PState *[1200];
выдает ошибку - это же одно и то же!!!*/

for(i = 0; i <1200;i++)pрs[i] = new PState ();
/*заполняем динамический массив динамически*/
/*
еще какой-то дурацкий код...
*/
delete [ ] pps;
/*А что удаляет этот оператор? Массив указателей или и все что внутри тоже?
А может правильно делать так:
for(i = 0; i <1200;i++)delete pрs[i];
delete [ ] pps:
*/

Господа, пожалуйста, делитесь ОПЫТОМ и КОДОМ, а не соображениями.

Спасибо.

Цитата

Все замолчали на эту тему, а, по-моему, она не закрыта

Чем же это она не закрыта? На все предыдущие вопросы ответы были данны.

Цитата

class PState{} PState * ps[1200]; // статически выделенная память for(i = 0; i <1200;i++)ps[i] = new PState (); /*заполняем предыдущий статический массив динамически*/ /* какой-то дурацкий код... */ delete [ ] ps; /* похоже, что этот оператор освободит КАЖДЫЙ динамически выделенный элемент массива*/

НЕЛЬЗЯ так делать! Я уже об этом говорил. delete[] применим ТОЛЬКО к указателям, для которых память была выделенна с помощью new[] . То, что на вашем компиляторе это может быть работает не означает, что так можно. Если в цикле выделяете, то и в цикле и освобождайте. То есть либо так:

Код

PState * ps[1200]; // статически выделенная память for(i = 0; i <1200;i++)ps[i] = new PState (); ...... for(i = 0; i <1200;i++) delete ps[i];

либо так:

Код

PState * ps=new PState[1200]; // статически выделенная память ....... delete [] ps;

Цитата

PState **pps = new PState *[1200]; /*кстати, не понятно почему PState *pps[ ] = new PState *[1200]; выдает ошибку - это же одно и то же!!!*/

Ничего это не одно и то же, если вы пишите на С++. На мой взгляд, это извращение высшего порядка - в С++ классы есть. А тип PState *[] это не то же самое, что PState **, а то, что они оба реализуются как указатель на указатель, не делают их одним и тем же типом. Это в Java вы можете писать:

Сpd pd[];
pd=new Cpd[10];

В С++ это не так.

Цитата

А может правильно делать так: for(i = 0; i <1200;i++)delete pрs[i]; delete [ ] pps;

Да. delete[] удаляет массив из нескольких элементов и для каждого из них вызывается деструктор, если тип нетривиальный. В данном случае - это массив указателей - тип тривиальный. Строчкой

delete [ ] pps;

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

Уважаемый, Fantasist

С этого все начиналось...

PayState *ptr; // класс поинтер вестимо
PayState *pPayState[1200];

for (i = 0; pPayState[i] != NULL; i++ )delete pPayState[i];
// я думал, что тут я освобождаю всю занятую память
i = 0;
do {
ptr = new PayState ( bc, ic ); // конструктор - аллокирует новую запись
if ( ptr != NULL ){ // успешно
// какой-то дурацкий код...
pPayState[i++] = ptr; //Поместить в массив для дальнейших нужд
}
else {
ShowMessage ("Not enough memory, Sorry");
exit ( EXIT_FAILURE );
}
} while ( ); // больше записей нет!

Я начал эту тему потому что НЕ работало.
Скопированное из хэлпа
delete[ ] pPayState;
прекрасно освобождало память. Это и послужило отправной точкой дискуссии.
Ответа на почему не работало не нашлось, хотя гипотезы и предположения были. Обратите внимание, в НЕработающем коде я делал именно так, как вы настаиваете и я изначально делал.

Цитата

НЕЛЬЗЯ так делать! Я уже об этом говорил. delete[] применим ТОЛЬКО к указателям, для которых память была выделенна с помощью new[] . То, что на вашем компиляторе это может быть работает не означает, что так можно. Если в цикле выделяете, то и в цикле и освобождайте

Согласен, что некрасиво, но почему то что красиво не согласно со мной?

Код

PState **pps = new PState *[1200]; /*кстати, не понятно почему PState *pps[ ] = new PState *[1200]; выдает ошибку - это же одно и то же!!!*/

Цитата

Ничего это не одно и то же, если вы пишите на С++. На мой взгляд, это извращение высшего порядка - в С++ классы есть. А тип PState *[] это не то же самое, что PState **, а то, что они оба реализуются как указатель на указатель, не делают их одним и тем же типом. Это в Java вы можете писать: Сpd pd[]; pd=new Cpd[10]; В С++ это не так.

Мы о Джаве не говорим.
А вот
void main(int argc, char *argv[ ]){...}
void main(int argc, char **argv ){...}
Это что не C++? Или что-то из этого не? Или это не одно и тоже?
Не буду утверждать безаппеляционно, как некоторые, но замечу, что по определению [ ](массив) это *(указатель), и если есть

Код

char a[10]; char *p; p = a; // абсолютно корректно, а  *( p + 5), например,  тоже самое что и a[5]

И причем тут классы? В С++ много чего есть! И при чем тут извращения?
Где сказано, что в ++ указатель на массив какого-либо типа и указатель на указатель того же типа, не одно и тоже? Единственный довод в Вашу пользу, что компилятор выдает ошибку, но это не самый сильный довод.

Цитата

Да. delete[] удаляет массив из нескольких элементов и для каждого из них вызывается деструктор, если тип нетривиальный. В данном случае - это массив указателей - тип тривиальный. Строчкой delete [ ] pps; массив указателей будет удален. Объекты, на которые эти указатели указывали никто удалять за вас не будет.

Похоже, что и это не так. Посмотрите опять на самое начало.

И давайте аргументировать, а не бросаться формулами.

С уважением,
BRB

Цитата

Я начал эту тему потому что НЕ работало. Скопированное из хэлпа delete[ ] pPayState; прекрасно освобождало память

Слушайте, а чем вы компилируете? ? ?

Вот код:

Код

CClass *pClass[1200]; for (int i=0; i<1200; ++i) {pClass[i]=new CClass(i);}; delete[] pClass;

Компилируем VS60. На последнюю строчку получаем:

Цитата

warning C4154: deletion of an array expression; conversion to pointer supplied

При исполнении все грохается на последней строчке, как и следовало ожидать. И это законно - освобождение стековой переменной.

Цитата

Согласен, что некрасиво, но почему то что красиво не согласно со мной?

Ну сколько можно! Это не не красиво, это НЕ ПРАВИЛЬНО по СТАНДАРТУ!

Цитата

но замечу, что по определению [ ](массив) это *(указатель), и если есть

Никогда такого опеределения не было. Поймите, С++ - это не С. Это разные языки, хоть и похожи.
В С++ вы неможете написать:

char p[]=new char[100];

Не можете, и все. Это СТАНДАРТ языка. ТО ЧТО МАССИВ(в реализации) ЯВЛЯЕТСЯ УКАЗАТЕЛЕМ НЕ МЕНЯЕТ ПОЛОЖЕНИЯ.

Удалено модератором

Fantasist

Цитата

Слушайте, а чем вы компилируете? ? ?

Билдером. Был первый, сейчас обзавелся 6-м

Цитата

Компилируем VS60. На последнюю строчку получаем:

Может VC6?

Цитата

Никогда такого опеределения не было. Поймите, С++ - это не С. Это разные языки, хоть и похожи.

C++ это надмножество С. Не верите? Спросите у Страуструпа(фамилию, наверняка, переврал, но все равно узнаваемо), он ++ сочинил!

Цитата

В С++ вы неможете написать: char p[]=new char[100];

Во-первых, частица НЕ с глаголами пишется раздельно, по крайней мере, на русском языке!
А, во-вторых, хотел бы узнать не то, что это ошибка /* Cannot convert 'char *' to 'char[]'*/, а почему это так!

Цитата

При исполнении все грохается на последней строчке, как и следовало ожидать. И это законно - освобождение стековой переменной.

Массив указателей был объявлен СТАТИЧЕСКИ! Может это причина? Почему же, все-таки первый вариант(внутри цикла до NULL) не работал? И мы именно с Вами это уже обсуждали!

Цитата

for (i = 0; pPayState[i] != NULL; i++ ){   delete pPayState[i];   pPayState[i] = NULL; } Странно, только, что 3-4 прохода, а это полный размер pPayState[]. Вот если бы вылетало на втором...

Последнее замечание оставленно Вами без ответа.
Тем не менее, от добра добра не ищут, и я оставил все как есть, потому что нет проблем!
Удалено модератором

Удалено модератором

Удалено модератором

Удалено модератором

Цитата

А, во-вторых, хотел бы узнать не то, что это ошибка /* Cannot convert 'char *' to 'char[]'*/, а почему это так!

Вот %^&$!! ДА ПОТОМУ ЧТО ЭТО РАЗНЫЕ ТИПЫ ОПРЕДЕЛЕННЫ СТАНТДАРТОМ ЯЗЫКА Mассив - это статический (причем с фиксированным размером!) кусок памяти и все тут. Исключения составляют формальные аргументы функций.

Удалено модератором

Удалено модератором

Модератор: ещё не забыли о чём спрашивалось в теме? Напоминаю -

Цитата

объясните в чем дело? Я и синтаксиса такого не знаю: delete[] pPayState;

2 brb: если на вопрос отвечено, то я тему закрываю, если нет, то спрашивайте по существу чтО именно не понятно.

2 Fantasist: Давайте без перехода на личности

Удалено модератором