problème avec une fonction

**M77ATTAR** · 28/02/2013, 12h59

bonjour;
dans un tableau de points je veux afficher seulement ceux qui vérifient une équation pour cela j'ai écris la fonction suivant:

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
 
typedef struct Point
{
float x;float y;float z;
};
 
Point *equation(Point a,Point b,Point tableau[taille])
{
Point *sorti=(Point *)malloc(taille*sizeof(Point )); 
int cpt=0;
float equation;
float landa=-((a.x*b.x)+(a.y*b.y)+(a.z*b.z));
 
for(int i=0;i<taille;i++)
{
equation=((a.x*tableau[i].x)+(a.y*tableau[i].y)+(a.z*tableau[i].z))+landa;
 	if(equation==0.0) 
	{ sorti[cpt].x=p[i].x;sorti[cpt].y=p[i].y;sorti[cpt].z=p[i].z;cpt++; }
}  
   return sorti;
}

malheuresement elle me renvoit des valeurs qui n'existe plus dans le tableau initial !

cordialement

**matafan** · 28/02/2013, 13h25

Ton code n'est pas correct, il ne devrait même pas compiler. On ne peut pas passer la taille d'un tableau de cette façon. Si tu veux passer à ta fonction un tableau et sa taille, il faut le faire en deux arguments : Point tableau[] et int taille.

**darkwall_37** · 28/02/2013, 20h17

Salut,

Je n'ai pas testé le code, mais cela devrait aller mieux comme ça.
Un conseil évident, fait en sorte que ton code soit plus lisible.
T'as une variable p qui traine, non déclarée que j'ai surligné en rouge dans le code.

Cordialement.

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58

typedef struct 
{
    float x;
    float y;
    float z;
}t_point;

t_point *point_create_struct(unsigned int x)
{
    t_point *p = NULL;
    if ( !(p=malloc(x*sizeof(t_point))) )
        printf("\n point_allocate_struct : echec de l'allocation");
    
    return p;
}

void point_destroy_struct(t_point *p)
{
    if(p)
        free(p), p=NULL;
    
    else printf("\n point_destroy_struct : p is already NULL");
}


t_point *equation(t_point *a, t_point *b, t_point *tab, unsigned int length)
{
    t_point *r=NULL;
    
    if ( (a && b && tab) )
    {
        if ( (r=point_create_structure(length)) );
        {
            unsigned int cpt=0;
            float r_equation=0.;
            float lambda = -((a->x*b->x)+(a->y*b->y)+(a->z*b->z));

            unsigned int i;
            for(i=0;i<length;i++)
            {
                r_equation = ((a->x*tab[i]->x)+(a->y*tab[i]->y)+(a->z*tableau[i]->z))+lambda;
                if( r_equation == 0.0 )
                { 
                    tab[cpt]->x=p[i]->x;
                    tab[cpt]->y=p[i]->y;
                    tab[cpt]->z=p[i]->z;
                    cpt++; 
                }
            }
        }
        else printf("\n equation : Echec de l'allocation de r");
    }
    else printf("\n equation : Au moins un des arguments est NULL");
    
    return r;
}

**M77ATTAR** · 01/03/2013, 00h39

-à matafan: c'est vrai merci pour la remarque.
-à darkwall_37: voilà j'ai modifier ce que tu ma proposé comme suit (ça marche bien,mais il reste un petit problème):

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
#include <stdio.h>
#include <stdlib.h>
 
typedef struct 
{
    float x;
    float y;
    float z;
}t_point;
 
 
t_point *equation(t_point *a, t_point *b, t_point **tab, unsigned int length)
{
	t_point **r=new t_point*[5];for(int j=0;j<5;j++){ r[j]=new t_point[6];}
 
    if ( (a && b && tab) )
    {
        if (*r)
        {
            unsigned int cpt=0;
            float r_equation=0.;
            float lambda = -((a->x*b->x)+(a->y*b->y)+(a->z*b->z));
 
            unsigned int i;
            for(i=0;i<length;i++)
            { r_equation = ((a->x*tab[i]->x)+(a->y*tab[i]->y)+(a->z*tab[i]->z))+lambda;
                if( r_equation == 0.0 )
                { 
                    r[cpt]->x=tab[i]->x;
                    r[cpt]->y=tab[i]->y;
                    r[cpt]->z=tab[i]->z;
                     cpt++;
              }
            }
        } else printf("\n equation : Echec de l'allocation de r");
    } else printf("\n equation : Au moins un des arguments est NULL");
 
    return *r;
}
int main(int argc,char **argv){
 
//declaration
 
t_point  *a=(t_point *)malloc(3*sizeof(t_point));
t_point  *b=(t_point *)malloc(3*sizeof(t_point)); 
t_point **t=(t_point **)malloc(5*sizeof(t_point));;for(int j=0;j<5;j++){ t[j]=(t_point *)malloc(sizeof(t_point));}
t_point **r=(t_point **)malloc(3*sizeof(t_point));for(int j=0;j<5;j++){ r[j]=(t_point *)malloc(sizeof(t_point));}
 
//initialisation
 
a->x=-1;a->y=1;a->z=0;
b->x=1;b->y=-1;b->z=0;
t[0]->x=0;t[0]->y=-2;t[0]->z=0;
t[1]->x=2;t[1]->y=0;t[1]->z=0;
t[2]->x=1;t[2]->y=1;t[2]->z=0;
t[3]->x=0;t[3]->y=6;t[3]->z=0;
t[4]->x=5;t[4]->y=2;t[4]->z=0;
 
//calcule des valeurs vérifiant l'equation
*r=equation(a,b,t,5);
 
//affichage des résultats
int k=0;
printf("%f %f %f\n",r[k]->x,r[k]->y,r[k]->z); 
 
system("pause");
return 0;
}

pour le test j'ai choisi l'équation: -x+y+2=0. par calcul manuel t[0] et t[1] sont des solutions. mais lorsque j'essaye d'afficher les résultats je n'obtient que la première (pour k=0 seulement) ! ma question est comment afficher tous les résultats dans le cas géneral ?

**darkwall_37** · 01/03/2013, 09h08

Je ne suis pas sur de bien avoir compris ce que tu voulais.
En gros tu veux récupérer toutes les solutions trouvés par la fonction ?
Si oui, cpt compte le nombre de solutions. Ils suffit de la passer en paramètre pour récupérer sa valeur après exécution.
La fonction trouve 2 solutions (0,-2,0) et (0,0,0).
Le problème des floats c'est que ce n'est pas "précis" alors pour trouver le 0 "parfait", ca peut faire rater quelques résultats.
Le mieux serait de créer un type pour stocker les nombres sous forme de fraction et d'implémenter les opérations basiques comme +-*/.

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
#include <stdio.h>
#include <stdlib.h>
 
typedef struct
{
    float x;
    float y;
    float z;
}t_point;
 
 
t_point *equation(t_point *a, t_point *b, t_point **tab, unsigned int length, unsigned int *cpt)
{
	t_point **r=new t_point*[5];for(int j=0;j<5;j++){ r[j]=new t_point[6];}
 
    if ( (a && b && tab) )
    {
        if (*r)
        {
            float r_equation=0.;
            float lambda = -((a->x*b->x)+(a->y*b->y)+(a->z*b->z));
            *cpt=0;
            unsigned int i;
            for(i=0;i<length;i++)
            { r_equation = ((a->x*tab[i]->x)+(a->y*tab[i]->y)+(a->z*tab[i]->z))+lambda;
                if( r_equation == 0.0 )
                {
                    r[*cpt]->x=tab[i]->x;
                    r[*cpt]->y=tab[i]->y;
                    r[*cpt]->z=tab[i]->z;
                     (*cpt)++;
              }
            }
        } else printf("\n equation : Echec de l'allocation de r");
    } else printf("\n equation : Au moins un des arguments est NULL");
 
    return *r;
}
int main(int argc,char **argv){
 
//declaration
 
t_point  *a=(t_point *)malloc(3*sizeof(t_point));
t_point  *b=(t_point *)malloc(3*sizeof(t_point));
t_point **t=(t_point **)malloc(5*sizeof(t_point));;for(int j=0;j<5;j++){ t[j]=(t_point *)malloc(sizeof(t_point));}
t_point **r=(t_point **)malloc(3*sizeof(t_point));for(int j=0;j<5;j++){ r[j]=(t_point *)malloc(sizeof(t_point));}
 
//initialisation
 
a->x=-1;a->y=1;a->z=0;
b->x=1;b->y=-1;b->z=0;
t[0]->x=0;t[0]->y=-2;t[0]->z=0;
t[1]->x=2;t[1]->y=0;t[1]->z=0;
t[2]->x=1;t[2]->y=1;t[2]->z=0;
t[3]->x=0;t[3]->y=6;t[3]->z=0;
t[4]->x=5;t[4]->y=2;t[4]->z=0;
 
//calcule des valeurs vérifiant l'equation
unsigned int *cpt = (unsigned int*)malloc(sizeof(unsigned int));
*r=equation(a,b,t,5, cpt);
 
//affichage des résultats
int k;
for (k=0;k<*cpt;k++)
    printf("%f %f %f\n",r[k]->x,r[k]->y,r[k]->z);
 
system("pause");
return 0;
}

**darkwall_37** · 01/03/2013, 10h13

J'ai repris un peu encore. Ce que tu alloues, tu dois le désallouer.
Aussi je vois que tu as utiliser un double pointeur. Peut être a cause de moi car j'avais mis des '->' au lieu de '.'. Une seule dimension suffit pour ce que tu souhaite faire.

Voilà ce que m'affiche le code.

Code :

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

1
2
3
4
5
6
7
test 0 => r_equation = 0.000000
test 1 => r_equation = 0.000000
test 2 => r_equation = 2.000000
test 3 => r_equation = 8.000000
test 4 => r_equation = -1.000000
Solution 1 : (0.000,-2.000,0.000)
Solution 2 : (2.000,0.000,0.000)

test 0 et test 1 sont des solutions de l'équation et les résultats correspondants s'affichent bien une fois sortie de la fonction.
Donc test 2..4 ne sont pas des solutions.

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <errno.h>
 
typedef struct
{
    float x;
    float y;
    float z;
}t_point;
 
void deallocate_1D_ui( unsigned int *p )
{
    if (p)
        free(p), p=NULL;
 
    else printf("deallocate_1D_ui -> %s\n",strerror(errno));
}
 
unsigned int *allocate_1D_ui( unsigned long long int n )
{
    unsigned long long int i;
    unsigned int *p = NULL;
    if ((p = (unsigned int *)malloc(n*sizeof(unsigned int))))
        for ( i=0 ; i<n ; i++ )
            p[i]=0;
 
    else printf("allocate_1D_ui -> %s\n", strerror(errno));
 
    return p;
}
 
t_point *point_create_struct(unsigned int x)
{
    t_point *p = NULL;
    if ( !(p=(t_point*)malloc(x*sizeof(t_point))) )
        printf("\n point_allocate_struct : echec de l'allocation");
 
    return p;
}
 
void point_destroy_struct(t_point *p)
{
    if(p)
        free(p), p=NULL;
 
    else printf("\n point_destroy_struct : p is already NULL");
}
 
t_point *equation(t_point *a, t_point *b, t_point *tab, unsigned int length, unsigned int *index)
{
	t_point *r=point_create_struct(5);
 
    if ( (a && b && tab && index) )
    {
        if (r)
        {
            float r_equation=0.;
            float lambda = -((a->x*b->x)+(a->y*b->y)+(a->z*b->z));
 
            unsigned int i;
            for(i=0;i<length;i++)
            {
                r_equation = ((a->x*tab[i].x)+(a->y*tab[i].y)+(a->z*tab[i].z))+lambda;
                printf("test %d => r_equation = %f \n", i, r_equation);
                if( r_equation == 0. )
                {
                    r[*index].x=tab[i].x;
                    r[*index].y=tab[i].y;
                    r[*index].z=tab[i].z;
                     (*index)++;
                }
            }
        } else printf("\n equation : Echec de l'allocation de r");
    } else printf("\n equation : Au moins un des arguments est NULL");
 
    return r;
}
int main(int argc,char **argv)
{
    //declaration
    t_point *a=point_create_struct(3);
    t_point *b=point_create_struct(3);
    t_point *t=point_create_struct(5);
    t_point *r=point_create_struct(5);
    unsigned int *index = allocate_1D_ui(1);
 
    //initialisation
    a->x=-1.;a->y=1.;a->z=0.;
    b->x=1.;b->y=-1.;b->z=0.;
    t[0].x=0.;t[0].y=-2.;t[0].z=0.;
    t[1].x=2.;t[1].y=0.;t[1].z=0.;
    t[2].x=1.;t[2].y=1.;t[2].z=0.;
    t[3].x=0.;t[3].y=6.;t[3].z=0.;
    t[4].x=5.;t[4].y=2.;t[4].z=0.;
 
    //calcule des valeurs vérifiant l'equation
    r=equation(a,b,t,5,index);
 
    //affichage des résultats
    unsigned int k;
    for (k=0;k<*index;k++)
        printf("Solution %d : (%3.3f,%3.3f,%3.3f)\n", k+1, r[k].x, r[k].y, r[k].z);
 
    point_destroy_struct(a);
    point_destroy_struct(b);
    point_destroy_struct(t);
    point_destroy_struct(r);
    deallocate_1D_ui(index);
 
    system("pause");
    return 0;
}

**M77ATTAR** · 01/03/2013, 18h05

oui c'est résolu; merci bien pour l'aide !

problème avec une fonction

C

Vue hybride

Discussions similaires

Partager

Partager