Mon code est correct , mais j'aurais peut être d'autres questions

Version imprimable

19/09/2007, 14h20
incal

Mon code est correct , mais j'aurais peut être d'autres questions

mon problème est résolut

Bonjour,

Code:

1
2
3
char a : représente une seule variable
char a[10] : représente un tableau de 10 caractères
a[0] : premier caractère et a[9] est le dernier.

Bon courage.

Vos

Code:

scanf ("%c", &poubelle);

destinés à récupérer les retours chariots qui traînent après la lecture d'un entier ne sont pas au bon endroit, il faut les placer après les

Code:

scanf ("%d",...);

Ainsi:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 ... printf ("Entrez le nom de l'organisme : "); scanf ("%c", &a); printf ("Entrez sa taille (en Kbp) : "); scanf ("%d", &i); scanf ("%c", &poubelle); printf ("Entrez le nom de l'organisme : "); scanf ("%c", &b); printf ("Entrez sa taille (en Kbp) : "); scanf ("%d", &j); scanf ("%c", &poubelle); printf ("Entrez le nom de l'organisme : "); scanf ("%c", &c); printf ("Entrez sa taille (en Kbp) : "); scanf ("%d", &k); ...

19/09/2007, 14h31
aoyou

Mauvaise utilisation de scanf.
Le sujet est maintes et maintes fois traité dans ce forum.

Tu trouveras des explications dans les topics suivants.
http://www.developpez.net/forums/sho...ighlight=scanf
http://www.developpez.net/forums/sho...ighlight=scanf

Si tu veux entrer des noms, effectivement, un char ne sera pas suffisant à moins que le nom des organismes ne comporte qu'une seule lettre.
19/09/2007, 14h40
stephl

Ensuite, si le nom peut contenir plus d'une lettre, il faut utiliser des tableaux de char et une fonction du type fgets() ou scanf().
19/09/2007, 15h25
incal

quand j'utilise fgets j'ai ce message

passing argument 2 of "fgets" makes integer from pointer without a cast

J'ai vu que d'autre ont eu ce problème , mais je ne sais pas comment résoudre ça ... il me faudrait une explication détaillée , je pense

merci
19/09/2007, 15h29
stephl
Citation:

Envoyé par K&R2

char *fgets(char *s, int n, FILE *stream)
fgets reads at most the next n-1 characters into the array s, stopping if a newline is encountered; the newline is included in the array, which is terminated by '\0'. fgets returns s, or NULL if end of file or error occurs.

Exemple:
Code:

1 2 3 4 char s[10]; ... fgets(s,sizeof s,stdin); ...
19/09/2007, 15h33
incal

je connait bien fgets
mais je n'ai encore jamais réussis à l'utiliser
j'ai toujours

passing argument 2 of "fgets" makes integer from pointer without a cast
19/09/2007, 15h36
stephl

Postez votre code s'il vous plaît.
19/09/2007, 15h36
BainE

Bonjour,

fait nous voir comment tu essaye de l'appeler on aura peut etre la reponse :D

ben voilà où j'en suis pour le moment

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
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
 
#include <stdio.h>
#include <stdlib.h>
 
int main (void)
 
{
 
  char poubelle;
 
  char a[10];
  char b[10];
  char c[10];
  int i;
  int j;
  int k;
  i = 0;
  j = 0;
  k = 0;
  printf ("Entrez le nom de l'organisme : ");
  fgets (&a[0], 9, stdin);  
  printf ("Entrez sa taille (en Kbp) : ");
  scanf ("%d", &i);
  scanf ("%s", &poubelle);
  printf ("Entrez le nom de l'organisme : ");
  fgets (&a[0], 9, stdin); 
  printf ("Entrez sa taille (en Kbp) : ");
  scanf ("%d", &j);
  scanf ("%s", &poubelle);
  printf ("Entrez le nom de l'organisme : ");
  fgets (&a[0], 9, stdin);
  printf ("Entrez sa taille (en Kbp) : ");
  scanf ("%d", &k);
  scanf ("%s", &poubelle);
 
  if (i<j && j<k) 
  {              
    printf ("\nGenome max : %c\n", c[0]);
    printf ("Genome min : %c\n", a[0]);
    printf ("Genome med : %c\n", b[0]);
  }
  else if (i<j && k<j)
  {
    printf ("\nGenome max : %c\n", b[10]);
    printf ("Genome min : %c\n", a[10]);
    printf ("Genome med : %c\n", c[10]);
  }
  else if (j<k && k<i)
  {
    printf ("\nGenome max : %c\n", a[10]);
    printf ("Genome min : %c\n", b[10]);
    printf ("Genome med : %c\n", c[10]);
  }
  else if (j<k && i<k)
  {
    printf ("\nGenome max : %c\n", c[10]);
    printf ("Genome min : %c\n", b[10]);
    printf ("Genome med : %c\n", a[10]);
  }
  else if (k<j && j<i)
  {
    printf ("\nGenome max : %c\n", a[10]);
    printf ("Genome min : %c\n", c[10]);
    printf ("Genome med : %c\n", b[10]);
  }
  else if (k<j && i<j)
  {
    printf ("\nGenome max : %c\n", b[10]);
    printf ("Genome min : %c\n", c[10]);
    printf ("Genome med : %c\n", a[10]);
  }
 
 
  return 0;
}

comme codé sur les thread referencés par les liens cité plus haut

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
 
/* utilisation de fgets */
        char buffer[15] = "";
        if (fgets(buffer, sizeof buffer, stdin) != NULL)
        {
            fclean(buffer, stdin);
            n = strtol(buffer, &p_end, 0);
        }
 
 
/* Elimine le caractere de fin de ligne terminant la saisie et vide le tampon
   du flux d'entree stream si necessaire */
static void fclean(char *buffer, FILE *stream)
{
    char *pc = strchr(buffer, '\n');
 
    if (pc != NULL)
    { /* Le caractere de fin de ligne a ete trouve. La saisie est complete */
 
        *pc = 0;
    }
    else
    { /* La saisie a ete tronquee. On vide le tampon du flux stream */
 
        /* Important: la fonction fgetc(stream) retourne un entier de type int,
           pas de type char. */
        int c;
 
        while ((c = fgetc(stream)) != '\n' && c != EOF)
        {
            /* On ignore les caractere presents dans le tampon */
        }
    }
}

(code de tierry chappuis, pris sur un des thread susmentionné)
pour le reste visite les thread, tout est dis dedans.

Voici ton programme (un peu) remanié...

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
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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
/* -tc- determine le nombre d'elements que contient un tableau statique */
#define NB_ELEMS(a) ( sizeof (a) / sizeof *(a) )
 
/* -tc- Une structure Organisme pour faire les choses proprement */
struct Organisme
{
    char nom[10];
    int taille;
};
 
/* -tc- fonction essentielle pour bien utiliser fgets() */
static void fclean(char *buffer, FILE *stream);
/* -tc- predicat a utiliser avec qsort() */
static int comparer(void const *o1, void const *o2);
 
int main (void)
{
    /* -tc- tampon utilise lors de la saisie d'un entier */
    char tampon[15] = "";
    struct Organisme organismes[3] = {{"", 0}};
    size_t i;
 
    for (i = 0; i < NB_ELEMS(organismes); ++i)
    {
        char *pend = NULL;
 
        printf ("Entrez le nom de l'organisme : ");
        /* -tc- necessaire pour forcer l'affichage de l'invite */
        fflush(stdout);
        fgets(organismes[i].nom, sizeof organismes[i].nom, stdin);
        fclean(organismes[i].nom, stdin);
 
        /* -tc- Saisie securisee d'un entier */
        do
        {
            printf ("Entrez sa taille (en Kbp) : ");
            fflush(stdout);
            fgets(tampon, sizeof tampon, stdin);
            fclean(tampon, stdin);
            organismes[i].taille = strtol(tampon, &pend, 0);
        }
        while (*pend != 0);
    }
 
    /* -tc- On trie le tableau organismes */
    qsort(organismes, NB_ELEMS(organismes), sizeof *organismes, comparer);
 
    /*-tc- Le tableau etant trie, il est facile de determiner max, min, et med */
    printf("Genome max: %s\n", organismes[NB_ELEMS(organismes) - 1].nom);
    printf("Genome min: %s\n", organismes[0].nom);
    printf("Genome med: %s\n", organismes[NB_ELEMS(organismes) / 2].nom);
 
    return EXIT_SUCCESS;
}
 
/* -tc- Predicat permettant de comparer deux organismes */
static int comparer(void const *o1, void const *o2)
{
    struct Organisme const *org1 = (struct Organisme const *) o1;
    struct Organisme const *org2 = (struct Organisme const *) o2;
 
    return org1->taille - org2->taille;
}
 
/* -tc- Elimine le caractere de fin de ligne terminant la saisie et vide le tampon
   du flux d'entree stream si necessaire */
static void fclean(char *buffer, FILE *stream)
{
    char *pc = strchr(buffer, '\n');
 
    if (pc != NULL)
    { /* -tc- Le caractere de fin de ligne a ete trouve. La saisie est complete */
 
        *pc = 0;
    }
    else
    { /* -tc- La saisie a ete tronquee. On vide le tampon du flux stream */
 
        /* -tc- Important: la fonction fgetc(stream) retourne un entier de type int,
           pas de type char. */
        int c;
 
        while ((c = fgetc(stream)) != '\n' && c != EOF)
        {
            /* -tc- On ignore les caractere presents dans le tampon */
        }
    }
}

Thierry

19/09/2007, 17h30
incal

(en se prosternant) fantastique , thierry !

un peu (beaucoup) remanié tout de même :aie:
je vais faire de mon mieu pour tout comprendre :D
19/09/2007, 17h51
Thierry Chappuis

Citation:

Envoyé par incal

un peu (beaucoup) remanié tout de même :aie:
je vais faire de mon mieu pour tout comprendre :D

Pas tant remanié que ça, à part l'introduction d'une structure Organisme pour encapsuler le nom et la taille dans une seule entité. Sinon, les répétitions de code on été remplacées par des boucles (si tu n'as que 3 organismes, ce n'est pas un problème de tout taper à la main, mais si le nombre d'organismes augmente... Les boucles et les tableaux sont faits pour ça!).

Le tri d'un tableau est une excellente manière d'obtenir le max, min et med d'un ensemble de valeurs. La fonction qsort() (déclarée dans le fichier d'en-tête standard stdlib.h) rend cette tâche beaucoup moins fastidieuse que la technique "a la mano" que tu as utilisée.

Pour le reste, je n'utilise jamais scanf() (ou presque jamais). C'est une fonction plus complexe que ne le laissent penser beaucoup de prof. ou livres d'introduction. Pour en maîtriser toutes les finesses, je recommande la lecture suivante qui est très instructive: http://xrenault.developpez.com/tutoriels/c/scanf/. Toutefois, lorsque tu auras pris l'habitude du travail en équipe, tu te rendras vite compte que si tu as su prendre le temps de devenir un gourou dans l'utilisation de scanf(), il y a 9 chances sur 10 que les programmeurs qui seront chargés de la maintenance n'afficherons, pour leur part, pas une telle maîtrise. EN conclusion, utilise fgets() et fclean() (voir détails dans les archives du forum) autant que possible et, si besoin, convertir le texte saisi avec strtol(), strtoul(), strtod() ou même sscanf().

N'hésite pas à poser des questions si tu ne comprends pas.

Thierry
19/09/2007, 18h46
incal

Je pense en comprendre une grande partie .

Pour l'instant je me demande si le tampon utilisé lors de la saisie d'un entier doit nécéssairement être plus grand que le nombre entre crochets , au début dans "char nom[10]" ?
(il me semble que ça vaut mieu)

J'aurais sûrement encore besoin d'aide , j'ai bien du chemin à faire...

:merci: