プロが教えるわが家の防犯対策術!

下のプログラムはコマンドライン引数できまった形で入力した数を2分木にしてそれを表示させるc言語のプログラムです。
きまった形というのは
短縮形は [ 8 [ 7 2 5 ] [ 3 1 _ ] ]
短縮形でないのは [ 8 [ 7 [ 2 _ _ ] [ 5 _ _ ] ] [ 3 [ 1 _ _ ] _ ] ] ]
のような形で、実行結果はそれぞれ

a.exe [ 8 [ 7 2 5 ] [ 3 1 _ ] ]
入力データ [ 8 [ 7 2 5 ] [ 3 1 _ ] ]

a.exe [ 8 [ 7 [ 2 _ _ ] [ 5 _ _ ] ] [ 3 [ 1 _ _ ] _ ] ] ]
入力データ [ 8 [ 7 2 5 ] [ 3 1 _ ] ]

のようになります。
これをこのような形ではなくコマンドラインで 8 7 3 2 5 1 と入力するだけで2分木になるように
下のプログラムを変えたいのです。
よろしくお願いします。

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

typedef int BITREE_TYPE; /* 格納データの型 */

struct node {
BITREE_TYPE value; /* ノードの値 */
struct node *left; /* 左ノードのアドレス */
struct node *right; /* 右ノードのアドレス */
};
typedef struct node BITREE_NODE;

void error(char *msg);
BITREE_NODE *createNode(BITREE_TYPE x);
void destroyBITree(BITREE_NODE *p);
int isLeafNode(BITREE_NODE *p);
void printBITree(BITREE_NODE *p, int tabs, int brief);
BITREE_NODE *inputBITree(char *str[], int len, int *end);



int gShortFormat = 1; /*1ならば短縮形で出力する*/

void error(char *msg){
fflush(stdout);
fprintf(stderr,"%s\n", msg);
exit(1);
}

BITREE_NODE *createNode(BITREE_TYPE x){
BITREE_NODE *new;

new = malloc(sizeof(struct node));
if(new == NULL)
error("createNode: メモリがありません");
new->value = x;
new->left = NULL;
new->right = NULL;
return new;
}

void destroyBITree(BITREE_NODE *p){
if(p == NULL)
return;

destroyBITree(p->left);
destroyBITree(p->right);
memset(p, 0, sizeof(struct node));
free(p);
}

int isLeafNode(BITREE_NODE *p){
return(p->left == NULL) && (p->right == NULL);
}

void printSubtree(BITREE_NODE *p){
if(p == NULL){
printf("_");
return;
}
if(gShortFormat != 0 && isLeafNode(p)){
printf("%d", p->value);
}else{
printf("[ ");
printf("%d ", p->value);
printSubtree(p->left);
printf(" ");
printSubtree(p->right);
printf(" ]");
}
}

void printBITree(BITREE_NODE *p, int tabs, int brief){
int i;

gShortFormat = brief;
for (i = 0; i<tabs; i++)
printf("\t");
printSubtree(p);
printf("\n");fflush(stdout);
}

BITREE_NODE *inputBITree(char *str[], int len, int *end){
BITREE_NODE *p;
int i =0;

if(len < 1)
error("inputBITree:データがありません");

/*短縮形の処理*/
if(strcmp(str[0], "[") != 0){
if(strcmp(str[0], "_") == 0)
error("inputBTITree:値に_は指定できません");
*end =1;
return createNode(atoi(str[0]));
}

p = createNode(atoi(str[1]));

i = 2;
if(strcmp(str[i], "_") != 0){
p->left = inputBITree(&str[i], len -i, end);
i+= *end;
}else{
i++;
}

if(strcmp(str[i], "_") != 0){
p->right = inputBITree(&str[i], len -i, end);
i+= *end;
}else{
i++;
}

if(strcmp(str[i], "]") != 0)
error("inputBITree: 入力データが]で終わっていません");
*end = i + 1;
return p;
}


int main(int argc, char *argv[]){
BITREE_NODE *p;
int end = 0;

p = inputBITree(argv+1, argc-1, &end);
printf("入力データ ");
printBITree(p,0,1);

destroyBITree(p);
return 0;
}

A 回答 (3件)

8 7 3 2 5 1


という入力から, どのような 2分木を作るの?

この回答への補足

     8
   / \
  7     3
 / \  /
2    5 1
こんな感じに上から数列の順にしたいのです

補足日時:2011/07/10 01:12
    • good
    • 0
この回答へのお礼

捕捉よりもお礼に書くべきでした

お礼日時:2011/07/10 23:57

上から下 (はある意味当然だけど), 左から右に詰めていく, ということね?



ちょっと無駄なことはしてるけど, これでできないかなぁ:
1. まず必要なデータを設定した上で, right を使って連結リストを作る.
2. p と q を, それぞれその連結リストの先頭と 2番目の要素へのポインタとする.
3. p->left = q; q = q ? q->right : NULL;
4. r = p->right; p->right = q; p = r; q = q ? q->right : NULL;

3 と 4 を p != NULL の間繰り返せばいいような気がするけど.... ただ, イメージで書いただけなのでデバッガとお友達になることは覚悟してください.
    • good
    • 0
この回答へのお礼

ありがとうございます

自分で書いてみましたができませんでした…

お礼日時:2011/07/10 10:18

#2 には書いてるんだけど, こっちも試したわけじゃないんで「あれでできる」とは言いません.



でも, 「どうやってどう『できない』のか」がわからんと, #2 の方法が実は間違っているのか, それともあの方針それ自体は正しいのかの判断もできません.
    • good
    • 0
この回答へのお礼

自分の現状じゃできなさそうです

ありがとうございました

お礼日時:2011/07/10 23:55

お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!