C++ Make Node函数.S有什么问题吗

发布于01月18日

我创建了2个struct dl和Node.我使用struct dl来创建Node并创建了函数MakeNode.请帮帮我!谢谢你们看了.

typedef struct _dl {
    char ProblemID[10];
    char TimePoint[20];
    char Status[10];
    int Point;
} dl;

typedef struct _Node {
    dl x;
    struct _Node* leftChild;
    struct _Node* rightChild;
} Node;

void cpy(dl x, dl a) {
    strcpy(x.ProblemID, a.ProblemID);
    strcpy(x.TimePoint, a.TimePoint);
    strcpy(x.Status, a.Status);
    x.Point = a.Point;
    printf("%s %s %s %d\n",x.ProblemID,x.TimePoint,x.Status,x.Point);
}

Node* makeNode(dl a) {
    Node* p = (Node*)malloc(sizeof(Node));
    if (p == NULL) {
        exit(1);
    }
    cpy(p->x, a);
    printf("%s %s %s %d\n",p->x.ProblemID,p->x.TimePoint,p->x.Status,p->x.Point);
    p->leftChild = NULL;
    p->rightChild = NULL;
    return p;
}

你能指出我的错误吗？谢谢大家.

推荐答案

函数cpy处理传递的参数表达式的值的副本.因此，它实际上不改变所创建 node 的数据成员x.

相反，你应该至少写下

void cpy( dl *x, const dl *a) {
    strcpy(x->ProblemID, a->ProblemID);
    strcpy(x->TimePoint, a->TimePoint);
    strcpy(x->Status, a->Status);
    x->Point = a->Point;
    printf("%s %s %s %d\n",x->ProblemID,x->TimePoint,x->Status,x->Point);
}

并且函数cpy在makeNode内被调用，该函数应该声明如下

Node* makeNode( const dl *a);

以下列方式

cpy( &p->x, a);

请注意，如果删除带有exit的语句会更好，而函数可以返回空指针，并且函数的调用者可以判断返回值.例如

Node* makeNode( const dl *a ) 
{
    Node *p = malloc( sizeof( *p ) );

    if ( p != NULL )
    {
        cpy( &p->x, a );
        //printf("%s %s %s %d\n",p->x.ProblemID,p->x.TimePoint,p->x.Status,p->x.Point);
        p->leftChild  = NULL;
        p->rightChild = NULL;
    }

    return p;
}