Invité de commande avec fgets

Version imprimable

Bonjour à tous,

Voilà j'ai décidé d'occuper mon après-midi en réalisant un programme ayant environ les mêmes fonctions que le célèbre client ftp (en ligne de commande :mrgreen:).

Mon premier but étant de réaliser un petit menu, j'envisageais une boucle dans laquelle on demande à l'utilisateur d'entrer une commande, ceci se répétant tant que la commande n'est pas bonne.

Quelque chose comme ça en somme :
Code:

1 2 3 4 5 6 7 8 FAIRE { Saisie de la commande Verification de la saisie (nettoyage) } TANT QUE saisie ne vaut rien (<enter>) switch(saisie) { ... etc ...
J'en venais donc à ce code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 char commd[10]; int size = 0; do { fgets(commd, sizeof(commd), stdin); size = sizeof(commd); if(size > 0) commd[size] = 0; } while (commd == NULL); switch(commd) { ...
J'aimerais donc un petit coup de main pour le nettoyage de la saisie, même si je pense que cela a déjà été demandé pas mal de fois, et aussi la valeur que je pourrais mettre entre les paranthèses du while, car j'imagine que ma chaine ne vaudra pas NULL si j'appuie sur enter :).

Merci beaucoup ;)

06/05/2006, 13h08
D[r]eadLock

Plusieurs erreurs déjà:
- sizeof(comm) te rendra toujours 10 (c'est un tableau donc sizeof te renvoie sa taille): http://c.developpez.com/faq/c/?page=...ABLEAUX_sizeof
- fgets mets déjà le \0 a la fin de la chaine.

Après pour le test du while, effectivement commd==NULL sera toujours faux (commd te donne l'adresse de ton tableau!). Tu peux soit utiliser strlen (longueur de la chaine different de zero) ou la comparaison a une chaine vide mais regarde là aussi : http://c.developpez.com/faq/c/?page=...hr_after_fgets

Enfin le switch ne marchera pas pour des chaines de caractère, il te faudra utiliser strcmp ou strcasecmp.

Et la FAQ est ton amie : http://c.developpez.com/faq/c/

Merci pour ta réponse.

J'ai modifié mon code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 char commd[10]; do { if(fgets(commd, sizeof(commd), stdin) != NULL) { char *p = strchr(commd, '\n'); if(p) { *p = 0; } } } while(strlen(commd) == 0); if(!strcmp(commd, "exit")) puts("OK !"); return 0;
Pour le moment ça fonctionne... Faudra juste faire plusieurs strcmp :P

On va dire que le problème est résolu, mais toutes vos remarques/suggestions sont les bienvenues !

Je laisse le topic comme ça au cas j'aurais d'autres questions ;)

Citation:

Envoyé par Freed0

Mon premier but étant de réaliser un petit menu,

Tu devrais commencer par étudier fgets() de plus près.

Ensuite, ce que tu veux faire n'est pas du tout un 'menu', mais un 'shell' (interpréteur de commandes) :

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
 
#include <stdio.h>
#include <string.h>
 
static void clean (char *s, FILE *fp)
{
   /* search ... */
   char *p = strchr (s, '\n'); /* <string.h> */
   if (p != NULL)
   {
      /* ... and kill */
      *p = 0;
   }
   else
   {
      /* purge */
      int c;
      while ((c = fgetc(fp)) != '\n' && c != EOF)
      {
      }
   }
}
 
int main()
{
   int end = 0;
   do
   {
      char cde[128];
      printf ("> ");
      fflush (stdout);
      fgets(cde, sizeof cde, stdin);
      clean(cde, stdin);
 
      /* analyser la commande et la traiter
         (c'est pas avec un 'switch' que tu vas t'en sortir)
         */
      if (strcmp(cde, "help") == 0)
      {
         printf ("\nhelp quit\n\n");
      }
      else if (strcmp(cde, "quit") == 0)
      {
         printf ("\nbye\n\n");
         end = 1;
      }
      else
      {
         printf("\nunknown command\n\n");
      }
   }
   while (!end);
 
   return 0;
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
> dsfgdfsg
 
unknown command
 
> help
 
help quit
 
> quit
 
bye

Des réponses à tes questions ici :

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

06/05/2006, 13h32
Freed0
Citation:
Envoyé par Emmanuel Delahaye

Tu devrais commencer par étudier fgets() de plus près.

Ensuite, ce que tu veux faire n'est pas du tout un 'menu', mais un 'shell' (interpréteur de commandes) :

Code:

...

Code:

1 2 ...

Des réponses à tes questions ici :

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers
http://emmanuel-delahaye.developpez....tes.htm#fflush
Merci pour ta réponse, je vais regarder ça de plus près.

J'ai maintenant un problème avec la connexion au serveur ftp...

Je pensais qu'il suffirait simplement de me connecter via le port 21, mais apparament non.

J'ai donc une fonction netw_connect(char *ip) qui se connecte à l'ip voulue.
Lorsque mon programme se lance, j'entre l'ip puis je crée le socket etc...

Voici quelques extraits de mon code :

Code:

1
2
3
4
5
6
7
8
9
10
char ip[20];
int ret_err = 0;
 
printf("Vers ");
fgets(ip, sizeof(ip), stdin);
clean(ip, stdin);
 
ret_err = netw_connect(ip);
 
return ret_err;

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
int netw_connect(char *ip)
{
    int ret_err = 0, sock_err = 0;
    SOCKET sock;
    SOCKADDR_IN sin = {0};
 
    sock = socket(PF_INET, SOCK_STREAM, 0);
 
    if(sock != INVALID_SOCKET)
    {
        printf("Socket %d ouvert.\n", sock);
 
        sin.sin_family = AF_INET;
        sin.sin_addr.s_addr = inet_addr(ip);
        sin.sin_port = htons(20);
 
        sock_err = connect(sock, (SOCKADDR *)&sin, sizeof(sin));
 
        if(sock_err != SOCKET_ERROR)
        {
            printf("Connecté à %s.", inet_ntoa(sin.sin_addr));
        }
        else
        {
            printf("Hote inconnu %s.", inet_ntoa(sin.sin_addr));
            ret_err = 1;
        }
    }
    else
    {
        perror("Socket.open");
        ret_err = 1;
    }
 
    return ret_err;
}

Et j'obtiens ce résultat :

Code:

1
2
3
4
5
 
ftp> open
Vers 193.74.22.160
Socket.open : No error
ftp> _

Une idée ?

Edit : J'avais oublié le WSAStartup(), honte à moi :oops: