un problème d'affichage

Version imprimable

Bonjour,
J'aurai besoin d'aide pour déboguer un code, pour vous expliquer le probleme en quelques lignes, je demande a afficher les valeurs dans un tableau en 2 reprise (c'est pour deboguer), mais je ne trouve pas la meme chose, le code est consiste simplement a donner les valeur propres d'une matrice 2x2, rien de compliqué, je resouds le determinent
je vous poste le code, il me semble que c'est un probleme classique mais je n'arrive pas a le resoudre et merci d'avance
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 void val_prop(double**vp, Image_Lab **mat) { int i; for(i=0; i<mat[0]->nblig * mat[0]->nbcol; i++) { eq2(vp[i],1,-(mat[0]->pix[i].l+mat[3]->pix[i].l),mat[0]->pix[i].l*mat[3]->pix[i].l+mat[2]->pix[i].l*mat[1]->pix[i].l); //resoud une equation du 2nd degré printf("vp[%d][0]=%lf\vp[%d][1]=%lf\n",j,vp[j][0],j,vp[j][1]); //Ici j'affiche les bonnes valeurs } for(i=0; i<(mat[0]->nblig * mat[0]->nbcol); i++) { printf("vp[%d][0]=%lf\vp[%d][1]=%lf\n",i,vp[i][0],i,vp[i][1]); //ici je n'obtiens que des zeros! contrairement a avant et c'est c'est ceci que je recupère qu main } }
Merci infiniment pour votre aide

10/01/2007, 10h55
Emmanuel Delahaye

Citation:

Envoyé par amine_en_france

J'aurai besoin d'aide pour déboguer un code,

Poste un code complet et réduit au minimum qui montre le problème. Tel quel, on ne peut rien dire. Et :tagcode:
10/01/2007, 11h05
Thierry Chappuis

Salut,

Il nous manque la définition de la fonction eq2 et celle de la structure Image_Lab, et idéalement un main minimal pour faire des tests de compilation et tenter de reproduire le problème.

Thierry

Pour al fonction eq2, c'est une simple resolution d'une equation du 2nd degré, je vous joins le code

Code:

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
 
void eq2( double x[2], double a, double b, double c)
{
          double des;
          x[0]=0;x[1]=0;
          des= b*b - 4*a*c;
          if(des<0)
          {
                 //printf("pas de solutons possibles\t");
                 x[0]=0;x[1]=0;
          }
 
          else if(des==0)
          {
                 //printf("une solution unique\t");
                  x[0]=x[1]=-b/(2*a);
          }
 
          else
          {
              //printf("2sol#\t");
              x[0]=(-b-sqrt(des))/(2*a);
              x[1]=(-b+sqrt(des))/(2*a);
          }
        //  printf("x1=%lf, x2=%lf\n", x[0],x[1]);
          //system("pause");
}

sinon la structure Image_Lab et la structure pixel_Lab appelée dans la structure de Image_Lab

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
typedef struct
{
	double l;
	double a;
	double b;
}pixel_Lab;
 
 
typedef struct
{
	int nbcol;
	int nblig;
	char comment[200];
	int max_val;
	pixel_Lab *pix;
}Image_Lab;

Merci d'avance

11/01/2007, 15h49
Thierry Chappuis

Salut,

j n'est pas déclarée, mais est utilisée dans le bout de code ci-dessous:

Code:

printf("vp[%d][0]=%lf\vp[%d][1]=%lf\n",j,vp[j][0],j,vp[j][1]);

Je ne voit à priori pas d'erreur dans les arguments passés à eq2. C'est très difficile à lire... pour une évaluation plus poussée: http://emmanuel-delahaye.developpez.com/goret.htm

Thierry
11/01/2007, 18h03
diogene

Code:

void val_prop(double**vp, Image_Lab **mat)

Probablement, le type de vp est incorrect.
vp est déclaré comme un pointeur sur un pointeur sur double ce qui correspond pour l'appel à un tableau de pointeur sur double autrement dit un tableau contenant les adresses des tableaux de deux double. La déclaration correspond à un paramètre d'appel du genre

Code:

double *X[...]

Si le paramètre correspondant à vp lors de l'appel est du genre

Code:

double X[...][2]

Le type de vp devrait être

Code:

void val_prop(double (*vp)[2], ...

Lors de l'évaluation de l'adresse de vp[i], l'adresse de départ sera augmentée de i*taille d'un pointeur (sur double) dans le premier cas et de i*2*taille d'un double dans le second ce qui n'est pas la même chose.

J'ai trouvé et corriger l'erreur

Je vous envois les codes qui m'ont permis de resoudre le problème, je n'ai pas bcp changé, et toutes les allocations sont dans le main, l'erreur est que je demandais d'envoyais un tableau statique ce qui est impossible en C

Code:

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
 
void eq2( double *x, double a, double b, double c)
{
          double des;
          x=(double *)malloc(2*sizeof(double));
          x[0]=0;x[1]=0;
          des= b*b - 4*a*c;
          if(des<0)
          {
                 //printf("pas de solutons possibles\t");
                 x[0]=0;x[1]=0;
          }
 
          else if(des==0)
          {
                 //printf("une solution unique\t");
                  x[0]=x[1]=-b/(2*a);
          }
 
          else
          {
              //printf("2sol#\t");
              x[0]=(-b-sqrt(des))/(2*a);
              x[1]=(-b+sqrt(des))/(2*a);
          }
}
 
void val_prop(double**vp, Image_Lab **mat)
{
 
       int i;
       for(i=0; i<mat[0]->nblig * mat[0]->nbcol; i++)
       {
		eq2(vp[i],1,-(mat[0]->pix[i].l+mat[3]->pix[i].l),mat[0]->pix[i].l*mat[3]->pix[i].l+mat[2]->pix[i].l*mat[1]->pix[i].l);
       }
}

Merci comme meme pour votre aide

12/01/2007, 09h47
diogene

Citation:

l'erreur est que je demandais d'envoyais un tableau statique ce qui est impossible en C

Pourquoi est-ce impossible ?
12/01/2007, 12h28
Emmanuel Delahaye

Citation:

Envoyé par diogene

Pourquoi est-ce impossible ?

Parce que c'est un choix des concepteurs le langage C. (et ce choix n'a pas été fait pour les structures).

Es-tu certain que ton problème est résolu ?
Parceque j'ai un sacré doute sur la façon d'écrire ceci:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
void eq2( double *x, double a, double b, double c)
{
          [...]
          x=(double *)malloc(2*sizeof(double)); 
          [...]
}

void val_prop(double**vp, Image_Lab **mat)
{

       [...]
       for(i=0; i<mat[0]->nblig * mat[0]->nbcol; i++)
       {
		eq2(vp[i],.............);
       }
}

Emmanuel

Citation:

Parce que c'est un choix des concepteurs le langage C. (et ce choix n'a pas été fait pour les structures).

Oui. Ce n'est pas ce que je voulais souligner (une variable locale de type tableau ne peut être créée automatiquement (et initialisée) par le simple passage d'un tableau par paramètre). Dans tous les cas, statique ou non, c'est l'adresse du premier élément du tableau qui est passée en paramètre et la variable locale créée et initialisée est un pointeur. Mais je ne voyais pas comment le changement de

Code:

void eq2( double x[2], double a, double b, double c)

en

Code:

void eq2( double *x, double a, double b, double c)

qui est équivalent peut résoudre un problème de "tableau statique"
Si on voit plus loin la version dite "correcte"
Code:

1 2 3 4 5 6 7 void eq2( double *x, double a, double b, double c) { double des; x=(double *)malloc(2*sizeof(double)); //Etrange, on écrase le paramètre x x[0]=0;x[1]=0; .... // et pas de return x !!!
On se demande comment cette fonction peut bien marcher et servir même à faire quelque chose.
C'est pourquoi, javais des doutes sur le sens à donner à sa phrase

Citation:

l'erreur est que je demandais d'envoyais un tableau statique ce qui est impossible en C

et demandais des éclaircissements à l'auteur

13/01/2007, 23h40
Médinoc

D'ailleurs, j'aurais interdit la première écriture, puisqu'elle est trompeuse.
14/01/2007, 11h19
gl

Citation:

Envoyé par Médinoc

D'ailleurs, j'aurais interdit la première écriture, puisqu'elle est trompeuse.

Malheureusement pour des tableaux multidimensionnel, il n'y a pas trop le choix
14/01/2007, 12h05
Médinoc
Ben si, pour les tableaux multidimensionnels, on a ça: Là encore, on remplace la dimension la plus "extérieure" par un pointeur:
Code:

1 2 3 4 5 6 7 8 9 void TestArray2(int (*tab)[5]) { } void TestArray(void) { int theTab[7][5]; TestArray2(theTab); }
14/01/2007, 14h47
gl

Citation:

Envoyé par Médinoc

Ben si, pour les tableaux multidimensionnels, on a ça: Là encore, on remplace la dimension la plus "extérieure" par un pointeur:

Et on conserve bien la notation "tableau" pour certaines dimensions. Tu utilises donc bien l'ecriture precedente, sauf pour une des dimensions du tableau.
14/01/2007, 14h48
Médinoc
Sauf que là, on ne passe plus un tableau en paramètre, mais un pointeur vers tableau. Et là,
1. la dimension compte réellement
2. on n'a pas l'illusion potentielle de passer un tableau par valeur.
14/01/2007, 19h31
gl

Citation:

Envoyé par Médinoc

Sauf que là, on ne passe plus un tableau en paramètre, mais un pointeur vers tableau.

De toute maniere on ne passe jamais un tableau en parametre, mais toujours un pointeur sur le premier element du tableau (et c'est peut etre la que se situe le probleme).

Citation:

Envoyé par Médinoc

la dimension compte réellement

La dimension compte uniquement pour l'acces aux elements (sinon comment serait-il possible d'utiliser [x][y]).
A une dimension, elle n'est pas necessaire. En dehors de ca il n'y a pas de difference.
sizeof par exemple continue de fournir la taille d'un pointeur.

Citation:

Envoyé par Médinoc

on n'a pas l'illusion potentielle de passer un tableau par valeur.

Ca reste subjectif. On peut avoir l'impression de passer un tableau de pointeur.
15/01/2007, 13h20
diogene
Je suis plutôt de l'avis de Medinoc sur le choix "pointeur" dans le choix entre la notation tableau et la notation pointeur dans la liste des paramètres. La notation pointeur a l'avantage de représenter correctement le type de la variable locale créée. Toutefois la notation tableau peut avoir l'avantage de préciser que l'argument réel est l'adresse d'un tableau, pas d'une variable simple. Dans ce cas, je préfère exclure la spécification de la valeur des dimensions "inutiles", puisqu'elle n'ont pas de sens pour la fonction.
En fait, tout dépend si on préfére préciser le type de la variable locale ou celui de la variable à l'appel.

Code:

void TestArray2(int (*tab)[5])

Citation:

Envoyé par gl

Ca reste subjectif. On peut avoir l'impression de passer un tableau de pointeur.

Je ne crois pas (surtout avec le parenthésage requis). On écrirait dans ce cas et suivant les préférences:
Code:

1 2 3 void TestArray2(int *tab[]) // sans la dimension ou void TestArray2(int ** tab)
15/01/2007, 18h58
gl

Citation:

Envoyé par diogene

Je suis plutôt de l'avis de Medinoc sur le choix "pointeur" dans le choix entre la notation tableau et la notation pointeur dans la liste des paramètres. La notation pointeur a l'avantage de représenter correctement le type de la variable locale créée.

Moi aussi, j'ai une preference pour cette notation.
Mon propos etait juste de faire remarquer que l'on ne pouvait pas completement se passer de la notation sous forme de tableau et qu'il restait des cas trompeurs

Citation:

Envoyé par diogene

Je ne crois pas (surtout avec le parenthésage requis). On écrirait dans ce cas et suivant les préférences:

Pour toi, et plus generalement pour quelqu'un ayant une certaine experience avec le C et les pointeurs, peut etre pas.
Pour quelqu'un de debutant ayant du mal avec la notation sous forme de tableau et le passage de tableau en parametre, je ne suis pas certain du tout.