C言語の構造体で2分木を作るプログラムの実行時エラーの解決策を教えてください。

Question

初心者なので不備が多いかと思いますが、とりあえず動くようにしたいです。
自分でいろいろいじってみましたがどうしてもわかりません。
どこをどう直せばいいのかどなたかご教授お願い致します。

###概要
英文が書かれたテキストファイルから単語（スペースで区切られた塊）
を切り出して2分木にソートして順番に表示するプログラム
###問題点
コンパイルは通るが、実行時に「動作を停止しました。」と出る。
デバッグすると、printTree()内のprintf()でアクセス違反が生じている。
###環境
windows10
Visual Studio 2013

###追記
お二方回答ありがとうございます。
fscanfで読み込んで、新しく型のエラーが生じたところを修正したところ、
一応例外は吐かなくなりました。残りは何とか考えますmm

###修正前コード

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

typedef char DATA;
struct node {
	DATA d;
	int count;
	struct node *left; // 左部分木
	struct node *right; // 右部分木
};
typedef struct node NODE;
typedef NODE *BTREE;
BTREE insertNode(BTREE, DATA*);
BTREE allocNode(DATA);
void printBtree(BTREE);
void freeBtree(BTREE);

int main(void){
	FILE *fp;
	BTREE root = NULL;
	DATA str[50];
	fp = fopen("./a.txt", "r");
	if (fp == NULL){
		printf("\aファイルをオープンできません。\n");
	}
	else {
		while (fgetc(fp) != EOF){
			root = insertNode(root, str);
		}
		printBtree(root);
		freeBtree(root);
		fclose(fp);
	}
	return 0;
}

BTREE insertNode(BTREE node, DATA *x) {
	BTREE  p = node;
	DATA dat = *x;
	if (p == NULL) {  // rootに挿入
		p = allocNode(dat);
		return  p;
	}
	if (dat == p->d)
	{
		p->count++;
	}
	else if (dat < p->d)
	{
		// 左部分木に割り当て
		p->left = insertNode(p->left, &dat);
		// 再帰呼出し
	}
	else  {
		// 右部分木に割り当て
		p->right = insertNode(p->right, &dat);
		// 再帰呼出し
	}
	return  p;
}

BTREE allocNode(DATA  x) {
	BTREE  p;
	p = (BTREE)malloc(sizeof(NODE));
	p->d = x;
	p->left = p->right = NULL;
	return  p;
}

void printBtree(BTREE p) {
	if (p != NULL) {
		printBtree(p->left);        // 左部分木の処理（再帰呼び出し）
		printf("WORD : %s COUNT : %d\n",  p->d, p->count);
		printBtree( p->right );     //  右部分木の処理（再帰呼び出し）
	}
	else printf("p==NULL\n");
	return;
}

void freeBtree(BTREE p) {
	if (p != NULL) {
		freeBtree(p->left);        // 左部分木の処理（再帰呼び出し）
		freeBtree( p->right );     //  右部分木の処理（再帰呼び出し）
		free( p );
	}
	return;
}

Answer 1

printfのフォーマット指定子が間違っているようです。

charを出力する際には%cをつかいます。
%sはchar*(NUL文字で終わる文字列)を出力する際に用いるものです。

Comments

[Breaststroke]

回答ありがとうございます!

Answer 2

ちょっと見ですが

        while (fgetc(fp) != EOF){
            root = insertNode(root, str);
        }

１．fgetc()で文字を読み込んでいない
２．この段階でstrのは何が入っているか分からない
３．strに文字列を格納するなら空白が出てくるまで読み込んで、文字列（終端'\0'）として渡す必要がある。
[追記]

struct node {
    DATA d; <-これは？これでいいのか？　文字列では？
    int count;
    struct node *left; // 左部分木
    struct node *right; // 右部分木
};

文字列と文字との処理が混在しているようですが？

Comments

[Breaststroke]

ありがとうございます！

[cateye]

私の方に勘違いがあったようで、ファイルから読み込んだ文字列中の文字の出現率をカウントしてるようですが・・・であればfgets()を使って一行丸ごと読んでから処理した方がいい気がします。この場合、空白もカウントしても（表示するときにカットするなどすれば）問題ないような気もします。

Answer 3

解決済ですがfgetcで作成してみました。

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

typedef char DATA;
struct node {
    DATA d[50];
    int count;
    struct node *left; // 左部分木
    struct node *right; // 右部分木
};
typedef struct node NODE;
typedef NODE *BTREE;
BTREE insertNode(BTREE, DATA *);
BTREE allocNode(DATA *);
void printBtree(BTREE);
void freeBtree(BTREE);

int main(void){
    FILE *fp;
    BTREE root  = NULL;
    BTREE rootS = NULL;
    char buf[50] = {0};
    char c;
    DATA *str = buf;
    int i = 0;
    fp = fopen("./a.txt", "r");
    if (fp == NULL){
        printf("\aファイルをオープンできません。\n");
    }
    else {
        while ((c=fgetc(fp)) != EOF){
          if ( c == '\n' || c == ' ' || c == '.' || c == ',' ){
            if ((strlen(str)) > 0){
                  root = insertNode(root, str);
               if ( rootS == NULL ) rootS=root;
            }
            i=0;
          }else{
            buf[i] = c;
            i++;
          }
          buf[i] = '\0';
        }

        printBtree(rootS);
        freeBtree(rootS);
        fclose(fp);
    }
    return 0;
}

BTREE insertNode(BTREE node, DATA *x) {
    BTREE  p = node;
    DATA *dat = x;
    if (p == NULL) {  // rootに挿入
        p = allocNode(dat);
        return  p;
    }
    if ((strcmp(dat,p->d)) == 0)
    {
        p->count++;
    }
    else if ((strcmp(dat,p->d)) < 0)
    {
        // 左部分木に割り当て
        p->left = insertNode(p->left, dat);
        // 再帰呼出し
    }
    else  {
        // 右部分木に割り当て
        p->right = insertNode(p->right, dat);
        // 再帰呼出し
    }
    return  p;
}

BTREE allocNode(DATA  *x) {
    BTREE  p;
    p = (BTREE)malloc(sizeof(NODE));
    strcpy(p->d, x);
    p->count=1;
    p->left = p->right = NULL;
    return  p;
}

void printBtree(BTREE p) {
    if (p != NULL) {
        printBtree(p->left);        // 左部分木の処理（再帰呼び出し）
        printf("WORD : %-14s COUNT : %d\n",  p->d, p->count);
        printBtree( p->right );     //  右部分木の処理（再帰呼び出し）
    }
    return;
}

void freeBtree(BTREE p) {
    if (p != NULL) {
        freeBtree(p->left);        // 左部分木の処理（再帰呼び出し）
        freeBtree( p->right );     //  右部分木の処理（再帰呼び出し）
        free( p );
    }
    return;
}