Не будет ли в данном случае многократное использование оператора new плохо сказываться на работе программы

[robix]

Здравствуйте, хотел уточнить, не будет ли в данном случае многократное использование
оператора new плохо сказываться на работе программы.
Имеется цикл do while c частотой 10 раз в секунду.

1. Первый вариант
Bitmap * F_Bitmap;
gdiBitmapData - это ссылка на массив пикселей. Он периодически меняет значения.
Color TheColor; //Цвет
int IMAGE_W = 800; // Ширина изображения в пикселях
IMAGE_W = 600; // Высота изображения в пикселях

do
{
MyFuncUpdateArrayOfPixels(gdiBitmapData); //Моя функция устанавливающая значения пикселей
//Создаем изображение через конструктор, передавая массив пикселей gdiBitmapData
F_Bitmap = new Bitmap(800,600,stride_size,f_format,gdiBitmapData);
}
while (myexit);

Но есть способ делать так:
2. Второй вариант

do
{
MyFuncUpdateArrayOfPixels(gdiBitmapData); //Моя функция устанавливающая значения пикселей
for (x = 0; x < IMAGE_W; x++)
for (y = 0; y < IMAGE_H; y++) {
TheColor = MyFuncGetColorFromArray(gdiBitmapData,x,y); //Моя функция возвращающая цвет
F_Bitmap.SetPixel(x,y,Color); //Меняем значение пикселя у однажды созданного объекта изображения
}
}
while (myexit);

Как думаете по производительности будет ли разница меду способами 1 и 2 ?
Если постоянно делаю new Bitmap, то памяти не будет больше потребляться, чем в варианте 2 ?
Вообще после new память освобождается?

[p51x]

Вообще после new память освобождается?

Ну если вы его переопределили, а так delete не зря ж есть... А по коду у вас утечка памяти.

[robix]

Спасибо, то есть получается что после
F_Bitmap = new Bitmap(800,600,stride_size,f_format ,gdiBitmapData);
память ранее выделенная под F_Bitmap остается в оперативной памяти программы и оперативная память постоянно растет?

А если я буду делать в цикле так, это нормально?
if(F_Bitmap) delete F_Bitmap;
F_Bitmap = new Bitmap(800,600,stride_size,f_format ,gdiBitmapData);

А по производительности эти два варианта как-то будут отличаться?

[ResourceSpace]

На чём пишете? Что есть Bitmap? А то вот такая штука бывает: http://www.vsokovikov.narod.ru/New_M...bitmapbits.htm

[jura_k]

Если есть new - должен быть delete.

"Если есть тьма - должен быть свет." (В. Цой)

[Krasiosoft]

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

F_Bitmap = new Bitmap(800,600,stride_size,f_format ,gdiBitmapData);
память ранее выделенная под F_Bitmap остается в оперативной памяти программы и оперативная память постоянно растет?

F_Bitmap - это просто указатель на объект класса Bitmap, если мы его изменим, предварительно не сохранив куда-то, чтоб потом можно было освободить память через delete, то будет утечка.

[robix]

Спасибо всем. Это просто для примера все.
Я понял, что если есть new, то должен быть delete.
Но использование в цикле new .... delete это не есть хороший стиль, если имеются альтернативные варианты или же это нормальная практика?

Тут суть в том, что в ЦИКЛЕ либо 1). методами класса перезаписывать массив-переменную класса, либо 2). создавать через new c предварительным delete объект класса, с передачей в него уже готового массива-переменной. Является ли вариант 2 не приемлемым? Или не хорошим стилем. Или это все мелочи, и то и то нормально?)

[Krasiosoft]

Если Вы просто загружаете изображение в Bitmap, то переписывать попиксельно тут не вариант (можете сами проверить, сколько по времени занимает один вариант и второй вариант). В классе Bitmap должны быть методы, которые позволяют загружать изображение целиком, не думаю, что только конструктор позволяет это сделать.

Но я так и не понял, какое отношение к этому всему имеет new и delete?

[Stilet]

Но использование в цикле new .... delete это не есть хороший стиль, если имеются альтернативные варианты или же это нормальная практика?

Для одной задачи это нормальная практика, для другой нет.
Какова цель таких действий? Для чего тебе это понадобилось? Может лучше использовать какой-нибудь ImageList для кучи картинок?

Тут суть в том, что в ЦИКЛЕ либо

Сделай объект битмапа до цикла, и освободи после. Один единственный. И с ним работай.
А пока непонятно что тебе надо, но работа попиксельно будет медлительна.

[jura_k]

Как только объект не нужен становится и указатель на него теряется, следует использовать delete.
Если уж совсем непонятно, можно создать "временный" указатель, копировать туда основной, а на следующей итерации цикла удалять старый объект через "временный" указатель. Но это такой способ, когда уж совсем ничего не помогает.