冰逸琉璃 发表于 2021-4-1 10:07:25

通讯录程序

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

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

struct Papt
{
        char lxr;
        char dh;
        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 21:59 编辑

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

你的问题有点多

首先为什么不能显示
if (*px == NULL)
      {
                *px = papt->pt; // *px = papt 不是 = papt->pt
                papt->pt = NULL; // 这里是没有错的
      }
      else
      {
                struct Papt *py;
                py = *px;
                *px = papt->pt; // 同上
                papt->pt = py;
      }

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


要改的话很容易的，我感觉你可以自己改，如果感觉不行可以看一下
void xb(struct Papt* papt)
{
    struct Papt* temp = papt; // 想一下为什么不能直接用 papt

    while (temp != NULL) {
      printf("联系人:%s\n", temp->lxr, 128);
      printf("电话号码:%s\n", temp->dh, 128);
      temp = temp->pt;
    }

}

我建议不要用vs提供的函数，就用C语言提供的比较好，vs用scanf报错，可以上网查一下怎么解决
    printf("请输入联系人名字:");
    scanf_s("%s", papt->lxr, 128);
    printf("请输入电话:");
    scanf_s("%s", papt->dh, 128);

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

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

本帖最后由 yuxijian2020 于 2021-4-1 11:14 编辑

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

冰逸琉璃 发表于 2021-4-1 14:32:57

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


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

yuxijian2020 发表于 2021-4-1 16:29:07

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

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

冰逸琉璃 发表于 2021-4-5 19:45:43

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

首先为什么不能显示


嗯嗯，谢谢大佬，抱歉，这么久才来看
页: [1]
查看完整版本: 通讯录程序