C言語の正規表現を使った関数でエラーが出てしまいます。

Question

以下のプログラムはテキストを読み込んでその中からメールアドレスのある行だけは出力するプログラムなのですが、何度確かめてもエラーが起きてしまいます。

当該箇所をお教え願います。

ソースコード

include <stdio.h>

include <stdlib.h>

include <dirent.h>

include <sys/types.h>

include <regex.h>

include <string.h>

const char *regex = "(^| )(\-|[a-z]|[A-Z]|[0-9]|\_|\.)+\@(a-z+|[0-9]?[0-9]?[0-9]?\.[0-9]?[0-9]?[0-9]?\.[0-9]?[0-9]?[0-9]?\.[0-9]?[0-9]?[0-9]?)( |$)";

int main(int ac, char *av[]) {
DIR *dir;

struct dirent *dp;

if ( (dir = opendir(av[1])) == NULL ) {
	perror("opendir");
	exit(EXIT_FAILURE);
}
regex_t preg;

if ( regcomp(&preg, regex, REG_EXTENDED|REG_NEWLINE) != 0 ) {
	perror("regex compile failed.");
	exit(EXIT_FAILURE);
}	

while ( dp = readdir(dir) ) {
	if ( dp->d_type & DT_REG )
		printf("%s\n", dp->d_name);

char	name[256];

strcpy(name,av[1]);
strcat(name, "/");
strcat(name, dp->d_name);

FILE *fp;
char buf[256];

if ( (fp = fopen(name, "r")) == NULL ) {
	perror("file open error");
	exit(EXIT_FAILURE);
}

int gyo;
int count = 0;
gyo = 1;

while ( fgets(buf, 256, fp) != NULL ) {
	if (regexec(&preg, buf, 0, NULL, 0) == 0)
		printf("%s:%d:%s\n", dp->d_name, gyo, buf);
		count++;
	}
gyo++;
}
printf("%sのメアドの数：%d\n", dp->d_name, count);

fclose(fp);

regfree(&preg);

closedir(dir);

return 0;

}

エラー

getMailAddress.c: 関数 ‘main’ 内:
getMailAddress.c:56:52: エラー: ‘count’ が宣言されていません (この関数内での最初の使用)
getMailAddress.c:56:52: 備考: 未宣言の識別子は出現した各関数内で一回のみ報告されます
getMailAddress.c:58:9: エラー: ‘fp’ が宣言されていません (この関数内での最初の使用)

Comments

[PineMatsu]

プログラムコードは```(shift+@)で囲うと見やすくなります。

[PineMatsu]

初心者マークをつけてください。

Answer 1

' {'と'}'の対応を見直しましょう。
例えば下記はどれが正解？

while ( fgets(buf, 256, fp) != NULL ) {
if (regexec(&preg, buf, 0, NULL, 0) == 0)
printf("%s:%d:%s\n", dp->d_name, gyo, buf);
count++;
} ←これの対応は？
gyo++;
} 

ソースを投稿する場合は整形しましょうｄ＾＾

Answer 2

変数のスコープの問題です。

  while ( dp = readdir(dir) ) {
    // ・
    // ・
    FILE *fp;
    char buf[256];
    // ・
    // ・
    int count = 0; 
    // ・
  }
  printf("%sのメアドの数：%d\n", dp->d_name, count); // <- スコープ外参照

  fclose(fp);  // <- スコープ外参照

スコープ外参照です。

Answer 3

getMailAddress.c:56:52: エラー: ‘count’ が宣言されていません (この関数内での最初の使用)
getMailAddress.c:58:9: エラー: ‘fp’ が宣言されていません (この関数内での最初の使用)

どちらの変数もwhile文の中で宣言されています。
ループの中で宣言した変数はそのループ内がスコープの範囲になります。
http://www7b.biglobe.ne.jp/~robe/cpphtml/html01/cpp01068.html

Answer 4

エラーが出ている変数のスコープを確認してください。

Answer 5

C言語ですよね。
変数の宣言は、関数（またはブロック{ }）の先頭で行う必要があります。
C++であれば、使用する前であればどの位置でも宣言は可能です。

Comments

[ttyp03]

それに関してはエラーが出ていないのでC99規格のコンパイラなのかと思います。

[PineMatsu]

何のコンパイラを使ってるのかを聞こうかなと思いましたが、たぶん初心者なのでわからないだろうと思いまして・・・。
それと、エラーが出てないからいい、というのではなくて、このソースをC99規格でないコンパイラーでコンパイルする可能性もあるので、変数はブロックの先頭で宣言する癖をつけておいたほうが良いのではないかと私は思います。

投稿者は、スコープを理解してないので、そこから勉強しなおしですね。

[ttyp03]

エラーが出てないからいいと言っているわけではなく、この質問に関してはそこは言及しなくていいのでは、ということ。
そうじゃないとエラーが出てないのになんで？ということになって余計混乱しそうですが。
まあ、いずれにしてもスコープの理解は必要ですね。