论坛 › 新手乐园 › 带你学C带你飞p47第一题回文链表

wyxnogiveup 发表于 2023-1-2 17:59:59

带你学C带你飞p47第一题回文链表

我真的崩溃了，大佬们告诉我这个程序哪里出问题了，它没报错但在DEV C++上就是不能运行
#include <stdio.h>
#include <stdlib.h>
struct Node
{
        int num;
        struct Node *next;
};

void touchafa(struct Node **head1,int input);
void touchafa(struct Node **head1,int input)
{
       
        struct Node *temp;
       
        struct Node *p=(struct Node *)malloc(sizeof(struct Node));
        p->num=input;
        if(*head1==NULL)
        {
                *head1=p;
                p->next=NULL;
        }
        else
        {
                temp=*head1;
                *head1=p;
                p->next=temp;       
        }
}

void weichafa(struct Node **head2,int input);
void weichafa(struct Node **head2,int input)
{
        static struct Node *store;
        struct Node *p=(struct Node *)malloc(sizeof(struct Node));
        p->num=input;
        if(*head2==NULL)
        {
                *head2=p;
                p->next=NULL;
        }
        else
        {
                store->next=p;
                p->next=NULL;
        }
        store=p;
}

void printnum(struct Node *head);
void printnum(struct Node *head)
{
        struct Node *p=head;
        while(p!=NULL)
        {
                printf("%d ",p->num);
                p=p->next;
        }
        putchar('\n');
}


int main()
{
        int input;struct Node *head1;struct Node *head2;
        printf("下面开始录入链表，录入完毕后将检查这是否是回文链表（即顺着倒着读结果一样的链表）:\n ");
        while(1)
        {
                printf("请输入一个整数(输入-1表示结束)：\n");
                scanf("%d",&input);
                if(input== -1)
                {
                        break;
                }
               
                else
                {
                        touchafa(&head1,input);
                        weichafa(&head2,input);
                }       
        }
        printf("录入的链表是 ");
        printnum(head2);

        while(head1!=NULL)
        {
                        if(head1->num==head2->num)
                        {
                                head1=head1->next;
                                head2=head2->next;
                        }
                        else
                        {
                                break;
                        }
        }
        if(head1==NULL&&head2==NULL)
        {
                printf("经检查，该链表确实是回文链表！\n");
        }
        else
        {
                printf("一眼就看出，这不是回文链表！\n");
        }
        return 0;
       
       
}

dolly_yos2 发表于 2023-1-2 18:28:26

head1 和 head2 需不需要初始化？

wyxnogiveup 发表于 2023-1-3 12:27:58

本帖最后由 wyxnogiveup 于 2023-1-3 18:05 编辑

dolly_yos2 发表于 2023-1-2 18:28
head1 和 head2 需不需要初始化？

试过了，没用，它是在尾插法函数死掉的



来自六小时后的回复：呜呜呜确实是初始化的问题，我太粗心了

人造人 发表于 2023-1-3 13:20:10

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

struct Node {
    int num;
    struct Node *next;
};

//void touchafa(struct Node **head1, int input);      需要这一行吗？为什么？
void touchafa(struct Node **head1, int input) {
    //struct Node *temp;

    //struct Node *p = (struct Node *)malloc(sizeof(struct Node));
    struct Node *p = malloc(sizeof(struct Node));       // 为什么要强制转换呢？少写几个字符不好吗？
    p->num = input;
    /*
    if(*head1 == NULL) {
      *head1 = p;
      p->next = NULL;
    } else {
      temp = *head1;
      *head1 = p;
      p->next = temp;
    }
    */
    p->next = *head1;
    *head1 = p;
}

//void weichafa(struct Node **head2, int input);      // 你在做什么？
void weichafa(struct Node **head2, int input) {
    //static struct Node *store;    // 这么写，用的时候不方便
    static struct Node **store;
    //struct Node *p = (struct Node *)malloc(sizeof(struct Node));
    //struct Node *p = malloc(sizeof(struct Node));   // 少写上几个字符吧，写那么多干嘛？
    struct Node *p = malloc(sizeof(*p));                // 一个更好的写法
                                                      // 这样写是为了在修改p的类型之后不需要修改sizeof里面的类型
                                                      // 少改一个地方就少一个出错的可能
    p->num = input;
    p->next = NULL;
    /*
    if(*head2 == NULL) {
      *head2 = p;
      p->next = NULL;
    } else {
      store->next = p;
      p->next = NULL;
    }
    store = p;
    */
    if(!store) store = head2;
    *store = p;
    store = &p->next;
}

//void printnum(struct Node *head);   // 同上
void printnum(struct Node *head) {
    struct Node *p = head;
    while(p != NULL) {
      printf("%d ", p->num);
      p = p->next;
    }
    putchar('\n');
}

void free_list(struct Node *head) {
    if(!head) return;
    free_list(head->next);
    free(head);
}

int main() {
    int input;
    //struct Node *head1;
    //struct Node *head2;
    // 你不初始化吗？为什么？
    struct Node *head1 = NULL;
    struct Node *head2 = NULL;
    printf("下面开始录入链表，录入完毕后将检查这是否是回文链表（即顺着倒着读结果一样的链表）:\n ");
    while(1) {
      printf("请输入一个整数(输入-1表示结束)：\n");
      scanf("%d", &input);
      /*
      if(input == -1) {
            break;
      } else {
            touchafa(&head1, input);
            weichafa(&head2, input);
      }
      */
      // 等价修改
      // else 什么？
      // 都跳出去了还else ？
      if(input == -1) break;
      touchafa(&head1, input);
      weichafa(&head2, input);
    }
    printf("录入的链表是 ");
    printnum(head2);

    // 你这么写，你还释放内存不了？
    // 申请了内存不释放就和借了东西不还一样恶心
    // 都像你这么写程序，别说16GB，就是16EB的内存也不够你用呀
    // 你这一直借东西不还，之后人家就不借给你了
    // 系统哪有那么多内存给你这么整？
    // 一直申请内存不释放，系统也没内存了，还怎么分配给你？
    /*
    while(head1 != NULL) {
      if(head1->num == head2->num) {
            head1 = head1->next;
            head2 = head2->next;
      } else {
            break;
      }
    }
    if(head1 == NULL && head2 == NULL) {
      printf("经检查，该链表确实是回文链表！\n");
    } else {
      printf("一眼就看出，这不是回文链表！\n");
    }
    */
    struct Node *t1 = head1;
    struct Node *t2 = head2;
    while(t1 != NULL && t2 != NULL) {
      if(t1->num != t2->num) break;
      t1 = t1->next;
      t2 = t2->next;
    }
    if(t1 == NULL && t2 == NULL) {
      printf("经检查，该链表确实是回文链表！\n");
    } else {
      printf("一眼就看出，这不是回文链表！\n");
    }

    // 释放内存，不要借了不还，这样很不礼貌
    free_list(head2);
    free_list(head1);
    return 0;
}

wyxnogiveup 发表于 2023-1-3 18:00:58

dolly_yos2 发表于 2023-1-2 18:28
head1 和 head2 需不需要初始化？

呜呜呜，确实是没有初始化的问题，感谢指正

wyxnogiveup 发表于 2023-1-3 18:03:56

人造人 发表于 2023-1-3 13:20


太谢谢大佬了！呜呜呜我以后肯定会牢记要释放内存的！那个将sizeof(struct Node)改为sizeof(*p）同样很巧妙，原来还能这样。真的谢谢大佬，细致地帮我指出问题！萌新一定好好改正陋习，不负教导！

查看完整版本: 带你学C带你飞p47第一题回文链表