problème mineur avec fork() et pipe()

Question

J'écris un petit programme qui implémente des tuyaux comme s'ils fonctionnaient dans la coque. -à-dire:

ls -hal | sort | grep p | wc 

il fonctionne très bien, la question mineure que sur une seule ligne, quand CMD_NO = n, la comparaison i biggerthan CMD_NO ne fonctionne pas, mais i = (CMD_NO-1) fait!. J'essaie de comprendre pourquoi dans ce cas particulier (la ligne est représentée par une ligne TROUBLED dans le code), ces instructions ne sont pas équivalentes. Merci beaucoup.

#include <sys/types.h> 
#include <unistd.h> 
#include <stdio.h> 
#include <string.h> 
#include <errno.h> 

#define READ_END 0 
#define WRITE_END 1 
#define CMDS_NO 5 

int main (int argc, char **argv) 
{ 
    pid_t pid; 
    int new_fds[2]; 
    int old_fds[2]; 

    char *array[CMDS_NO]; 
    char *param[CMDS_NO]; 

    array[0]="ls"; 
    array[1]="sort"; 
    array[2]="grep"; 
    array[3]="grep"; 
    array[4]="wc"; 

    param[0]="-hal"; 
    param[1]=NULL; 
    param[2]="p"; 
    param[3]="out"; 
    param[4]=NULL; 


    for (int i=0; i<CMDS_NO; i++) { 

     if (i<CMDS_NO) //if there is a next command 
      pipe(new_fds); 

     pid=fork(); 

     if (pid==0) { //if child 
      if (i!=0) { //if there is ap revoius commmand 
       dup2(old_fds[0], 0); 
       close(old_fds[0]); 
       close(old_fds[1]); 
      } 

      if (i!=(CMDS_NO-1)) { //TROUBLED LINE i<CMDS_NO does not work, 
            //if there is a next command 
       close(new_fds[0]); 
       dup2(new_fds[1],1); 
       close(new_fds[1]); 
      } 
      execlp(array[i], array[i], param[i], NULL); 

      } else { 
       if (i!=0) { //if there is a previous command 
        close(old_fds[0]); 
        close(old_fds[1]); 
       } 

       if (i<CMDS_NO) { //if there is a next command 
        old_fds[0] = new_fds[0]; 
        old_fds[1] = new_fds[1]; 
       } 
      } 
    } 
    if (CMDS_NO>1) { 
     close(old_fds[0]); 
     close(old_fds[1]); 
    }  

    while (1) { //wait for child processes to end 
     wait(NULL); 
     if(errno== ECHILD) { 
      printf("all children ended\n"); 
      break; 
     } 
    } 

return 0; 
} 

Answer 1

La condition i<CMDS_NO sera toujours vrai parce que i fonctionne 0-CMDS_NO-1. Je pense que vous vouliez écrire votre condition comme i<CMDS_NO-1. Bien sûr, i!=CMDS_NO-1 est également valide.

Notez que cela affecte plusieurs autres endroits où vous avez if (i<CMDS_NO); ceux-ci devraient également lire if (i<CMDS_NO-1).

Answer 2

C'est normal.

for (int i=0; i<CMDS_NO; i++) {

Cela signifie "boucle tant que i est inférieur à CMDS_NO". Dès que i == CMDS_NO la boucle s'arrêtera; donc, à l'intérieur de la boucle, le i le plus élevé atteindra jamais est CMDS_NO - 1.

PS: CMDS_NO est un nom de variable vraiment pauvre, appelez-le par exemple MAX_COMMANDS