c言語 文字数のカウント 合わない

こんにちは．

c言語のプログラムを書いて疑問に思ったことがあるのでお尋ねします．以下のプログラムで，data.txtに書かれている文字を読み取り，総文字数，スペース，タブ，ニューラインの数をカウントしようと思いました．スペース，タブ，ニューラインは正しくカウントするのですが，総文字数char_counterの数が合いません．ファイルから文字を読み取る時に使った，loop_counterもchar_counterと同義だと思いloop_counterを表示させてみたところ，こちらは文字数を正しくカウントしているようです．
I am checking
how it works.
という文字列が入ったファイルなのですが，char_counterは100文字を越えてしまいます．どうしてloop_counterとchar_counterで違う値が出てしまうのでしょうか？解答をお待ちしております．その他正しく動いている部分に関しても，変な書き方のところがあればそれも指摘して頂ければ嬉しいです．

なお，原因解明のために試行錯誤していて，ファイルの読み取りでは配列の[]の中身を足していき，文字を比較する際にはポインタの値を足していくという変なプログラムになっています．ご了承下さい．


#include <stdio.h>
#include <stdlib.h>
#define BUF_SIZE (256)
#define EXIT_FAILURE (1)

int main()
{
FILE *fp;
char read_line[BUF_SIZE];
int loop_counter = 0;

fp = fopen("data.txt", "r");
if(fp == NULL)
{
printf("file open error\n");
exit(EXIT_FAILURE);
}


while((read_line[loop_counter] = getc(fp)) != EOF)
{
loop_counter++;
}

read_line[loop_counter] = '\0';
fclose(fp);
if(fp == NULL)
{
printf("file close error\n");
exit(EXIT_FAILURE);
}

/* this counts the whole char numbers including space*/
int char_counter = 0;
int space_counter = 0;
int tab_counter = 0;
int newline_counter = 0;
char *read_line_address;
read_line_address = &read_line;
while(*read_line_address != EOF)
{
char_counter++;
switch(*read_line_address)
{
case ' ':
space_counter++;
break;
case '\t':
tab_counter++;
break;
case '\n':
newline_counter++;
break;
default:
break;
}
read_line_address++;
}
printf("%s\n", read_line);
printf("space %d ", space_counter);
printf("tab %d ", tab_counter);
printf("newline %d ", newline_counter);
/* printf("whole chars %d\n", char_counter); */
printf("whole chars %d\n", loop_counter);

return 0;
}

No.4 回答者： Tacosan 回答日時： 2013/08/23 16:49

さらに追加すると

read_line_address = &read_line;
は両辺の型があっていない. それと,
fclose(fp);
if(fp == NULL)
{
printf("file close error\n");
exit(EXIT_FAILURE);
}
の if は何も役に立たない (ここに来たとき fp の値が NULL であることはあり得ないので).

No.3 ベストアンサー 回答者： Tacosan 回答日時： 2013/08/23 12:29

余談:

read_line[loop_counter] = getc(fp)
は典型的な「やってはいけない」パターンだし,
#define EXIT_FAILURE (1)
の必要性が理解できない (これ, 未定義動作にならないか?).

あと, EXIT_FAILURE を使うなら EXIT_SUCCESS を使ってあげてもいいんじゃないかなと思う今日この頃.

No.2 回答者： tsunji 回答日時： 2013/08/23 10:09

print文で1文字ずつ、配列の内容（文字と16進値）とカウンタを表示してみれば？

No.1 回答者： episteme 回答日時： 2013/08/23 06:30

> while(*read_line_address != EOF)

文字列の末尾にあるのはなんですか? EOFですか