プログラマ専用SNS ミクプラ

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

int data[256];

void countLtr(FILE *fp);

int main(void)
{
	FILE *fp;
	int c;

	//強制終了
	if ((fp = fopen("test.txt", "r")) == NULL) {
		fprintf ( stderr, "Can't open.\n" );
		exit (2);
	}
	printf("test.txt\n");
	
	countLtr(fp) ;

	printf("---------\n");
		for (c = 0; c <= 127; c++){
		printf("%c:%d\t",  c, data[c] );
	}
	
	fclose(fp);
	
	return 0;
}
	
//ファイル中の英文字の出現頻度を調べる関数
void countLtr(FILE *fp)
{
	int c;

	while ((c = getc(fp)) != EOF)
		if (c >= 0 && c <= 127){
			data[c]++;
		}
}

[color=#d0d0ff" face="monospace]
            if(isprint(c))
                printf(" %2c : %d\t",  c, data[c] );
            else
                printf("[%02x]: %d\t", c, data[c] );[/color]

printf("%c:%d\t",  c, data[c] );

から

        if(isprint(c))
            printf(" %2c : %d\t",  c, data[c] );
        else
            printf("[%02x]: %d\t", c, data[c] );
        if(c%8==7)
            printf("\n");

に変えてやることで、正常にかつきれいに表示されるでしょう。

> 後、配列の中身が初期化されていない

data[/url]は関数の外側にあるので、明示的な初期化は特にいりません。
仮に明示的な初期化を行なう場合は、

> int data[256]={};

ではなく（コンパイル時に「初期化子がない」のようなエラーが出ませんか？）、

 int data[256]={ 0 };

です。

/*sample1*/

#include <stdio.h>

void main(void){
	int a[5];
	printf("%d",a[0]);
}

実行結果

4214511


/*sample2*/

#include <stdio.h>

void main(void){
	int a[5]={};
	printf("%d",a[0]);
}

実行結果

0

> 	int a[5]={};

当方の環境（Windows2000, BCC5.5.1）では
コンパイル時に「式が必要」のエラーが出ます。

ほぼすべてのコンパイラを通すには、初期化するための
値を明記する方がよさそうです。