trop de pointeur

**Aquadrox** · 30/05/2005, 12h11

Salut tout le monde, j'ai un programme a traduire mais y' a un peut trop de pointeur a mon gou, pourriez vous m'aider a le comprendre??
Merci d'avance

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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
void filter_pwv_data(float *X,float *Y,double *mem_1,double *mem_2)
{
	float *x,*y;
	double *m1,*m2;
	double o,w;
	long i,j;
	double *temp_a_coefs,*temp_b_coefs;
 
	x = X;
	y = Y;
 
	for(i=0; i<(ivec_length-1); i++)
	{
		m1 = mem_1;
		m2 = mem_2;
 
		temp_a_coefs = a_coefs;
		temp_b_coefs = b_coefs;
 
		o = *x++ * gain;
 
		for(j=0; j<numb_quads; j++)
		{
			*temp_b_coefs++;
			w = o - *m1 * *temp_b_coefs++;
			w = w - *m2 * *temp_b_coefs++;
 
			*temp_a_coefs++;
			o = w + *m1 * *temp_a_coefs++;
			o = o + *m2 * *temp_a_coefs++;
 
			*m1++ = w;
		}	*m2++ = *m1;
 
 
		*y++ = (float)o;
	}
}

**Caine** · 30/05/2005, 12h31

Bonjour,

Code :

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

1
2
 
filter_pwv_data(float *X,float *Y,double *mem_1,double *mem_2)

Signifie que la fonction modifie ses arguments, en C++, mieux vaut utiliser les références.

Code :

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

1
2
 
filter_pwv_data(float& X,float& Y,double* mem_1,double* mem_2)

Code :

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

1
2
3
 
x = X;
y = Y;

On assigne à des pointeurs locaux les adresses des argurments X et Y.
Je pense que c'est un sécurité pour la réentrance de la fonction.

x et y n'auront plus besoin d'être des pointeurs avec les références.

Avec les références, la déclaration devient:

Code :

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

1
2
3
4
5
6
 
float x,y;
double m1,m2;
double o,w;
long i,j;
double *temp_a_coefs, *temp_b_coefs;

Code :

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

1
2
 
o = *x++ * gain;

Ici *x++ déréférence le pointeur local x, et le multiplie à gain.
Avec les références:

Code :

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

1
2
 
o = x++ * gain;

Note, dans le code initial, il y a une imprécision, car ++ est prioritaire sur *, je ne suis pas sûr que le code soit bon.

Code :

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

1
2
 
*temp_b_coefs++;

l'opérateur ++ est prioritaire sur *, donc on incrémente le pointeur d'un case mémoire. Mais je ne suis pas sûr que c'est ce qui est voulu.

Information, mem1 et mem2 sont des tableaux.

w = o -*m1 **temp_b_coefs++;

Ici, on multiplie le contenu de m1 avec celui de temp_b_coefs, puis on va à la case mémoire suivante.

Envoyé par Aquadrox

Salut tout le monde, j'ai un programme a traduire mais y' a un peut trop de pointeur a mon gou, pourriez vous m'aider a le comprendre??
Merci d'avance

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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
void filter_pwv_data(float *X,float *Y,double *mem_1,double *mem_2)
{
	float *x,*y;
	double *m1,*m2;
	double o,w;
	long i,j;
	double *temp_a_coefs,*temp_b_coefs;
 
	x = X;
	y = Y;
 
	for(i=0; i<(ivec_length-1); i++)
	{
		m1 = mem_1;
		m2 = mem_2;
 
		temp_a_coefs = a_coefs;
		temp_b_coefs = b_coefs;
 
		o = *x++ * gain;
 
		for(j=0; j<numb_quads; j++)
		{
			*temp_b_coefs++;
			w = o - *m1 * *temp_b_coefs++;
			w = w - *m2 * *temp_b_coefs++;
 
			*temp_a_coefs++;
			o = w + *m1 * *temp_a_coefs++;
			o = o + *m2 * *temp_a_coefs++;
 
			*m1++ = w;
		}	*m2++ = *m1;
 
 
		*y++ = (float)o;
	}
}

**gege2061** · 30/05/2005, 12h33

Bonjour,

Envoyé par Aquadrox

y' a un peut trop de pointeur a mon gou,

Les pointeurs son des variables comme les autres: http://chgi.developpez.com/pointeur/

Envoyé par Aquadrox

Code :

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

*x++

Vérifie l'ordre de priorité des opérateurs: http://www-clips.imag.fr/commun/bern...C/node100.html
Avec ce code, tu mofifie l'adresse pointée par x.

Envoyé par Aquadrox

Code :

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

1
2
x = X;
y = Y;

Est ce bien utile

:

Bref l'algo ne semble pas compliquer mais le code oui.

**Aquadrox** · 30/05/2005, 12h52

Ne vous souciez pas de l'exactitude du code, il est juste et marche tres bien, je m'occupe juste de le translater dans un autre langage, mais bon pour cela il faut le comprendre avant

Merci pour votre aide je vais me pencher la dessus

**Caine** · 30/05/2005, 12h58

J'ai été assez clair dans mes explications ou tu es encore dans le flou?

**Aquadrox** · 30/05/2005, 14h15

C'etait parfait merci!

**Emmanuel Delahaye** · 30/05/2005, 14h17

Envoyé par Aquadrox

Code :

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

			*temp_b_coefs++;

n'a aucun sens. C'est soit

Code :

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

			temp_b_coefs++;

soit

Code :

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

			(*temp_b_coefs)++;

mat.M · 30/05/2005, 14h22

double *temp_a_coefs,*temp_b_coefs;

comme Emmannuel je ne vois vraiment pas l'intérêt de déclarer des pointeurs
Totalement inutile et source de plantages si mauvaise gestion

**Aquadrox** · 30/05/2005, 14h25

Oui je suis d'accord mais le porgrammes n'est pas de moi, je ne fait que le changer de langage, de mon cote j'evite les pointeur si ce n'est pas necessaire

**Caine** · 30/05/2005, 14h39

Envoyé par mat.M

double *temp_a_coefs,*temp_b_coefs;

comme Emmannuel je ne vois vraiment pas l'intérêt de déclarer des pointeurs
Totalement inutile et source de plantages si mauvaise gestion

Sauf pour blinder le code par rapport au Multithreading.

**Caine** · 30/05/2005, 14h41

Envoyé par Aquadrox

Oui je suis d'accord mais le porgrammes n'est pas de moi, je ne fait que le changer de langage, de mon cote j'evite les pointeur si ce n'est pas necessaire

Astuce: Si tu veux enlever les deux dernier pointeurs, qui visiblement sont des "tableaux ouverts" façon C, il te faudra passer par un vector ou un list.

Ha oui,

Code :

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

1
2
 
*x++

si le développeur initial connassait la préséance des opérateurs signifie:

Code :

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

1
2
 
*(x++)

sino, comme 90 % des newbies, il a cru coder:

Code :

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

1
2
 
(*x)++

Je te conseille avant de transformer l'ancien code de le tester pour t'assurer qu'il n'y avait pas de bug. Si tu n'en a pas la possibilité il faut analyser l'algorithme pour savoir si la forme codée est correcte.

N'hésites pas à demander, je connais bien ces deux langages.

**Aquadrox** · 30/05/2005, 15h21

C'etait loin d'etre un newbie c'est un ingenieur en informatique, pour ce qui est du prog il est teste et marche tres bien de A a Z, merci pour ton aide.

PS: desole j'ai pas d'accent sur le clavier ici et j'ai pas envi de passer ma vie a copier coller des caracteres speciaux...

**Trap D** · 30/05/2005, 15h28

Envoyé par Caine

Ha oui,

Code :

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

1
2
 
*x++

si le développeur initial connassait la préséance des opérateurs signifie:

Code :

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

1
2
 
*(x++)

sino, comme 90 % des newbies, il a cru coder:

Code :

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

1
2
 
(*x)++

Hum hum, dans le code *x++, la valeur est obtenue AVANT l'incrémentation de x.

Ce petit programme de démo :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#include <stdio.h>
 
int main(void)
{
  char *p = "bonjour";
  char *q = p;
 
  char c= *q++;
 
  printf("Valeur de c '%c'\nvaleur de p %p\nvaleur de q %p\n", c, p, q);
  return 0;
}

sortie :

Valeur de c 'b'
valeur de p 00420054
valeur de q 00420055

**Aquadrox** · 30/05/2005, 15h31

Merci pour ta demo ca eclair tout!

**Emmanuel Delahaye** · 30/05/2005, 15h32

Envoyé par Aquadrox

C'etait loin d'etre un newbie c'est un ingenieur en informatique,

Huh! Et il ecrit *x++ comme ça, sans vergogne ? Demande lui ce qu'il a voulu faire, parce que ce n'est pas clair du tout.

C'est x++ ou (*x)++, ce qui n'a pas du tout la même signification.

Si tu dois le traduire dans un autre langage, il faut absolument connaitre les intentions du codeur initial, fusse-t-il ingénieur (ça mord pas)....

pour ce qui est du prog il est teste et marche tres bien de A a Z, merci pour ton aide.

Alors il faut probablement enlever cette '*' inutile, on y verra plus clair...

PS: desole j'ai pas d'accent sur le clavier ici et j'ai pas envi de passer ma vie a copier coller des caracteres speciaux...

Aucune importance. De toutes façons, les accents ne sont pas portables en C

**Emmanuel Delahaye** · 30/05/2005, 15h36

Envoyé par Trap D

Hum hum, dans le code *x++, la valeur est obtenue AVANT l'incrémentation de x.

Quelle valeur ? * ne fait rien dans ce cas là :
C'est pas

Code :

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

y = *x++;

c'est

Code :

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

*x++;

tout cours. Relis le code original... ce que je viens de faire...

Il s'agit bien sûr de

Code :

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

*temp_b_coefs++;

car effectivement,

Code :

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

o = *x++;

est à peu près clair (mais je l'écris toujouirs en deux fois pour éviter les ambiguités de relecture de code)

Code :

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

1
2
o = *x;
x++;

**gege2061** · 30/05/2005, 15h44

Envoyé par Emmanuel Delahaye

C'est x++ ou (*x)++, ce qui n'a pas du tout la même signification.

*x++ a une signification, c'est la même chose que:

Code :

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

1
2
3
4
5
6
int i = 0;
<...>
x[i];
<...>
i++;
<...>

Mais dans ce cas, on ne garde pas de trace du pointeur d'origine (x[0])

**Aquadrox** · 30/05/2005, 15h47

Envoyé par Emmanuel Delahaye

Huh! Et il ecrit *x++ comme ça, sans vergogne ? Demande lui ce qu'il a voulu faire, parce que ce n'est pas clair du tout.

Malheureusement il est parti dans une autre societe sinon c'est lui qui se serai taper le travail!! lol
Sinon quand il a ecris ca ou bien il a eu un gros coup de chance pour que ca marche ou bien il avait deja essaye avant, le principal etant que ca marche.
Pour ce qui est de la clarete je te l'accorde ca laisse a desirer...