Caractère répété deux fois

Version imprimable

Bonjour :)

J'ai un problème assez bizarre, je pense savoir d'où ça vient, mais pas comment y remédier.
Je débute en C, j'ai crée un programme en PHP que je voudrais convertir en C sans interface, juste en command-line, j'ai donc crée ce script pour la base:
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 int main() { printf("Usage:\n"); printf("c : Create a new file.\n"); printf("q : Quit.\n"); char command; while(1) { printf(">"); command=getchar(); if(command=='c') { printf("Type file name:\n"); char filename[80]; scanf("%s",filename); char buf[]="test\n"; int fd; fd = open(filename, O_CREAT | O_RDWR, 0644); write(fd, buf, sizeof buf); } if(command=='b') { printf("blablabla\n"); } if(command=='q') { printf("Quit\n"); break; } } return 0; }
En gros, l'utilisateur tape un lettre et une commande s'exécute, ça marche, mais il y a un hic: à chaque lettre tapée, le caractère ">" est répété deux fois:
http://img166.imageshack.us/img166/1899/image3es1.jpg
Je pense savoir d'où ça vient:
quand je tape disons 2 lettres à la suite j'obtiens ça:
>>>
et si j'appuie juste sur entrer ça donne:
> (ce qu'il faut)
J'ai l'impression que quand je tape un caractère, le caractère de retour à la ligne est aussi pris en compte, et à cause de la boucle while, tous les caractères sont traités un par un à chaque itération de la boucle, le retour à la ligne compris.

Seulement voilà, je n'ai pas la moindre idée de comment résoudre ça :cry:

Si quelqu'un pouvait m'aider... merci d'avance :)

29/08/2006, 20h37
DaZumba

Tu peux consulter les très nombreux nombreux posts de ce forum disant pourquoi il ne faut pas utiliser scanf() lorsqu'on débute.

EDIT: bon, le scanf() n'est pas le probleme en fait (enfin, pas encore...). Comme tu l'indiques, il faut ajouter un test pour 'entrée'. C'est le caractere '\n'.
Egalement, declare command comme un int pour se conformer au retour de getchar().

Puis tu dois aussi purger stdin, comme suit et ca marchera:

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
 
static void PurgeFile (char *s, FILE *fp)
{
   /* search ... */
   char *p = strchr (s, '\n');
   if (p != NULL)
   {
      /* ... and kill */
      *p = 0;
   }
   else
   {
      /* purge */
      int c;
      while ((c = fgetc(fp)) != '\n' && c != EOF)
      {
      }
   }
}
 
int main(int argc, char *argv[])
{
  char command;
  int fd;
  char filename[80];
  char buf[]="test\n";
 
  printf("Usage:\n");
  printf("c : Create a new file.\n");
  printf("q : Quit.\n");
 
  while(1)
  {
   printf(">");
   command=getchar();
 
   if(command=='c')
   {
    printf("Type file name:\n");
	scanf("%s",filename);
 
	fd =  open(filename, O_CREAT | O_RDWR, 0644);
 
    write(fd, buf, sizeof buf);
	}
 
	if(command=='b')
	{
	 printf("blablabla\n");
	}
 
	if(command=='q')
	{
	 printf("Quit\n");
	 break;
	}
	PurgeFile(buf, stdin);
  }
  system("PAUSE");
  return EXIT_SUCCESS;
}

Citation:

Tu peux consulter les très nombreux nombreux posts de ce forum disant pourquoi il ne faut pas utiliser scanf() lorsqu'on débute.

Par contre, il est vrai qu'il te faudra remplacer scanf par fgets.

Puis les variables, ca serait bien que tu les déclares toutes en début de fonction.

29/08/2006, 20h56
homeostasie
Code:

1 2 3 fd = open(filename, O_CREAT | O_RDWR, 0644); write(fd, buf, sizeof buf);
Utiliser fopen et fwrite pourrait être mieux.

Bonne soirée

Citation:

Envoyé par Horrigan

En gros, l'utilisateur tape un lettre et une commande s'exécute, ça marche, mais il y a un hic: à chaque lettre tapée, le caractère ">" est répété deux fois:
http://img166.imageshack.us/img166/1899/image3es1.jpg
Je pense savoir d'où ça vient:
quand je tape disons 2 lettres à la suite j'obtiens ça:
>>>
et si j'appuie juste sur entrer ça donne:
> (ce qu'il faut)
J'ai l'impression que quand je tape un caractère, le caractère de retour à la ligne est aussi pris en compte, et à cause de la boucle while, tous les caractères sont traités un par un à chaque itération de la boucle, le retour à la ligne compris.

Ce code est incomplet. Il manque les headers. Le comportement est indéfini.
open() n'est pas standard. C'est fopen() etc.
apprendre à utiliser switch-case. Ces 'if' sont horribles !
en principe, on ne met pas de 0 dans un fichier texte. Utiliser strlen().
le fichier n'est jamais fermé.

Les saisies sont erronées. Voici comment on peut faire proprement :

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

(Ton code mis au point. pose des questions si tu ne comprends pas. )
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 #include <stdio.h> #include <string.h> static void get_s(char *s, size_t size) { fgets(s, size, stdin); { char *p = strchr(s, '\n'); if (p != NULL) { *p = 0; } else { int c; while ((c = getchar()) != '\n' && c != EOF) { } } } } static int get_c(void) { int x = EOF; char s[8]; get_s(s, sizeof s); x = *s; return x; } int main() { printf("Usage:\n"); printf("c : Create a new file.\n"); printf("q : Quit.\n"); char command; while (1) { printf(">"); command = get_c(); if (command == 'c') { printf("Type file name:\n"); char filename[80]; /* edit (C90) */ { char buf[] = "test\n"; FILE *fp = fopen(filename, "w"); if (fp != NULL) { fwrite(buf, 1, strlen (buf), fp); fclose (fp), fp = NULL; } } } if (command == 'b') { printf("blablabla\n"); } if (command == 'q') { printf("Quit\n"); break; } } return 0; }

30/08/2006, 01h49
Horrigan

Whaouw, c'est vraiment sympa de carrément corriger tout le code, mais j'en demandais pas tant :oops:

Citation:

Ce code est incomplet. Il manque les headers. Le comportement est indéfini.
open() n'est pas standard. C'est fopen() etc.
apprendre à utiliser switch-case. Ces 'if' sont horribles !
en principe, on ne met pas de 0 dans un fichier texte. Utiliser strlen().
le fichier n'est jamais fermé.

-Le header c'est juste #include <Carbon/Carbon.h>, je sais c'est bizarre, c'est Xcode, je développe sous mac, les stdio.h je connais pas ^^

-Pour les fichiers j'ai utilisé un script que j'ai trouvé sur internet, j'ai un livre sur le C (de Tony Zhang), mais je suis pas au point là dessus, je vais réparer ça :p

-Pour le switch, je l'avais utilisé pour mon premier essai, mais je l'ai remplacé par des if et j'ai même essayé les goto pour voir si ça résolvait le problème des >>, j'ai juste oublié de le remettre ^^

J'ai essayé ton code, il marche très bien :) je le comprends à peu près (vu de loin), mais jamais j'aurais réussi à faire ça moi même, il faudrait que je reprenne mon livre depuis le début...

Je comprends bien à quoi servent les fonctions get_s et get_c mais pas exactement comment elles marchent, c'est surtout les pointeurs qui me troublent.
Sinon pour l'écriture du fichier et le reste j'ai compris ^^

Vraiment merci beaucoup pour cette aide :)
30/08/2006, 09h45
Emmanuel Delahaye

Citation:

Envoyé par Horrigan

Je comprends bien à quoi servent les fonctions get_s et get_c mais pas exactement comment elles marchent, c'est surtout les pointeurs qui me troublent.

Tu ne peux pas faire de C si tu ne maîtrises pas les pointeurs. Fait des petits programmes d'essais, passe du temps là dessus, pose des questions précises.

Pour commencer :

http://emmanuel-delahaye.developpez.....htm#pointeurs
30/08/2006, 14h27
Horrigan

Sisi, j'ai bien compris ce qu'étaient les pointeurs et comment on s'en servait, mais ce que j'ai pas compris, c'est leur utilité, mon livre n'en fait pas mention ^^
Accéder à une variable par un pointeur j'en vois pas l'utilité, j'utilise directement le nom de la variable, c'est plus pratique ^^
30/08/2006, 16h40
Emmanuel Delahaye

Citation:

Envoyé par Horrigan

mais ce que j'ai pas compris, c'est leur utilité, mon livre n'en fait pas mention. Accéder à une variable par un pointeur j'en vois pas l'utilité, j'utilise directement le nom de la variable, c'est plus pratique.

Certes, mais il y a des cas (paramètres, notamment) où on est obligé de passer l'adresse via un pointeur (tableaux, chaines, structures, données à modifier...).

Tu verras ça à l'usage.

De plus le but de la programmtion n'est pas que ce soit "pratique" (à écrire), mais que le comportement et les interfaces soient conformes à la specification et que les performances soient tenues.

Attends toi à un usage massif des pointeurs en C 'avancé' (et encore, pas trop...)

- Allocation dynamique
- Fichiers
- Structures de données non linéaires (tableaux a plusieurs dimensions, liste, arbres)...
- pointeurs de fonctions...

bref, c'est le quotidien du programmeur C...
30/08/2006, 17h46
Horrigan

Arrête de me faire peur ou je vais passer à Java :mrgreen:
J'essaierai de tester de bout de code dès que j'aurais le temps :)