Problème de boucle [Débutant(e)]

**jacko842** · 04/06/2007, 07h19

Bonjour j ai un probleme dans ma boucle et je ne trouve pas la solution donc voila je dois rentré 5 noms et 5 cotes example :
jean
8
pierre
5
jin
8
jack
7
sylv
10
mais au lieu de ca j ai trop de noms a rentré soit que j ai mal paramétré mes boucles ou alors j avais pensé au buffer du clavier
voici le code

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
#include<conio.h>
main()
	{
	 char eleve[5][15];
	 int cote[5];
	 int i,j;
	 clrscr();
	 printf("Entrez 5 noms d'‚lŠve et leurs c“te .\n");
	 for(i=0;i<4;i++)
		{
		gets(eleve[i]);
			for(j=0;j<=4;j++)
			scanf("%d",&cote[j]);
 
 
 
		}
 
	 getch();
 
 
	}

merci d avance

**lechewal** · 04/06/2007, 10h34

mh, ce magnifique gets !

**Skyounet** · 04/06/2007, 10h58

Alors tellement de choses.

Présentation horrible.
Utilisation de gets qui vaut a elle seule une flagélation en place publique.

Pour ton problème je te conseille d'utiliser une structure

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
typedef struct 
{
   char nom[256];
   int cote;
} Eleve;
 
int main(void)
{
   Eleve tab[5];
   int i = 0;
 
   for(i = 0; i < sizeof tab; i++)
   {
      fgets(tab[i].nom, sizeof tab[i].nom, stdin);
      scanf("%d", &tab[i].cote);
   }
 
   return 0;
}

Voilà comment tu pourrais faire (et encore on pourrait rajouter un contrôle sur fgets et un contrôle sur scanf voire même ne pas l'utiliser et utiliser une fonction de saisie maison).

**Emmanuel Delahaye** · 04/06/2007, 11h55

Envoyé par Skyrunner

Utilisation de gets qui vaut a elle seule une flagélation en place publique.

**stranger** · 04/06/2007, 13h24

Lol emmanuel,
vous êtes si dur avec les débutant(e)s

**Médinoc** · 04/06/2007, 13h46

Ou avec leurs profs.
Aucun prof ne devrait enseigner gets(), tout comme aucun prof ne devrait enseigner scanf() à des débutants (ce n'est PAS une fonction pour débutants)...

**muad'dib** · 04/06/2007, 13h57

Envoyé par Médinoc

Aucun prof ne devrait enseigner gets(), tout comme aucun prof ne devrait enseigner scanf() à des débutants (ce n'est PAS une fonction pour débutants)...

Tous les profs enseignent gets() et scanf(). Je pense qu'ils sont au courant (du moins la plupart) que ce sont de mauvaises fonctions, mais pédagogiquement c'est plus simple de faire un simple gets() qu'un fgets qui inclut la notion de clavier en tant que fichier, ce qui peut amener l'étudiant débutant à confusion.

Ils pensent probablement que c'est l'expérience qui amènera les étudiants à remplacer gets par fgets (l'expérience ou les forums de developpez)

**Skyounet** · 04/06/2007, 14h39

Ben moi à l'IUT on nous a enseigné scanf pour tout ce qui était saisie d'entier et même pour les chaines de caractères (en nous précisant que ça s'arretait au premier espace).

Puis quand on a voulu saisir des phrases par exemple on nous a de suite appris fgets. Mais jamais gets on nous a prévenu qu'il ne fallait jamais l'utiliser.

Pis apprendre fgets sans connaitre la gestion des flux... c'est comme quand on nous apprend FILE * tout le monde demande c'est quoi l'étoile et on te répond c'est des pointeurs on verra plus tard.

**souviron34** · 04/06/2007, 15h53

Envoyé par muad'dib

mais pédagogiquement c'est plus simple de faire un simple gets() qu'un fgets qui inclut la notion de clavier en tant que fichier, ce qui peut amener l'étudiant débutant à confusion.

Mais c'est au contraire une bonne notion à enseigner, car ça unifie ...

C'est le principe même du succès de la notion "stdin/stdout/stderr/fichiers", appliquée dans je ne sais combien de langages : IL N'Y A PAS DES CENTAINES DE TYPE DE CHOSES...

TOUTE interface "système" est un fichier....

(voir les /dev , les ports, etc sur les systèmes *n*x également, les sockets, ...).

Donc penser que différencier le clavier d'un fichier est plus simple est simplement une aberration....

**Médinoc** · 04/06/2007, 16h01

Non, tout n'est pas fichier sous les systèmes unixoïdes. Notamment les objets de synchronisation, qui à l'inverse des fichiers, sont incompatibles avec select() (Windows domine complètement les systèmes unixoïdes sur ce plan-là).
C'est plus simple quand on n'a pas encore parlé des fichiers (même si ça n'excuse pas l'emploi de gets(), et je pense qu'il est préférable de mentionner stdin en tant que "on en parlera plus tard quand on fera les fichiers").

**souviron34** · 04/06/2007, 16h09

Envoyé par Médinoc

Non, tout n'est pas fichier sous les systèmes unixoïdes. Notamment les objets de synchronisation, qui à l'inverse des fichiers, sont incompatibles avec select() (Windows domine complètement les systèmes unixoïdes sur ce plan-là).
C'est plus simple quand on n'a pas encore parlé des fichiers (même si ça n'excuse pas l'emploi de gets(), et je pense qu'il est préférable de mentionner stdin en tant que "on en parlera plus tard quand on fera les fichiers").

Sur le premier point, vaguement OK... Mais que vaguement, car, même si il y a quelques exceptions, l'écrasante majorité des interfaces matérielles sont des fichiers. Et select prend entre autres un socket, qui est un fichier de type particulier.... Et pour ce qui est de la "domination" de windows.. je ne ferais pas de troll ici

Mais pour le point 2, pas du tout.. La notion de fichier DEVRAIT être enseignée dès le départ, puisque ces chères petites têtes blondes créent bien des fichiers, pour faire leurs programmes, non ???

**Médinoc** · 04/06/2007, 16h11

Tu confonds fichier sur le disque et "fichier ouvert" par un programme C (je ne parle pas de descripteur, car le C standard ne sait pas ce que c'est).

Edit: Puisqu'on crie au troll, j'éviterai de m'étendre là-dessus. Reste que l'affirmation "tout est fichier" est fausse.
De plus, ce n'est pas le même "niveau" de fichier (FILE* contre descripteur).

**Emmanuel Delahaye** · 04/06/2007, 16h13

Envoyé par Médinoc

Non, tout n'est pas fichier sous les systèmes unixoïdes. Notamment les objets de synchronisation, qui à l'inverse des fichiers, sont incompatibles avec select() (Windows domine complètement les systèmes unixoïdes sur ce plan-là).

* Troll Alert *

**Emmanuel Delahaye** · 04/06/2007, 16h16

Envoyé par souviron34

Mais pour le point 2, pas du tout.. La notion de fichier DEVRAIT être enseignée dès le départ, puisque ces chères petites têtes blondes créent bien des fichiers, pour faire leurs programmes, non ???

Personnellement, j'ai fait le choix d'enseigner les flux juste après les pointeurs. A partir de ce moment là, on peut faire des entrées sorties (stdin/stdout, fichiers, peu importe. Je suis d'accord avec le notion "d'unification").

**jacko842** · 04/06/2007, 17h21

merci pour vos explications mais je suis newbee dans le c c est pour ca que je m en remets a vous merci

**jacko842** · 04/06/2007, 17h26

j aurais quand meme bien voulu qu on reponde a ma question pcq je prefere faire quelle que chose de pas tres bon mais que je comprenne pq c est pas bon que quelle que chose de tres bon et que je ne comprend pas le deroulement de mon programme

**Médinoc** · 04/06/2007, 17h29

Réfère-toi au message de Skyrunner et dis-nous ce que tu ne comprends pas.

**Emmanuel Delahaye** · 04/06/2007, 17h31

Envoyé par jacko842

Bonjour j ai un probleme dans ma boucle et je ne trouve pas la solution donc voila je dois rentré 5 noms et 5 cotes example :
jean
8
pierre
5
jin
8
jack
7
sylv
10
mais au lieu de ca j ai trop de noms a rentré soit que j ai mal paramétré mes boucles ou alors j avais pensé au buffer du clavier
voici le code
<...>

Ton code est horriblement mal présenté.

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
 
#include<conio.h>
main ()
{
   char eleve[5][15];
   int cote[5];
   int i, j;
   clrscr ();
   printf ("Entrez 5 noms d'eleves et leurs cote .\n");
   for (i = 0; i < 4; i++)
   {
      gets (eleve[i]);
      for (j = 0; j <= 4; j++)
         scanf ("%d", &cote[j]);
   }
 
   getch ();
}

De plus, il est incomplet :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:3: warning: return type defaults to `int'
main.c:3: warning: function declaration isn't a prototype
main.c: In function `main':
main.c:7: warning: implicit declaration of function `clrscr'
main.c:8: warning: implicit declaration of function `printf'
main.c:11: warning: implicit declaration of function `gets'
main.c:13: warning: implicit declaration of function `scanf'
main.c:22: warning: control reaches end of non-void function
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 2 seconds)
0 errors, 7 warnings

Voici une version corrigée et commentée avec une manière correcte de faire les entrées en C. Pose des questions si tu ne comprends pas :

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
 
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
 
/* nettoyage de la chaine saisie par fgets(). */
static void clean(char *s)
{
   /* search ... */
   char *p = strchr(s, '\n');
 
   if (p != NULL)
   {
      /* ... and kill */
      *p = 0;
   }
   else
   {
      /* purge */
      int c;
      while ((c=getchar()) != '\n' && c!= EOF)
      {
      }
   }
}
 
int main (void)
{
   /* tu n'as pas appris les structures ? */
   char eleve[5][15];
   int cote[5];
 
   int i;
 
   printf ("Entrez 5 noms d'eleves et leurs cote .\n");
   for (i = 0; i < 5; i++)
   {
      /* saisie d'une chaine de caracteres. */
      fgets (eleve[i], sizeof eleve[i], stdin);
      clean(eleve[i]);
 
      /* saisie d'un nombre entier */
      {
         char s[8];
         fgets(s, sizeof s, stdin);
         clean (s);
         cote[i] = (int) strtol(s, NULL, 10);
      }
   }
 
   /* car main() retourne un int. */
   return 0;
}

**jacko842** · 04/06/2007, 17h48

merci pour la version commenté mais quand j execute le code source avec turbo c ca fonctionne ??pq chez toi non (pas les meme dependances dans ton editeurs??)

ps:non je n aie pas appris les structures

**odsen.s** · 04/06/2007, 17h57

Juste une petite question au passage :

Code :

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

1
2
3
4
5
6
7
/* saisie d'un nombre entier */
{
   char s[8];
   fgets(s, sizeof s, stdin);
   clean (s);
   cote[i] = (int) strtol(s, NULL, 10);
}

Ici, si l'on fait un "bloc" (accolades), c'est pour que le tableau de char soit supprimé à la fin ?
Sinon, quelle est la raison ?
Merci.

Problème de boucle [Débutant(e)]

Réseau C

Discussions similaires

Partager

Partager