initialisation array avec memset

Bonjour,

je recontre actuellement un probleme etrange:

je cree un tableau je souhaite l'initialiser avec memset:

Code:

---

1 2 3

unsigned int *countIntensity = new unsigned int[numValues]; memset(countIntensity ,0,sizeof(countIntensity));

---

cependant cette initialisation ne marche pas en effet, juste apres,

Code:

---

1 2 3 4 5

std::cout << "countIntensity[0]: " <<countIntensity[0] << std::endl; std::cout << "countIntensity[1]: " <<countIntensity[1] << std::endl; std::cout << "countIntensity[2]: " <<countIntensity[2] << std::endl; std::cout << "countIntensity[3]: " <<countIntensity[3] << std::endl;

---

retourne
0
0
13113
29


Merci de votre aide !

Pourquoi passer par un tableau alloué dynamiquement au lieu d'utiliser un std::vector ?

je suis actuellement en train de debugger un gros code donc j'essaye de corriger les bugs en utilisant ce qui est deja present dans le code. Tout rechanger prendrait enormement de temps.

Tu gagnerai à changer mais bon.

Si je regarde mon man memset, j'ai:

Code:

---

void *memset (void *s, int c, size_t n);

---

Citation:

---

La fonction memset() remplit les n premiers octets de la zone de mémoire pointée par s avec l'octet c.

---

Ce qui me ferai penser que

Code:

---

memset(countIntensity ,0,sizeof(countIntensity));

---

devrait plutôt être remplacé par :

Code:

---

memset(countIntensity ,0,numValues);

---

Car sizeof(countIntensity) ne renvoit pas la longeur du tableau.

Ne serait-ce pas plutôt :

Code:

---

memset(countIntensity ,0,numValues * sizeof(*countIntensity));

---

?

Oui, c'est un oubli de ma part (j'utilise jamais memset, ca s'explique ^^)

J'écrirai plutôt

Code:

---

memset(countIntensity ,0,numValues * sizeof(unsigned int));

---

Mais il y a des subtilités avec l'opérateur sizeof quand il prend des variables, que je connais très mal pour les éviter comme la peste, donc peut-être que ce que tu as écrit est juste aussi ;).

Citation:

---

Envoyé par white_tentacle

J'écrirai plutôt

Code:

---

memset(countIntensity ,0,numValues * sizeof(unsigned int));

---

Mais il y a des subtilités avec l'opérateur sizeof quand il prend des variables, que je connais très mal pour les éviter comme la peste, donc peut-être que ce que tu as écrit est juste aussi ;).

---

L'opérateur sizeof n'évalue pas son expression donc il n'y a pas de risque à l'utiliser sur un pointeur, y compris NULL.
Ce comportement est spécifié dans la norme ISO/CEI 14882:2003 5.3.3.1 :

Citation:

---

The operand is either an expression, which is not evaluated,...

---

Cela permet donc de faire abstraction du type de l'objet pointé.
Si plus tard ce sont des double qui sont utilisés à la place des unsigned int l'opération memset n'aura pas à être modifiée, le compilateur inférant le type de *countIntensity.

Citation:

---

L'opérateur sizeof n'évalue pas son expression donc il n'y a pas de risque à l'utiliser sur un pointeur, y compris NULL.

---

Pas de risque côté compilateur, mais j'ai trop souvent vu des choses comme ça pour en recommander l'utilisation :

Code:

---

1 2 3

char* buf = new char[50] for(int i = 0; i < sizeof(buf); ++i) // Mauvais code buf[i] = 0;

---

La faute à

Code:

---

1 2 3

char buf[50] for(int i = 0; i < sizeof(buf); ++i) buf[i] = 0;

---

qui lui marche...

Oui en effet, les tableaux étant bien trop souvent identifiés à des pointeurs.

pourquoi ne pas utiliser std::fill() ?

Code:

---

std::fill(countIntensity ,countIntensity+numValues , 0);

---