入力のリダイレクトを用いた処理

Question

質問

作成したプログラム以下の構文でシェルに与えることを考えます。

./a.out < test.txt

その際、"test.txt"の終端EOFを読み取ってプログラムを終了したいのですが、どうすればよいでしょうか？

現状

int main(){
   char buf[1024];
   char prompt[NAMESIZE] = "Command: ";
   for(;;){
      printf("%s", prompt);
      if(fgets(buf, 1024, stdin) == NULL){
         exit(0);
      }
      (以下、入力がきちんと与えられたときの処理)
   }
   return 0;
}

入力のリダイレクトに対応するため、if(scanf("%1023s", buf) == EOF){のブロックを追加したのですが、ファイルの終端でもプログラムが終了しません。

test.txtの内容

pushd
dirs
cd
ls *
alias sl ls
popd
sl -l

と書いているのですが、上から6行はきちんと読み取れていることを確認していますが、
7行目は「sl -lushd」
となる上、無限ループが終了しません。

さらに、（架空の8行目？として）dirsが読み取られています。…以下同様に無限ループが終了しない

Comments

[jimbe]

fgets が NULL を返したら EOF (かエラー) だったと思いますが、それ以外になぜ判定が必要なのでしょう。

[SUNMOON_14]

それも理解していますし、scanfが明らかにおかしいことも分かっていますが、解決策が分かりません。
標準入力をファイルの中身に変えた場合、ファイルの終わりで読み取りを終了したいのですが、どうすればよいでしょうか？

EOFを検出してプログラムを終了させればよいのでしょうが、その方法が分かりません。。。

[SUNMOON_14]

for(;;){
printf("%s", prompt_str); // プロンプトを表示
if(fgets(command_buffer, BUFLEN, stdin) == NULL){
printf("\n");
continue;
}
(以降の処理省略：command_bufferを加工して引数化して、自作コマンドに対応)

という記述です。つまり、自作シェルのコマンドプロンプトを表示し、標準入力が何もなかったら改行するという一節なのですが、ここにreturn文を入れてしまうと、リダイレクトではない標準入力でバグが出ると思います。

リダイレクトによる入力にも対応するための方法が分からないです。

[jimbe]

> リダイレクトではない標準入力でバグが出ると思います

「出ました」では無く「出ると思います」とはどういうことでしょう。
既存のシェル(sh? csh?)の動作を参考に作られていると思いましたが、fgets のみで作った場合に、コンソール入力・ファイル入力(リダイレクト)両方で、既存のシェルの動作と同じになっているかの確認はされていないのでしょうか。

[SUNMOON_14]

>「出ました」では無く「出ると思います」とはどういうことでしょう。

自明ではないですか？
「Command: 」という待機画面でエンターキーのみを押した場合、改行されて再度「Commands: 」を出力する本プログラムにおいて、改行後にreturn 0;と記述されているプログラムは間違いなく動作終了してまったく使い物にならないと思います。（これを「バグ」と表現しました）

コンソール入力には対応していますが、リダイレクトには対応していません。
それを解決する方法が分からないため、質問しています。

[SUNMOON_14]

コンソールで標準入力する際には、exitと打つことで動作終了するようにプログラムしています。
リダイレクトのときはユーザーがexitと打つことなく動作終了したいのですが、どうすればよいでしょうか？

[jimbe]

すいません、作ろうとされているプログラムの仕様を確認させてください。
まず、作られているのは ```./a.out < test.txt``` にある a.out に該当するプログラムですね。
そしてそのソースコードは「現状」の所にある c のコードですね。
仕様としては、プロンプト表示後、一行入力されたら、現在（省略していますが）continue; としている部分で行の内容に合わせた処理(cd や ls等)を実行し、またプロンプトの表示に戻る、ということで合っていますか。
そして、標準入力がファイルにリダイレクトされている場合は、そのファイルの各行の処理を（ファイルを読みながら連続して）実行（＝バッチ処理）して、またプロンプト表示に戻る、ということでしょうか。

[SUNMOON_14]

以下に述べる箇所を除いて、おっしゃる通りです。

[jimbe]

上のコメントを書くのに時間が掛かってしまい exit の件が読めませんでした。さらに考え直します。

[SUNMOON_14]

>そして、標準入力がファイルにリダイレクトされている場合は、そのファイルの各行の処理を（ファイルを読みながら連続して）実行（＝バッチ処理）して、またプロンプト表示に戻る、ということでしょうか。

リダイレクトした場合はファイル内容を読み切ったのち、プロンプトに戻らず、終了したいです。

Answer 1

EOFになれば、fgetsがNULLを返すので、if(fgets(buf, 1024, stdin) == NULL)が成立して、
次にscanfでもすでにEOFなのでEOFが返り、return 0;が実行されるはずです。

そうなっていないのなら、実行しているプログラムと質問文に書いてあるプログラムが違うのでしょう。

なお、EOFの判定を2回するのは無意味なので、if(fgets(buf, 1024, stdin) == NULL)の判定だけで良いです。

Comments

[SUNMOON_14]

プログラムを書き直しました。
これでもファイルの内容を無限ループで読み取ってしまいます。
実行しているプログラムと質問文に書いているプログラムは本質的に同じだと思いますが。。。

[otn]

> 実行しているプログラムと質問文に書いているプログラムは本質的に同じだと思いますが。。。
ということは違うプログラムと言うこと？？？？
質問文に書いたプログラムを実行もせずに質問しているのですか？？？？

[SUNMOON_14]

違うプログラムです。というか、600行程度あるので関係のないところは除いています。
質問文に書いているプログラムは実行しています。

[SUNMOON_14]

test.txtを
dirs
cd

とすればプログラムは終了できたので別のところに原因がありそうです。

[otn]

質問文のコード（日本語の行は削除）で、質問文のtest.txtで、無限ループにはならないので、やはり質問文のコードを実行しているとは思えないですね。手順をよく見直しましょう。

[SUNMOON_14]

ありがとうございます。
test.txtを
dirs
cd
として、./a.out < test.txtとしてprintfで出力したとき
dirsはdirs
cdはcdcと出力されました。（テキストの内容を変えても毎回最後の行が変です）

何か原因は考えられますか？

―――
test.txtを
dirs
cd
history
としたとき、
dirsはdirs、
cdはcd、
historyはhistorycd
（なぜかもう一度読み取って）historyhistor

[otn]

質問文のコードではそういうことは起こらないので、ちゃんと質問文のコードが実行されていることを確認しましょう。
コンパイル済みの実行ファイルを全部削除してから、質問文のコードをコンパイルすれば、間違って他のプログラムを実行することも無いでしょう。
プログラムの冒頭に printf("2021-07-20 01:35\n"); とか入れて、確かにこのコードかどうか確認するとか。

もしくは、入力ファイルの中身がおかしいとか。
cat -v test.txt
してちゃんと表示されますか？

Answer 2

試し々々で以下のようなコードになりました。
リダイレクトされているかを判断している fromFile 関数の中の _S_ISREG() につきましては、当初 FIFO のつもりでしたが、こちらの環境 Windows/MinGW では判断が付かなかったので、実際にリダイレクト有り/無しで試して _S_IFMT の中で値が異なっていた REG ビットを使いました。
fprintf(stderr,～ でイロイロ表示していますが、どれも確認の為なので全て消して大丈夫…だと思います。

#include <stdio.h>
#include <string.h>
#include <sys/stat.h>

//コマンド実行
void run(char *commandline) {
  fprintf(stderr, "[run - '%s']\n", commandline);
}
//標準入力がリダイレクトされているかを返す.
int fromFile() {
  struct stat sb;
  if(fstat(STDIN_FILENO, &sb) != -1) {
    fprintf(stderr, "[sb/st_mode=0x%04x]\n", sb.st_mode);
    return _S_ISREG(sb.st_mode); //レギュラー？
  }
  return 0; //fstatのエラーは無視
}

int main() {
  char commandline[256];
  int isBatch = fromFile();
  fprintf(stderr, "[isBatch=%s]\n", isBatch?"yes":"no");
  while(1) {
    if(!isBatch) printf("prompt:");
    memset(commandline, 0, sizeof(commandline));
    if(fgets(commandline, 256, stdin) == NULL) break; //EOF
    //改行コード削除
    if(commandline[strlen(commandline)-1] == '\n') commandline[strlen(commandline)-1]='\0';
    //終了
    if(!strcmp(commandline,"exit")) break;
    //コマンド実行
    run(commandline);
  }
  fprintf(stderr, "[exit]\n");
}

Answer 3

fgetsのみが入力をとって、NULLを返したらEOF。
つまりそのコードで、scanfでなく、sscanfをつかうようにすればいいのでは。

Answer 4

if(fgets(buf, 1024, stdin) == NULL){

     if(scanf("%1023s", buf) == EOF){

1行読み捨てて、それからscanfしてるけど、これってなにしてるんでしょう
＃ああ、違うか。全行読み捨てたあとscanfしてるのか。よけい意味不明

Comments

[SUNMOON_14]

添付したプログラムでは省略していますが、fgetsで得たbufは、後に加工します。
scanfに関しては適当です。
fgetsで標準入力を取得する仕様は変えたくないです。（他のプログラム600行に影響が出るため）

入力のリダイレクトでファイルの中身が入力として与えられた場合、ファイルの終わりでプログラムを終える方法が分かりません。
教えていただけないでしょうか。