質問編集履歴

コード修正

2018/03/20 18:11

投稿

ijuya_yika

スコア50

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -502,7 +502,7 @@
     if( pid2 == 0 ){
-        if( (fd = open(cmd_right[0], O_RDWR | O_CREAT, 0600)) < 0){
+        if( (fd = open(cmd_right[0], O_RDWR | O_CREAT | O_APPEND, 0600)) < 0){
             perror("open failed\n");

コード修正

2018/03/20 18:11

投稿

ijuya_yika

スコア50

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -166,7 +166,7 @@
     if( (*type) != 0 ){
-        int start = ( (*type) == 1 )? splitPosition + 2 : splitPosition + 3;
+        int start = ( (*type) == 1 )? splitPosition + 1 : splitPosition + 2; /* 修正：>> or | の前後にスペース無しに対応 */
         int len   = strlen(command) - start;

コード修正

2018/03/20 03:50

投稿

ijuya_yika

スコア50

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -172,9 +172,15 @@
-        substr(command, part1, 0, splitPosition);
+        if( substr(command, part1, 0, splitPosition) < 0 ||
-        substr(command, part2, start, len);
+            substr(command, part2, start, len) < 0){
+            perror("substr\n");
+            exit(1);
+        }
     } else {

コード修正訂正、質問文訂正

2018/03/20 03:12

投稿

ijuya_yika

スコア50

test CHANGED Viewed

	@@ -1 +1 @@
1	- 【C言語】シェルでリダイレクトを実装したいが無限ルー~~プになってしまう~~
1	+ 【C言語】シェルでリダイレクトを実装したいがプロセスが無限に作成されるエラーが起きる

test CHANGED Viewed

@@ -12,7 +12,7 @@
 ***
+【追記１】
 指摘を受けて：
@@ -32,6 +32,12 @@
+***
+【追記２】
+239行目に`exit(1)`を追加した所正常に動くようになりました。なぜここに`exit(1)`がないと無限プロセス作成エラーになるのか教えていただける方がいたらお願いしたいです。
 ### 該当のソースコード
@@ -518,6 +524,12 @@
         close(fd);
+        /**** 追記２ : exit(1)がないと無限プロセスになるため追加 ****/
+        exit(1);
     }
     else {

コード修正/訂正

2018/03/20 02:57

投稿

ijuya_yika

スコア50

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -10,22 +10,416 @@
+***
+指摘を受けて：
+`getCommand`の他、省略していた関数やデバッグ文を記述しました。
+リダイレクトのみ機能していない状態だったので該当のコードのみ載せたのですが誤解を招いてしまいすみません。
+以下のコードをコンパイルして動かすとパイプ、リダイレクトを含まないコマンド及びパイプコマンドは問題なく動くのですが、リダイレクトの場合にプロセスが無限に作られてしまい`CTRL+C`等でプログラムを終了できない状態になってしまいます。
 ### 該当のソースコード
-```C
-void Redirect(char* part1, char* part2){
-    char* Left[20];
-    char* Right[20];
+```C
+#include <stdio.h>
+#include <stdlib.h>
+#include <unistd.h>
+#include <string.h>
+#include <errno.h>
+#include <sys/wait.h>
+#include <sys/stat.h>
+#include <sys/types.h>
+#include <fcntl.h>
+#define MAX_ARG 50
+#define READ    0
+#define WRITE   1
+/* @戻り値	0 => 普通のコマンド, 1 => パイプコマンド, 2 => リダイレクトコマンド */
+void Type(char* command, int* type, int* splitPosition){
+    int pos = 0;
+    (*type) = 0;
+    while(command[pos] != '\0'){
+        if(command[pos] == '|'){
+            (*type) = 1;
+            (*splitPosition) = pos;
+        }
+        else if(command[pos] == '>' && command[pos+1] == '>'){
+            (*type) = 2;
+            (*splitPosition) = pos;
+        }
+        pos++;
+    }
+}
+int substr(char* S1, char* S2, int P, int L){
+    if( P < 0 || L < 0 || L > strlen(S1) ){
+        return -1;
+    }
+    for( S1 += P; *S1 != '\0' && L > 0; L--){
+        *S2++ = *S1++;
+    }
+    *S2 = '\0';
+    return 0;
+}
+void split(char* command, char* part1, char* part2, int* type){
+    int splitPosition = 0;
+    Type(command, type, &splitPosition);
+    #ifndef DEBUG
+    printf("splitPosition = %d\n", splitPosition);
+    #endif
+    if( (*type) != 0 ){
+        int start = ( (*type) == 1 )? splitPosition + 2 : splitPosition + 3;
+        int len   = strlen(command) - start;
+        substr(command, part1, 0, splitPosition);
+        substr(command, part2, start, len);
+    } else {
+        strcpy(part1, command);
+    }
+    #ifndef DEBUG
+    printf("part1 = %s\n", part1);
+    printf("part2 = %s\n", part2);
+    #endif
+}
+void getCommand(char* command){
+    scanf("%255[^\n]%*[^\n]", command);
+    scanf("%*c");
+    #ifndef DEBUG
+    printf("command = %s\n", command);
+    #endif
+}
+int to_token(char* command, char** tokens){
+    int argc = 0;
+    tokens[argc] = strtok(command, " ");
+    while(argc < MAX_ARG){
+        tokens[++argc] = strtok(NULL, " ");
+        if(tokens[argc] == NULL){
+            break;
+        }
+    }
+    return argc;
+}
+int Parse(char* command, char** parsedCommand){
+    int argc = to_token(command, parsedCommand);
+    int ret;
+    if(strcmp( parsedCommand[argc - 1], "&") == 0){
+        parsedCommand[argc - 1] = NULL;
+        ret = 1;
+    } else {
+        ret = 0;
+    }
+    return ret;
+}
+void RegCmd(char* part1){
+    char* parsedCommand[20];
+    int background;
+    pid_t pid;
+    background = Parse(part1, parsedCommand);
+    if( (pid = fork()) == 0){
+        execvp(parsedCommand[0], parsedCommand);
+        perror("execvp failed\n");
+        exit(1);
+    } else {
+        if(!background){
+            waitpid(pid, NULL, 0);
+        }
+    }
+}
+void PipeCmd(char* part1, char* part2){
+    char* cmd_left[20];
+    char* cmd_right[20];
     int fds[2];
+    pid_t pid1, pid2;
+    int background;
+    if( pipe(fds) < 0){
+        perror("pipe failed\n");
+        exit(1);
+    }
+    background = Parse(part1, cmd_left) ||
+                 Parse(part2, cmd_right);
+    if( (pid1 = fork()) < 0){
+        perror("fork failed\n");
+        exit(1);
+    }
+    if( pid1 == 0 ){
+        dup2(fds[READ], 0);
+        close(fds[WRITE]);
+        execvp(cmd_right[0], cmd_right);
+        perror("execvp failed\n");
+    }
+    if( (pid2 = fork()) < 0){
+        perror("fork failed\n");
+        exit(1);
+    }
+    if( pid2 == 0 ){
+        dup2(fds[WRITE], 1);
+        close(fds[READ]);
+        execvp(cmd_left[0], cmd_left);
+        perror("execvp failed\n");
+    }
+    else {
+        close(fds[0]);
+        close(fds[1]);
+        if(!background){
+            waitpid(pid1, NULL, 0);
+            waitpid(pid2, NULL, 0);
+        }
+    }
+}
+void RedirectCmd(char* part1, char* part2){
+    char* cmd_left[20];
+    char* cmd_right[20];
+    int fds[2];
     int fd;
     int count;
@@ -38,50 +432,90 @@
-    pipe(fds);
+    if( pipe(fds) < 0){
+        perror("pipe failed\n");
+        exit(1);
+    }
+    background = Parse(part1, cmd_left) ||
+                 Parse(part2, cmd_right);
-    pid1 = fork());
+    if( (pid1 = fork()) < 0){
+        perror("fork failed\n");
+        exit(1);
+    }
-    if(pid1 == 0){
+    if( pid1 == 0 ){
-        // >> より左側のコマンドを実行
         dup2(fds[WRITE], 1);
         close(fds[READ]);
-        execvp(Left[0], Left);
+        execvp(cmd_left[0], cmd_left);
+        perror("execvp failed\n");
-        exit(1);
+        exit(1);
-    }
+    }
-    pid2 = fork();
+    if( (pid2 = fork()) < 0){
+        perror("fork failed\n");
+        exit(1);
+    }
-    if(pid2 == 0){
+    if( pid2 == 0 ){
-        // >> より右側のファイルを作成し左側のコマンドを書き込む
-        fd = open(Right[0], O_RDWR | O_CREAT, 0600)) < 0);
+        if( (fd = open(cmd_right[0], O_RDWR | O_CREAT, 0600)) < 0){
+            perror("open failed\n");
+            close(fd);
+            exit(1);
+        }
         dup2(fds[READ], 0);
         close(fds[WRITE]);
         while( (count = read(0, &c, 1)) > 0){
             write(fd, &c, 1);
         }
         close(fd);
     }
@@ -108,9 +542,9 @@
     char cmd[256];
-    char p1[128];
+    char part1[128];
-    char p2[128];
+    char part2[128];
     int  type = -1;
@@ -124,15 +558,41 @@
     while( (strcmp(cmd, "quit")!=0) &&
-           (strcmp(cmd, "exit")!=0)    )
+           (strcmp(cmd, "exit")!=0)    ){
-        // commandline は常に **** >> **** か quit or exit の形式であると想定
+        split(cmd, part1, part2, &type);
+        if(type == 0){
+            RegCmd(part1);
+        }
+        else if(type == 1){
+            PipeCmd(part1, part2);
+        }
+        else if(type == 2){
-        Redirect(p1, p2);
+            RedirectCmd(part1, part2);
+        }
+        #ifndef DEBUG
+        printf("\nType = %d", type);
+        #endif
-        printf("\nMyshell> ");
+        printf("\n> ");
         getCommand(cmd);
@@ -142,4 +602,8 @@
 }
 ```

文章訂正

2018/03/20 02:49

投稿

ijuya_yika

スコア50

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -4,7 +4,7 @@
 自作シェルでリダイレクト(>>)を実装したいのですが
-以下のコードで実行すると無限ループに陥ってしまいます。
+以下のコードを実行すると無限ループに陥ってしまいます。
 このコードのどこが無限ループの原因となっているのか教えていただきたいです。