Schreiben meiner eigenen shell... stecken auf Rohre?

Erster Vorschlag: Symbolische Konstanten, die besser sind als die magischen zahlen.

const int PIPE_READ = 0;
const int PIPE_WRITE = 1;
int fd[2];
pipe(fd);
//Now you can refer to fd[PIPE_READ] and fd[PIPE_WRITE].

Zweiter Vorschlag: machen Sie einen Schritt zurück und überlegen Sie, was Sie versuchen zu erreichen.

Sie wollen, um zu laichen zwei Prozessen, wobei der erste Prozess stdout verbunden, um den zweiten Prozess stdin. Richtig?

So, in C bedeutet dies, dass Sie benötigen, um call pipepass fd[PIPE_WRITE] auf das erste child-Prozesses, der dup2 1, und pass fd[PIPE_READ] dem zweiten Kind Prozess wird dup2 0.

Einfach an forkAndExecute's Prototyp zeigt, dass es nicht tun:

void forkAndExecute( char* arrayOfWords[] , vector *vectorOfPIDs , 
    bool hasNextCmd , bool hasPrevCmd);

Es behandelt nur einen einzelnen Befehl, und aus der Blick auf die argument-Liste, es sei denn, es greift zu bösen globalen Variablen, es gibt keine Möglichkeit für Sie zu erhalten, eine Datei-Deskriptor von seinem PrevCmd oder erhalten Sie einen Datei-Deskriptor von seinem NextCmd.

Überlegen, wie die Verwaltung der Datei-Deskriptoren, die Sie benötigen, und die Neugestaltung forkAndExecute in der Lage sein, diese zu verwenden.

InformationsquelleAutor der Antwort Josh Kelley

ok Das ist für mich arbeiten. Hoffe das hilft dir:

/************************
function: void pipeCommand(char** cmd1, char** cmd2)
comment: This pipes the output of cmd1 into cmd2.
**************************/
void pipeCommand(char** cmd1, char** cmd2) {
  int fds[2]; //file descriptors
  pipe(fds);
  //child process #1
  if (fork() == 0) {
    //Reassign stdin to fds[0] end of pipe.
    dup2(fds[0], STDIN_FILENO);
    close(fds[1]);
    close(fds[0]);
    //Execute the second command.
    //child process #2
    if (fork() == 0) {
        //Reassign stdout to fds[1] end of pipe.
        dup2(fds[1], STDOUT_FILENO);
        close(fds[0]);
        close(fds[1]);
        //Execute the first command.
        execvp(cmd1[0], cmd1);
    }
    wait(NULL);
    execvp(cmd2[0], cmd2);
    }
    close(fds[1]);
    close(fds[0]);
    wait(NULL);
}

InformationsquelleAutor der Antwort wakingrufus

Lesen Sie die Quellcode von Bashum zu sehen, wie Sie es Taten.

InformationsquelleAutor der Antwort RoliSoft

Hier sind die Hinweise auf Rohr aus der Systemprogrammierung-Klasse habe ich im letzten semester.

InformationsquelleAutor der Antwort thepocketwade

gut, ich habe darauf keine Antwort, aber ich arbeite an dem gleichen problem atm. Werde ich teilen, was ich habe. Es funktioniert für die beiden Befehle, aber nach dem es fertig ist läuft, i/o-unterbrochen sind. auf eine seltsame Weise, die ich nicht in der Lage gewesen, um herauszufinden, noch. den Klempner anrufen!

void pipeCommand(char** cmd1, char** cmd2) {
  int fds[2]; //file descriptors
  pipe(fds);
  int oldIn, oldOut;
  //child process #1
  if (fork() == 0) {
    //Reassign stdin to fds[0] end of pipe.
    oldIn = dup(STDIN_FILENO);
    dup2(fds[0], STDIN_FILENO);
    close(fds[1]);
    close(fds[0]);
    //Execute the second command.
    execvp(cmd2[0], cmd2);
  //child process #2
  } else if ((fork()) == 0) {
    oldOut = dup(STDOUT_FILENO);
    //Reassign stdout to fds[1] end of pipe.
    dup2(fds[1], STDOUT_FILENO);
    close(fds[0]);
    close(fds[1]);
    //Execute the first command.
    execvp(cmd1[0], cmd1);
  //parent process
  } else
    wait(NULL);
    dup2(oldIn, STDIN_FILENO);
    dup2(oldOut, STDOUT_FILENO);
    close(oldOut);
    close(oldIn);
}

Habe ich das Gefühl es hat zu tun mit dem, was bin oder nicht bin, machen die nach dem wait()

InformationsquelleAutor der Antwort wakingrufus