[已解决]通讯录程序

冰逸琉璃

我在实现显示当前通讯录这个功能的时候打印不出来，不知道为什么，有没有大佬帮忙看看

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

struct Papt
{
        char lxr[128];
        char dh[128];
        struct Papt *pt;
};

void xa(struct Papt **px);
void xb(struct Papt *papt);

void xa(struct Papt **px)
{
        struct Papt *papt;

        papt = (struct Papt *)malloc(sizeof(struct Papt));
        if (papt == NULL)
        {
                printf("内存分配失败！");
        }

        printf("请输入联系人名字:");
        scanf_s("%s", papt->lxr, 128);
        printf("请输入电话:");
        scanf_s("%s", papt->dh, 128);

        if (*px == NULL)
        {
                *px = papt->pt;
                papt->pt = NULL;
        }
        else
        {
                struct Papt *py;
                py = *px;
                *px = papt->pt;
                papt->pt = py;
        }
}

void xb(struct Papt *papt)
{
        printf("联系人:%s\n", papt->lxr,128);
        printf("电话号码:%s\n", papt->dh,128);
}

int main(void)
{
        printf("| 欢迎使用通讯录管理程序 |\n");
        printf("|--- 插入新的联系人:1 ---|\n");
        printf("|--- 查找已有联系人:2 ---|\n");
        printf("|--- 更改已有联系人:3 ---|\n");
        printf("|--- 删除已有联系人:4 ---|\n");
        printf("|--- 显示当前通讯录:5 ---|\n");
        printf("|--- 退出通讯录程序:6 ---|\n");
        printf("\n");

        struct Papt *px = NULL;

        int zl;

        while (1)
        {
                printf("请输入指令:");
                scanf_s("%d", &zl);

                if (zl == 1)
                {
                        xa(&px);
                }
                else if (zl == 5)
                {
                        xb(px);
                }
                else if (zl == 6)
                {
                        printf("程序结束\n");
                        break;
                }
        }
        return 0;
}

最佳答案

月排行榜 / 总排行榜

李京

2021-4-1 10:07:26

冰逸琉璃 发表于 2021-4-1 14:32
我是这样想的
if，如果这个链表是空链表(也是头指针指向NULL)，那么头指针指向新增加的节点，再把这个新 ...

你的问题有点多

首先为什么不能显示

1.   if (*px == NULL)
   
2.         {
   
3.                 *px = papt->pt; // *px = papt 不是 = papt->pt
   
4.                 papt->pt = NULL; // 这里是没有错的
   
5.         }
   
6.         else
   
7.         {
   
8.                 struct Papt *py;
   
9.                 py = *px;
   
10.                 *px = papt->pt; // 同上
    
11.                 papt->pt = py;
    
12.         }

复制代码

然后你的显示也只是显示了最后一个输入的
你用的是头插法，所以显示最后一个输入的


要改的话很容易的，我感觉你可以自己改，如果感觉不行可以看一下

1. void xb(struct Papt* papt)
   
2. {
   
3.     struct Papt* temp = papt; // 想一下为什么不能直接用 papt
   
4. 
   
5.     while (temp != NULL) {
   
6.         printf("联系人:%s\n", temp->lxr, 128);
   
7.         printf("电话号码:%s\n", temp->dh, 128);
   
8.         temp = temp->pt;
   
9.     }
   
10. 
    
11. }

复制代码

我建议不要用vs提供的函数，就用C语言提供的比较好，vs用scanf报错，可以上网查一下怎么解决

1.     printf("请输入联系人名字:");
   
2.     scanf_s("%s", papt->lxr, 128);
   
3.     printf("请输入电话:");
   
4.     scanf_s("%s", papt->dh, 128);

复制代码

还有你最后没有释放堆上面申请的空间，这是很不好的习惯

跳转到最佳答案楼层

李京

冰逸琉璃 发表于 2021-4-1 14:32
我是这样想的
if，如果这个链表是空链表(也是头指针指向NULL)，那么头指针指向新增加的节点，再把这个新 ...

你的问题有点多

首先为什么不能显示

1.   if (*px == NULL)
   
2.         {
   
3.                 *px = papt->pt; // *px = papt 不是 = papt->pt
   
4.                 papt->pt = NULL; // 这里是没有错的
   
5.         }
   
6.         else
   
7.         {
   
8.                 struct Papt *py;
   
9.                 py = *px;
   
10.                 *px = papt->pt; // 同上
    
11.                 papt->pt = py;
    
12.         }

复制代码

然后你的显示也只是显示了最后一个输入的
你用的是头插法，所以显示最后一个输入的


要改的话很容易的，我感觉你可以自己改，如果感觉不行可以看一下

1. void xb(struct Papt* papt)
   
2. {
   
3.     struct Papt* temp = papt; // 想一下为什么不能直接用 papt
   
4. 
   
5.     while (temp != NULL) {
   
6.         printf("联系人:%s\n", temp->lxr, 128);
   
7.         printf("电话号码:%s\n", temp->dh, 128);
   
8.         temp = temp->pt;
   
9.     }
   
10. 
    
11. }

复制代码

我建议不要用vs提供的函数，就用C语言提供的比较好，vs用scanf报错，可以上网查一下怎么解决

1.     printf("请输入联系人名字:");
   
2.     scanf_s("%s", papt->lxr, 128);
   
3.     printf("请输入电话:");
   
4.     scanf_s("%s", papt->dh, 128);

复制代码

还有你最后没有释放堆上面申请的空间，这是很不好的习惯

yuxijian2020

1. if (*px == NULL)
   
2.         {
   
3.                 *px = papt->pt;         //这里看不懂你要干什么   papt->pt都没初始化你为什么要赋值给传入的参数
   
4.                 papt->pt = NULL;       //这两句完事   你啥也没干成
   
5.         }
   
6.         else
   
7.         {
   
8.                 struct Papt *py;         //这里逻辑就更乱了  不知道你要干什么
   
9.                 py = *px;
   
10.                 *px = papt->pt;
    
11.                 papt->pt = py;
    
12.         }

复制代码

冰逸琉璃

yuxijian2020 发表于 2021-4-1 11:11

我是这样想的
if，如果这个链表是空链表(也是头指针指向NULL)，那么头指针指向新增加的节点，再把这个新增加的节点指向NULL。
else，如果这个链表里已经有数据了，那么先定义一个临时指针，来存放头指针指向的地址，然后把头指针指向新增加的节点，新增加的节点再指向这个临时指针。

yuxijian2020

冰逸琉璃 发表于 2021-4-1 14:32
我是这样想的
if，如果这个链表是空链表(也是头指针指向NULL)，那么头指针指向新增加的节点，再把这个新 ...

你要理清楚逻辑
if,是空链表,则头指针 = 当前创建的指针
else,非空链表,则,创建临时指针遍历链表,指向当前链表的尾结点,尾结点->next = 当前创建的指针

冰逸琉璃

李京 发表于 2021-4-1 16:01
你的问题有点多

首先为什么不能显示

嗯嗯，谢谢大佬，抱歉，这么久才来看