Discussion :

[squale69]

Bonjour à tous,

Au secours, je n'y connais rien en C/C++ et il me le rend bien

Je vous soumet un petit morceau de code (pour info buffOut représente une série de pixel de valeur variant de 0 à 255 et je cherche bêtement le 1° pixel non nul).

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
int _tmain(int argc, _TCHAR* argv[])
{
	unsigned char* buffOut = new unsigned char[10];
	int j;
 
	buffOut[0] = 0;
	buffOut[1] = 0;
	buffOut[2] = 255;
	j = 0;
 
	while (buffOut[j] = 0)
	{
		j++;
	}
	return 0;
}

Pouquoi l'expression derrière le while n'est pas un test logique ? (pas tapé sur la tête)
Merci.

[groscask]

Tout bêtement parce que c'est une affectation ;-)

Code :

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

1
2
3
 
buffOut[j] = 0; /* Affectation */
buffOut[j] == 0; /* Test */

Rajoute donc simplement un signe égal et le tour sera joué ! (au moins pour ça...)

EDIT:

Hep hep hep ! Ton PC c'est pas une poubelle : alors quand tu alloues de la mémoire de manière dynamique (new...) il faut aussi penser à la libérer

Code :

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

1
2
3
 
delete buffOut;
return 0;

[squale69]

Arff ... je suis vraiment une bille !
2h que je tourne autour.
Merci.

[granquet]

new c'est du C++ ... c'est le forum d'à coté

[groscask]

Aussi ouais... Mais chut, laissons-le savourer sa joie ;-)

[Sve@r]

Hep hep hep ! Ton PC c'est pas une poubelle : alors quand tu alloues de la mémoire de manière dynamique (new...) il faut aussi penser à la libérer

Code :

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

1
2
3
 
delete buffOut;
return 0;

On se demande d'ailleurs pourquoi une allocation dynamique...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
int _tmain(int argc, _TCHAR* argv[])
{
	unsigned char buffOut[10];
	int j;

	buffOut[0] = 0;
	buffOut[1] = 0;
	buffOut[2] = 255;
	
	for (j=0; buffOut[j] == 0; j++);
	return 0;
}

Et hop, on ne se rend même plus compte qu'il s'agit de C++...

[Trap D]

Code :

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

int _tmain(int argc, _TCHAR* argv[])

On se rend surtout compte que c'est du Micro$oft

[Médinoc]

1.

Code :

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

int _tmain(int argc, _TCHAR* argv[])

On se rend surtout compte que c'est du Micro$oft

Et alors ?

2. En effet, c'est du C++. D'un autre côté, Squale69 a annoncé la couleur en disant "C/C++"...

3. Attention, en C++ pour un tableau on utilise delete[], pas delete.
Il est possible que sous certaines implémentation, ça "tombe en marche" dans le cas de chars (pour des raisons d'optimisation), mais en règle générale, utiliser delete quand il faut un delete[] est un comportement indéfini, donc un bug.

PS: Ça fait beaucoup de petits nouveaux qui viennent du Rhône, dis donc...
squale69, sylvain69, lucas69...

[groscask]

Oh non !

Mea culpa pour cette erreur indigne...

Mériterais-je un jour de remonter dans les bonnes grâces des développeurs C(++) ?