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;
}

**darkwall_37** · 01/03/2013, 11h10

J'ai rajouté une fonction pour initialiser les valeurs des structures points

Résultat de l’exécution :

Code :

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

1
2
3
4
5
6
7
8
 
test 0 => r_equation = -1* 0+ 1*-2+ 0* 0+ 2 =  0
test 1 => r_equation = -1* 2+ 1* 0+ 0* 0+ 2 =  0
test 2 => r_equation = -1* 1+ 1* 1+ 0* 0+ 2 =  2
test 3 => r_equation = -1* 0+ 1*-6+ 0* 0+ 2 = 8
test 4 => r_equation = -1* 5+ 1* 2+ 0* 0+ 2 = -1
Solution 1 : (0.000,-2.000,0.000)
Solution 2 : (2.000,0.000,0.000)

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
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
#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 = %2.f*%2.f+%2.f*%2.f+%2.f*%2.f+%2.f = %2.f\n", i, a->x,tab[i].x,a->y,tab[i].y,a->z,tab[i].z,lambda, 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;
}
 
t_point *init_point_values(float a, float b, float c)
{
    t_point *p=NULL;
    if ( (p=point_create_struct(1)) )
        p->x=a, p->y=b, p->z=c;
 
    else printf("\n init_point_values : Echec de l'allocation");
    return p;
}
 
t_point init_point_tab( float a, float b, float c )
{
    t_point p;
    p.x = a;
    p.y = b;
    p.z = c;
    return p;
}
 
int main(int argc,char **argv)
{
    //declaration
    t_point *a=point_create_struct(1);
    t_point *b=point_create_struct(1);
    t_point *t=point_create_struct(5);
    t_point *r=point_create_struct(5);
    unsigned int *index = allocate_1D_ui(1);
 
    //initialisation
       a = init_point_values(-1.,  1., 0.);
       b = init_point_values( 1., -1., 0.);
    t[0] = init_point_tab( 0., -2., 0.);
    t[1] = init_point_tab( 2.,  0., 0.);
    t[2] = init_point_tab( 1.,  1., 0.);
    t[3] = init_point_tab( 0.,  6., 0.);
    t[4] = init_point_tab( 5.,  2., 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

Discussions similaires

Partager

Partager