14

私はCで小さなLinuxシェルを書いていますが、これで終わりに近づいています。ユーザーからコマンドを受け取り、スペースで区切ってargsに格納します。次の例では、argsに次のものが含まれているとしましょう。

args[] = {"ls", "-l", "|", "wc"};

私の関数は引数を取り込んで、パイプの数も取り込んでいます。私は自分のコードに可能な限りコメントを付けました。ここにあります：

int do_command(char **args, int pipes) {
    // The number of commands to run
    const int commands = pipes + 1;
    int i = 0;

    int pipefds[2*pipes];

    for(i = 0; i < pipes; i++){
        if(pipe(pipefds + i*2) < 0) {
            perror("Couldn't Pipe");
            exit(EXIT_FAILURE);
        }
    }

    int pid;
    int status;

    int j = 0;
    int k = 0;
    int s = 1;
    int place;
    int commandStarts[10];
    commandStarts[0] = 0;

    // This loop sets all of the pipes to NULL
    // And creates an array of where the next
    // Command starts

    while (args[k] != NULL){
        if(!strcmp(args[k], "|")){
            args[k] = NULL;
            // printf("args[%d] is now NULL", k);
            commandStarts[s] = k+1;
            s++;
        }
        k++;
    }



    for (i = 0; i < commands; ++i) {
        // place is where in args the program should
        // start running when it gets to the execution
        // command
        place = commandStarts[i];

        pid = fork();
        if(pid == 0) {
            //if not last command
            if(i < pipes){
                if(dup2(pipefds[j + 1], 1) < 0){
                    perror("dup2");
                    exit(EXIT_FAILURE);
                }
            }

            //if not first command&& j!= 2*pipes
            if(j != 0 ){
                if(dup2(pipefds[j-2], 0) < 0){
                    perror("dup2");
                    exit(EXIT_FAILURE);
                }
            }

            int q;
            for(q = 0; q < 2*pipes; q++){
                    close(pipefds[q]);
            }

            // The commands are executed here, 
            // but it must be doing it a bit wrong          
            if( execvp(args[place], args) < 0 ){
                    perror(*args);
                    exit(EXIT_FAILURE);
            }
        }
        else if(pid < 0){
            perror("error");
            exit(EXIT_FAILURE);
        }

        j+=2;
    }

    for(i = 0; i < 2 * pipes; i++){
        close(pipefds[i]);
    }

    for(i = 0; i < pipes + 1; i++){
        wait(&status);
    }
}

私の問題は、プログラムが正しく実行されているのに、プログラムが奇妙に動作していることです。あなたが私を助けてくれることを望んでいました。

たとえば、私がlsを実行する場合| wc、出力はls|の出力です。wcですが、出力のwcである必要がある場合でも、そのすぐ下にある単純なlsの出力も出力します。

別の例として、ls-l|を試してみると wc 、 wcの最初の数が表示されますが、出力のwcである必要がある場合でも、ls-lの出力がその下に表示されます。

前もって感謝します！:)

4

1 に答える 1

9

さて、私は1つの小さなエラーを見つけました。これ

       if( execvp(args[place], args) < 0 ){

する必要があります

       if( execvp(args[place], args+place) < 0 ){

あなたのバージョンは、他のすべての最初のコマンドに引数を使用しました。それ以外は、私のために働きます。

于 2012-10-01T19:17:37.830 に答える