Discussion :

[nocolach]

Bonjour,

je recontre actuellement un probleme etrange:

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 !

[Davidbrcz]

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

[nocolach]

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.

[Davidbrcz]

Tu gagnerai à changer mais bon.

Si je regarde mon man memset, j'ai:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

devrait plutôt être remplacé par :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

memset(countIntensity ,0,numValues);

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

[Pragmateek]

Ne serait-ce pas plutôt :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

?

[Davidbrcz]

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

[white_tentacle]

J'écrirai plutôt

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 .

[Pragmateek]

J'écrirai plutôt

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

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.

[white_tentacle]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

La faute à

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

qui lui marche...

[Pragmateek]

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

[jmv]

pourquoi ne pas utiliser std::fill() ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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