c言語のリダイレクトのやり方

Question

C言語を使用して、ファイル内の空白や改行の数をカウントして出力するプログラムを作っています。

#include<stdio.h>
#include<stdlib.h>
#include <ctype.h>

int main(void){
    char filename[30];
    scanf("%s",filename);/*ファイルの名前を指定*/
    FILE *f;

    int c;
    int d=0,o=0,nl=0,s=0;

    f=fopen(filename,"r");/*ファイルを開く*/

    if(f==NULL){
        printf("%sがオープンできません！\n",filename);
       return 1;
    }


    while((c=fgetc(f))!=EOF){
        if(isdigit(c) !=0) d++; /*数字カウント*/
        if(c==' ') s++;　　　　 /*空白カウント*/
        if(c=='\n') nl++;　　　 /*改行カウント*/
        if(c!=' ' && c!='\n') o++;　/*それ以外カウント*/
    }
    
    fclose(f);/*ファイルを閉じる*/
    printf("%d  digit\n%d  spaces\n%d  new lines\n%d  others",d,s,nl,o);
    return 0;

} 

以上のようにコードを書きました。
ここから、リダイレクションを使って、コマンドプロンプトにファイル名を入力するとデータをとってきて出力したいのですがどうすれば良いのでしょうか...？

count.cというプログラム名で書いており、

$ ./count < sample.dat

と入力してもエラーとなってしまいます。
どなたかお助けいただけると大変助かります。

Comments

[LouiS0616]

コンパイルはしてないんですか？

[null0723]

すみません、説明不足でした。コンパイルはこのように行いました。
$ gcc -o count count.c

Answer 1

そりゃ標準入力からファイル名読んでオープンしようとしてもエラーになりますわな。
ファイル名読むのをどうにかしましょうよ。

そのうえで、すべての読み込みを標準入力からにする、ということでコード書いてみてくだされ

Comments

[null0723]

コンパイル後、リダイレクションを使わずにプログラム名のみ実行した時はできたのですが、何が違ったのでしょうか? 的外れな質問であれば申し訳ないです。

[y_waiwai]

リダイレクトしたときはまずファイル名を標準入力から読みますわな。
それでどーなるか考えてみましょう

[null0723]

リダイレクトの入力とscanfの入力と被っていることが問題なのでしょうか?
このプログラムにおいてscanfは必要なかったのでしょうか?

[null0723]

すみませんリダイレクトに関する知識が浅かったです。
これをきっかけにもう一度考え、ファイルがいかに開かれるかがわかりました、ありがとうございました!
.

[y_waiwai]

リダイレクトでするなら、必要なかったということですな。
別回答のコードでもその箇所は削除されてるでしょ

Answer 2

リダイレクトを使うなら、次のように書けば充分だと思いますよ。

#include<stdlib.h>
#include <ctype.h>

int main(void){
    int c;
    int d=0,o=0,nl=0,s=0;

    while((c=fgetc(stdin))!=EOF){
        if(isdigit(c) !=0) d++; /*数字カウント*/
        if(c==' ') s++;         /*空白カウント*/
        if(c=='\n') nl++;       /*改行カウント*/
        if(c!=' ' && c!='\n') o++;  /*それ以外カウント*/
    }

    printf("%d  digit\n%d  spaces\n%d  new lines\n%d  others",d,s,nl,o);
    return 0;
}

標準入力をsample.datにして読み取ってくれます。

Comments

[null0723]

できました、ありがとうございます！
fopen ではできないのはなぜなのでしょうか?

[LouiS0616]

scanf("%s",filename) がnull0723さんの想定通りに働いていないのです。
sample.datのテキストを読み込んで、その名前のファイルを開こうとしてしまっています。

filenameを出力してみるとなんとなく状況が分かるかと。
未定義の動作を避けるため、いちおう出力前に filename[29] = '\0' しておいてください。

[null0723]

なるほど、、、
リダイレクトをしなければこのプログラム自体は動いたのですが、プログラム自体におかしなところはないでしょうか?

[null0723]

トンチンカンな質問でしたすみません、、、
リダイレクトしているのにfopenやscanfで訳の分からない指定を下からおかしかったんですね,,,