Очистить массив до NULL, не зная тип

Question

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

template <typename T> 
    static void Nullify(T** _Array, int _Size, unsigned int _SizeOf) 
    { 
    for (int i = 0; i < _Size; i++) 
    { 
    _Array[i * _SizeOf] = NULL; 
    } 
    } 

Что дает мне ошибки (не сама функция, но я попирая память не должна быть попирая в результате ошибки памяти позже). Я передаю массив (_Array - уже инициализирован _Size), его размер (_Size) и sizeof (Pointer *) как _SizeOf.

Любая помощь будет принята с благодарностью :)

Answer 1

Вам не нужно _SizeOf. Это то, что вы хотите:

template <typename T> 
static void Nullify(T** _Array, int _Size) 
{ 
    for (int i = 0; i < _Size; i++) 
    { 
     _Array[i] = NULL; 
    } 
} 

Компилятор знает размер указателя, и делает математику для вас во время разыменования массива.

Answer 2

Вы не должны делать _Array[i * _SizeOf] = NULL, а скорее _Array[i] = NULL.

Answer 3

Я согласен с ответом Рида, конечно же, вы также должны удалить '* _SizeOf' в своем задании.

Если возможно, я также предлагаю вам отойти от «сырых» массивов и использовать в вашем коде std :: vector. Это более безопасная структура с массивной семантикой и накладывает лишь небольшие накладные расходы.

Answer 4

Вы не должны полагаться на значение _SizeOf вашего значения, чтобы узнать его тип, вы должны использовать функцию, чтобы определить ее тип.

В какой момент вы объявляете свой T** _Array? Возможно, здесь вы должны убедиться, что ваш метод гарантирует определенный тип Array для любой части вашей программы, которая его называет, не так ли?

Answer 5

template <class T> 
void Nullify(T** the_array, unsigned int size) 
{ 
    std::fill(the_array, the_array + size, static_cast<T*>(0)); 
} 

template <class T, unsigned int N> 
void Nullify(T* (&the_array)[N]) 
{ 
    Nullify(the_array, N); 
} 

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

Foo* x[10] = {...} 
Nullify(x); // sets all pointers in x to 0 (NULL) 

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

template <class T> 
void Nullify(T** the_array, unsigned int size) 
{ 
    for (unsigned int j=0; j < size; ++j) 
     the_array[j] = 0; 
} 

Обратите внимание, что я также не использовать NULL по той же причине Страуструп делает это (исключает необходимость включения cstddef). Когда C++ 0x будет более широко внедряться, ключевое слово nullptr сделает приятную замену.

BTW: ключевое слово static игнорируется для шаблонов функций. У них уже есть внутренняя связь, поэтому лишний добавить ее.