Объявить указатель внутри функции, которая вызывается несколько раз

Question

Я хотел бы знать, могу ли я сделать это с помощью функции Idle(), которая вызывается несколько раз в программе opengl. Это нормально или я получу память утечка?

void Idle() 
{ 
    //PXCPointF32 is a struct 
    PXCPointF32 *uvmap=0; 

    uvmap=new PXCPointF32[640*480]; 
    if(uvmap!=NULL) 
    { 
     //do some processing 
     //fill uvmap values 
    } 

    if (uvmap) 
    { 
     delete[] uvmap; 
     uvmap=NULL; 
    } 
} 

Answer 1

код должен работать, но его не оптимален:

void Idle() 
{ 
    //PXCPointF32 is a struct 
    PXCPointF32 *uvmap=0; // why initialize to 0 only to replace its value? 

    uvmap=new PXCPointF32[640*480]; 
    if(uvmap!=NULL) 
    { 
     //do some processing 
     //fill uvmap values 
    } 

    if (uvmap) // no need to test for null before deleting 
    { 
     delete[] uvmap; 
     uvmap=NULL; // no point setting to NULL because its local 
    } 
} 

Вместо этого вы могли бы сделать:

void Idle() 
{ 
    // set it directly 
    PXCPointF32* uvmap = new PXCPointF32[640*480]; 

    if(uvmap) 
    { 
     //do some processing 
     //fill uvmap values 
    } 

    delete[] uvmap; // don't bother checking for null (delete[] does that) 
} 

Вы также можете рассмотреть вопрос о смарт-указатель, чтобы очистить память в случае Исключение:

void Idle() 
{ 
    // set it directly 
    std::unique_ptr<PXCPointF32[]> uvmap(new PXCPointF32[640*480]); 

    if(uvmap) 
    { 
     //do some processing 
     //fill uvmap values 
    } 

    // no delete[] here 
} 

Answer 2

Нет причин для утечки памяти. Но это не действительно художественное программирование. В такой ситуации я бы предпочел резервировать память один раз в начале программы и удерживать ее до конца программы, чтобы избежать фрагментации памяти и штрафов за скорость. Также код имеет некоторые «грузовые» линии:

void Idle() 
{ 
//PXCPointF32 is a struct 
PXCPointF32 *uvmap=new PXCPointF32[640*480]; 

if(uvmap==NULL) 
{ 
    // panic and return 
} 

//do some processing 
//fill uvmap values 

delete[] uvmap; 
// uvmap=NULL; out of scope anyway 
} 

Answer 3

Это не будет течь, если только не будет сгенерировано исключение до удаления и после того, как новый. Использование интеллектуального указателя предотвратит это.

Answer 4

Я предполагаю, что эта функция называется частью потока GUI. Это означает, что мне не нужно беспокоиться о том, что несколько потоков обращаются к этой функции одновременно.

Итак, давайте просто выделим память статически.

void Idle() 
{ 
    //PXCPointF32 is a struct 
    static PXCPointF32 uvmap[640*480]; 

    //do some processing 
    //fill uvmap values 
} 

Этот код не может просочиться в память (поскольку он не выделяет). Ваш код также прекрасен, если ничего в Idle() не запускает исключение.