C言語を使ってファイル内の特定文字列のみを除いて表示したい

Question

###前提・実現したいこと
C言語に関して、strstr関数を用いてテキストファイルの内容を特定文字列(各行の最後にあるとします)を除いて標準出力したいと考えています。
例:sample.txt

HOSTNAME = HOST
ipaddres = 192.168.0.2
subnet mask = 255.255.255.0
default gateway = 192.168.0.1
DNS = 8.8.8.8

から192.168.0.2を非表示にして

HOSTNAME = HOST
ipaddres = 
subnet mask = 255.255.255.0
default gateway = 192.168.0.1
DNS = 8.8.8.8

の様に出力させたいということです。

###ソースコード

#include<stdio.h>
#include <string.h>
int main(){
    FILE *fp;
    char *p;
    char a[10000];
    p = a;
    if ((fp = fopen("sample.txt", "r"))==NULL) {
        fprintf(stderr,"ファイルを開くのに失敗しました\n");
        exit(1);
    }
    while(fgets(p,10000,fp)!= NULL){
        if(NULL != strstr(p,"192.168.0.2")){
            p = strstr(p,"192.168.0.2");
            *p = '\n';
            *p++;
        }
    }
    fprintf(stdout,"%s\n",a);
    fclose(fp);
    return 0;
}

1行ずつ読み込み、それを配列に保存
その行に消したい文字列があればその先頭を改行して、配列の次の要素からファイル内容を読み込むようにする
をループして、全て読み込んだら配列を出力する。
という動作をするようにコーディングしてみました。

###実行結果

ipaddres =
DNS = 8.8.8.8

となり、最終行と該当文字列がある行しか表示されません。
どのように改変すれば欲しい結果が得られるでしょうか。
アドバイスよろしくお願いします。

Answer 1

対象のIPアドレスがないときは
if(NULL != strstr(p,"192.168.0.2")){
の中に入らないので、ポインタpが進まないからではないでしょうか。
同じ位置に読み込んでいることになるので、前回読み込んだ文字列を上書きしているからだと思います。

上記の条件に該当しないときは、pが指している文字列の最後までポインタを進めれば、読み込んだ文字列がバッファに追加されていくと思います。

Answer 2

ファイルを全部読み取ってから、一括出力というやり方もできますが、1行ずつ読み取って1行ずつ出力がわかりやすいでしょう。

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


int main()
{
    char exclusion[] = "192.168.0.2";
    FILE *fp;
    char buffer[10000];
    char *found_exclusion_at;

    if ((fp = fopen("sample.txt", "r")) == NULL) {
        fprintf(stderr,"ファイルを開くのに失敗しました\n");
        exit(1);
    }

    while (fgets(buffer, sizeof(buffer), fp) != NULL) {
        if (NULL != (found_exclusion_at = strstr(buffer, exclusion))) {
            found_exclusion_at[0] = '\n';
            found_exclusion_at[1] = '\0';
        }
        fputs(buffer, stdout);
    }

    fclose(fp);
    return 0;
}

Comments

[KEILA]

回答ありがとうございます。
Chironianさんのアドバイスをもとに自分なりにコードは書いたつもりでしたが、より洗練されたコードを教えていただき勉強になりました。
ただ、自分の不勉強からくる疑問なのですが

char *found_exclusion_at;

と定義したポインタを配列のように扱っておられますが、このような場合found_exclusion_at[1]が既に使用されていてアドレスの重複などが起きる危険性はないのでしょうか。

[naomi3]

すみません。「fgets(buffer, sizeof(buffer) - 1, fp)」としてください。そうすれば、「found_exclusion_at
+ 0」および「found_exclusion_at + 1」は、「buffer + 0」～「buffer + 9999」のいずれかです。したがって、アドレスの重複などが起きることはありません。ポインタを配列のように扱うことはC言語では良くあることです。

[KEILA]

回答ありがとうございます。
>ポインタを配列のように扱うことはC言語では良くあることです。
そうなのですか、勉強になりました。
重ね重ねどうもありがとうございました。

Answer 3

こんにちは。

細かくは見ていないのですが、気になった点があります。
fgets(p,10000,fp)のpはこのwhile文の中で更新し、バッファaの先頭を指していません。結果、メモリは10,000バイト確保できていないため、メモリ破壊が発生する可能性があります。
しかし、全部で10,000バイトもないのでこれは原因ではないでしょう。

あまり難しいことをしないで、１行づつ処理した方が簡単です。
「１行入力して、該当の文字列があったらそこを削除した行を出力、該当の文字列がなければそのまま出力」を繰り返しては如何でしょうか？

Comments

[KEILA]

回答していただきありがとうございます。

ご指摘いただいたように、１行ごとに入力から出力までの処理をするようにしてみたところ無事目的の結果を得ることがが出来ました。
ありがとうございました。