c - 循环执行多个管道

我很快就要找出一个我一直在为用c编写的linux shell编写的程序了。我一直想让这个程序运行一段时间，我决定重新拿起它，在过去的几周里一直在修改它。
对于以下代码，请记住名为arrayofcommands的数组是动态填充的。我的代码用当前运行的命令填充arrayofcommands。为了我的示例，我们将运行命令ls-l wc，arrayofcommands将填充以下内容，具体取决于循环的时间：

//Pass #1
arrayOfCommands[]= ("ls", "-l", NULL)

//Pass #2
arrayOfCommands[]= ("wc", NULL)

以下是我目前掌握的情况：

//PIPING
int do_command(char **args, int pipes) {
    // pipes is the number of pipes in the command
    // (In our example, one)


    // The number of commands is one more than the
    // number of pipes (In our example, two)
    const int commands = pipes + 1;  //Ex: 2
    int i = 0;

    // Set up the pipes
    int pipefds[2*pipes];

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

    // Variables
    int pid;
    int status;
    char *str_ptr;

    int j = 0;
    for (i = 0; i < commands; ++i) {

        // A magic function that updates arrayOfCommands with
        // the current command goes here.  It doesn't make
        // sense in the context of the code, so just believe me! :)

        // Ex: The contents will be "ls -l" or "wc" depending on
        // which time through the loop we are

        pid = fork();

        if(pid == 0) {
            //if not last command
            if(i < commands){
                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);
                }
            }

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

            // Should any of the below inputs be *arrayOfCommands or
            // **arrayOfCommands or &arrayOfCommands?
            // I'm REALLY bad with pointers

            if( execvp(arrayOfCommands, arrayOfCommands) < 0 ){
                    perror(arrayOfCommands);
                    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);
}

当我运行这个时，我会得到一些错误：
dup2:错误的文件描述符
ls:：没有这样的文件或目录
ls:wc:没有这样的文件或目录
有人能帮我弄清楚以下两件事吗：
为什么我会犯这些错误？
在execvp函数中，我要查找哪种if指针？ArrayOfCommands已初始化为char*ArrayOfArgs[]

最佳答案

第一件事：

//if not last command
if(i < commands)

应该是

if(i < commands -1)

因为i从0变为commands -1
这应该可以解决dup2：错误的文件描述符
ls:：没有这样的文件或目录
ls:wc:没有这样的文件或目录
是由格式不正确的arrayOfCommands引起的。它必须由

char * arrayOfCommands[] = {"ls", "-l", NULL};

和

char * arrayOfCommands[] = {"wc", NULL};

分别通过execvp(arrayOfCommands[0], arrayOfCommands)调用
本质上arrayOfCommands的格式必须与argv的参数向量（通常为int main(int argc, char** argv)）的格式相同。