Discussion :

[Bylox]

Bonjour à tous,

Etudiant en informatique, il m'est demandé de créer un shell en C qui execute toujours la même commande quelque soit ce que l'utilisateur écrive (et exit lorsque l'utilisateur rentre "exit").
J'ai réussi à le faire pour des commandes simples tel que "mkdir", "ls", etc dont 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
23
24
25
26
27
28
29
30
31
32
33
34
35
 
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <string.h>
 
int main ()
{   int i;
    char *ligne;
    char *tokens[100];
    char *cmd;
    char *commande = "mkdir brol";
 
    ligne=(char*)malloc(300);
    printf("$ ");
    fgets(ligne,256,stdin);
 
    while (strcmp(ligne,"exit\n"))
    {  
        i=0;
        cmd = strdup(commande);
        tokens[i]=strtok(cmd, " \n");
        while (tokens[i] != NULL) tokens[++i]=strtok(NULL," \n");
        if (fork()==0)
        {   execvp(tokens[0],tokens);
            printf("Commande invalide\n");
            exit(1);
        }
        wait(0);
        free(cmd);
        printf("$ ");
        fgets(ligne,256,stdin);
    }
    exit(0);
}

Le problème, c'est qu'il m'est demandé de faire cela pour la commande "mkdir brol && (cd brol;>f)" . Lorsque j'utilise cette commande, le programme crée 3 dossiers "&&", "brol;>f", "brol", et "(cd".

Comment faire pour que ce programme execute bien la commande "mkdir brol && (cd brol;>f)", c'est à dire, qu'elle crée un dossier "brol" et crée un fichier "f" dedans ?

Merci d'avance pour votre aide

[LittleWhite]

Bonjour,

Je souhaite avant même de répondre à votre question, vous faire quelques retours sur le code actuel.
Premier conseil, toujours initialisé les variables à leurs déclarations :

Code :

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

1
2
3
4
5
int i;
    char *ligne;
    char *tokens[100];
    char *cmd;
    char *commande = "mkdir brol";

deviendra :

Code :

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

1
2
3
4
5
int i = 0;
    char *ligne = NULL;
    char *tokens[100] = {0};
    char *cmd = NULL;
    char *commande = "mkdir brol";

Ensuite, pour chaque allocation de mémoire (malloc), il vous faut un free correspondant (sinon, c'est une fuite de mémoire) :
Pour

Code :

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

ligne=(char*)malloc(300);

faire un

Code :

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

free(ligne);

avant la fin du programme.

Ce test while (strcmp(ligne,"exit\n")) ne fonctionnera pas si j'entre "exit " (avec un espace à la fin). Il faudrait trouver une astuce pour enlever les surplus d'espace.
Maintenant, à propos de la gestion de la commande, l'idée est de comprendre comment on doit la gérer. En effet, même si l'espace est un délimiteur, cela ne suffit pas toujours.
Pour votre commande exemple

mkdir brol && (cd brol;>f)

elle doit être délimiter par '&&' (et exécuter la suite que si la première commande à réussie), puis par le ";". Les espaces ne doivent pas vraiment être pris en compte.
Je vous conseille, au lieu d'utiliser strtok, d'utiliser une fonction bien à vous, qui sera spécialisé dans la décomposition de la commande et qui pourra aussi vous dire si le résultat d'une commande doit être vérifié. Pour ce genre de fonction, je verrai une signature du genre :

Code :

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

int splitCommand(const char* input, char*** out, size_t* nbCommand);

Le type de retour peut être remplacé par un enum. input est l'entrée et out un tableau de chaine de caractère qui sera alloué dans la fonction (un peu dangereux certes), ce qui explique que j'ai trois niveaux de pointeurs. nbCommand contiendra le nombre de commande décomposées (un pointeur car c'est la fonction qui détermine la valeur).
Il est évidant qu'il faudra bien faire attention aux pointeurs et allocations de mémoire.

(Peut être qu'il y a plus simple, mais dans l'immédiat, je ne vois pas )