コンパイルエラー 整数型とポインタの比較

Question

基礎的な質問でごめんなさい
textin.txt ファイルの空白、改行をすべて取り除いて、textout.txtファイルに出力したいです

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

int main(){

    FILE *fp,*fq;
    char str[100];
    int i;

    fp=fopen("textin.txt","r");
    if(fp==NULL){
        printf("File open error1\n");
        return 1;
    }//File読み込みに失敗した場合

    fq=fopen("textout.txt","w");
    if(fq==NULL){
        printf("File open error2\n");
        return 1;
    }//File読み込みに失敗した場合

    while(fscanf(fp,"%s",str)!=NULL){

        
        for(i=0;i<=strlen(str)-1;i++){
            fputc(str[i],fq);
        }

    }

    fclose(fp);
    fclose(fq);

    return 0;
}

このコードだとコンパイル時にwhile文の部分で「整数とポインタを比較しています」というエラーが出ます

疑問点は2つです
①なぜこのようなエラーがでるのか
（fscanfがint型としてNULLがポインタ型と見られてしまうのはなぜか）

②エラーが出ない書き方はどのようなものか

ご教授お願いしますm(__)m

Answer 1

＞なぜこのようなエラーがでるのか
（fscanfがint型としてNULLがポインタ型と見られてしまうのはなぜか）

fscanf()の戻り値がintでNULLが（大抵）void*型だからです。

＞エラーが出ない書き方はどのようなものか
fscanf()の戻り値を確認しましたか？・・・EOF(-1)あるいは読み取った項目数です。

[たとえば・・・]
文字列から空白と改行を取り除く関数を作って、fgets()で一行読んで処理をさせ書き出すようにします

時間があったので作ってみました。（stdin→stdoutです）

~/test/ctst >cat tst08.c 
#include <stdio.h>
#include <ctype.h>

void DelSpace(char *strp);

int main()
{
    char buf[1024];
    //
    while(fgets(buf, sizeof buf, stdin)){
        DelSpace(buf);
        fputs(buf, stdout);
    }
    //
    return 0;
}


void DelSpace(char *strp)
{
    char *sop= strp;
    char *dip= strp;
    //
    while(*sop){
        if(!isspace(*sop)){
            *dip++= *sop;
        }
        sop++;
    }
    *dip= '\0';
}

~/test/ctst >./a.out <tst08.c
#include<stdio.h>#include<ctype.h>voidDelSpace(char*strp);intmain(){charbuf[1024];//while(fgets(buf,sizeofbuf,stdin)){DelSpace(buf);fputs(buf,stdout);}//return0;}voidDelSpace(char*strp){char*sop=strp;char*dip=strp;//while(*sop){if(!isspace(*sop)){*dip++=*sop;}sop++;}*dip='\0';}~/test/ctst >

Comments

[FumiakiNakao]

回答ありがとうございました
NULLは*void型なのですね　勉強不足でした

サンプルプログラムも大変参考になりました

Answer 2

cateyeさんの回答が正鵠を射ていますが、補足。

fscanf関数は、scanf/printf系統に分類され、正常に入力された項目数(読み込みエラー時はEOF)を返します。

fgets関数は、gets/puts系統に分類され、入力された内容へのポインタ(エラー時はNULL)を返します。

要するに、似た目的で利用される関数ではあるが、返り値の仕様が違うわけです。昔からの伝統で規格化されたので、こういうもの、と割り切ってもらうしかありません。

本問では、条件式をwhile(fscanf(fp,"%s",str)>0)とすれば、正しく動作します。

以下蛇足：本問とは直接関係がないので、余裕があれば読んでください。

このプログラムには、あと2ヶ所、問題があります。
1つ目は、書き込み用ファイルが開けない場合の処理です。読み込み用ファイルが開いたままに、なりませんか?
2つ目は、長い単語入力があった場合です。もし101桁の数字が入力されたら、バッファオーバーフローが発生します。バッファオーバーフローは、時に深刻なセキュリティホールになります。修正方法ですが、while(fscanf(fp,"%99s",str)>0)と、fscanf関数の引数で桁数を指定すれば、この問題は回避されます。

Comments

[FumiakiNakao]

回答ありがとうございました
よくわかりました

その他の問題点についてもご指摘ありがとうございます
次回からはそのことにも気をつけてプログラムを組んでいこうと思います