Discussion :

[k.cheker]

salut tout le monde
s'il vous plait j'ai besoin d'aide a propos des fonction en c
je voudrais faire entrer un tableaus structuré comme parametre dans la fonction mais ca ne marche pas voila la structure

Code :

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

1
2
3
4
5
6
7
struct geo 
{ char ville[15]; 
char pays[15]; 
char continent[15]; 
long nhab; 
}; 
struct geo t[200] ;

et la fonction

Code :

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

1
2
3
4
5
6
void afficher_villes(geo t) 
{int i; 
for(i=0;i<nb_v;i++) 
if((strcmp(t[i].pays,"maroc"))&& (t[i].nhab>100000)) 
printf("- %s",t[i].ville); 
}

merci d'avance pour votre aide.
k.cheker@hotmail.fr

[Franck.H]

Salut et bienvenue sur les forums de developpez.com


En fait, ce que tu as fait là, c'est de mettre une simple variable du type de ta structure en argument à la fonction mais pour un tableau, il faut que ton argument soit au moins un pointeur (plusieures pratique possible mais je préfère celle là) donc, voici ta fonction:

Code :

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

void afficher_villes(geo * t, size_t size)

Je t'ai mis un second argument car les tableaux en C ne connaissent pas leur propre taille, il faut donc donner l'info à la fonction qui va traiter ton tableau !

Ici je vois que tu utilises une variable qui n'est même pas déclarée dans ta fonction, je redoute l'utilisation d'une variable globale ... à surtout éviter ce genre de pratique !

Donc pour finir, voici une implémentation possible pour ta fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void afficher_villes (geo * t, size_t size)
{
   size_t i = 0;
 
   for (i = 0; i < size; i++)
   {
      if ((strcmp (t[i].pays, "maroc")) && (t[i].nhab >100000))
      {
         printf ("- %s", t[i].ville);
      }
   }
}

PS: pas testé !

[lun4t1k]

t[i]->pays plutot nan?

[Franck.H]

t[i]->pays plutot nan?

Nan ! L'argument est certes un pointeur mais ce qu'on utilise est un tableau avant tout, tu peux toujours essayer mais en réglant ton compilateur sur au moins les options:

Code :

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

-W -Wall -pedantic

tu verras bien le message qu'il va te donner:

/home/franhec/Documents/Programmation/CodesTests/ANSI/App/main.c:14: error: invalid type argument of ‘->’

[k.cheker]

Salut et bienvenue sur les forums de developpez.com


En fait, ce que tu as fait là, c'est de mettre une simple variable du type de ta structure en argument à la fonction mais pour un tableau, il faut que ton argument soit au moins un pointeur (plusieures pratique possible mais je préfère celle là) donc, voici ta fonction:

Code :

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

void afficher_villes(geo * t, size_t size)

Je t'ai mis un second argument car les tableaux en C ne connaissent pas leur propre taille, il faut donc donner l'info à la fonction qui va traiter ton tableau !

Ici je vois que tu utilises une variable qui n'est même pas déclarée dans ta fonction, je redoute l'utilisation d'une variable globale ... à surtout éviter ce genre de pratique !

Donc pour finir, voici une implémentation possible pour ta fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void afficher_villes (geo * t, size_t size)
{
   size_t i = 0;
 
   for (i = 0; i < size; i++)
   {
      if ((strcmp (t[i].pays, "maroc")) && (t[i].nhab >100000))
      {
         printf ("- %s", t[i].ville);
      }
   }
}

PS: pas testé !

salut j'ai remplacer le tableau comme pointeur mais lors de la compilation
une erreur <improper use of typedef 'geo'>
merci co meme

[Franck.H]

salut j'ai remplacer le tableau comme pointeur mais lors de la compilation
une erreur <improper use of typedef 'geo'>
merci co meme

Ta structure, tu en as fait un typedef ou pas ?

Si oui alors:

Code :

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

void afficher_villes(geo * t, size_t size)

Si non:

Code :

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

void afficher_villes(struct geo * t, size_t size)

Le mieux serait encore que tu montre le code complet de ton programme !

[k.cheker]

Ta structure, tu en as fait un typedef ou pas ?

Si oui alors:

Code :

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

void afficher_villes(geo * t, size_t size)

Si non:

Code :

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

void afficher_villes(struct geo * t, size_t size)

Le mieux serait encore que tu montre le code complet de ton programme !

voila le code complet du programme
mais quel est l'utilite de size_t car le tableau est deja remplie

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
#include<stdio.h>
#include<conio.h>
#include<string.h>
#include<stdlib.h>
struct geo
{    char ville[15];
     char pays[15];
     char continent[15];
     long nhab;
};
struct geo t[200] ;
 
int creer(int x);
void afficher_villes(struct geo *t);
 
 
int nb_v;
 
 
void main()
{
printf("\n\nle nombre total des villes saisies est:   %d",creer(nb_v));
printf("\n\nliste des villes de maroc ayant un nombtre d habitant >100000 :\n");
afficher_villes(geo *t);
 
getch();
}
 
 
 
int creer(int x)
{  int test,i;
   test=1;
   i=x=0;
   while(test==1)
   {printf("\nsaisir la ville :  ");
   scanf("%s",t[i].ville);
   printf("\nsaisir le pays :  ");
   scanf("%s",t[i].pays);
   printf("\nsaisir le continent :  ");
   scanf("%s",t[i].continent);
   printf("\nsaisir le nombre d habitant :  ");
   scanf("%l",t[i].nhab);
   x++;
   printf("\npour saisir une autre ville tapez '1':\n");
   scanf("  %d",&test);
   i++;
   };
   return x;
}
 
void afficher_villes(struct geo *t)
{int i;
  for(i=0;i<nb_v;i++)
    if((strcmp(t[i].pays,"maroc"))&& (t[i].nhab>100000))
    printf("- %s",t[i].ville);
}

k.cheker@hotmail.fr

[Franck.H]

mais quel est l'utilite de size_t car le tableau est deja remplie

le type size_t est ceci:

Code :

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

typedef unsigned long size_t;

définit dans stddef.h et autres headers également. C'est un type utilisé surtout pour les données relatives à une taille comme les indices des tableaux, les tailles transmises pour les allocations dynamique, etc... Ce n'est pas obligatoire de l'utiliser mais je préfère, question d'habitude !


Pour ton code je vais voir ca...