设为首页收藏本站
切换到窄版

鱼C论坛

 找回密码
 立即注册
鱼C论坛»论坛 › 技术交流 › 新手乐园 › 带你学C带你飞p47第一题回文链表
返回列表 发新帖
查看: 2085|回复: 5

[已解决]带你学C带你飞p47第一题回文链表

[复制链接]
wyxnogiveup
发表于 2023-1-2 17:59:59 | 显示全部楼层 |阅读模式

马上注册，结交更多好友，享用更多功能^_^

您需要 登录 才可以下载或查看，没有账号？立即注册

x
我真的崩溃了，大佬们告诉我这个程序哪里出问题了，它没报错但在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;
        
         
}
最佳答案
月排行榜 / 总排行榜
人造人
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;
}
想知道小甲鱼最近在做啥？请访问 -> ilovefishc.com
回复

使用道具 举报

dolly_yos2
发表于 2023-1-2 18:28:26 From FishC Mobile | 显示全部楼层
head1 和 head2 需不需要初始化？
想知道小甲鱼最近在做啥？请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

wyxnogiveup
 楼主| 发表于 2023-1-3 12:27:58 From FishC Mobile | 显示全部楼层
本帖最后由 wyxnogiveup 于 2023-1-3 18:05 编辑
dolly_yos2 发表于 2023-1-2 18:28
head1 和 head2 需不需要初始化？


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



来自六小时后的回复：呜呜呜确实是初始化的问题，我太粗心了
想知道小甲鱼最近在做啥？请访问 -> ilovefishc.com
回复 支持 反对

使用道具 举报

人造人
发表于 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;
}
想知道小甲鱼最近在做啥？请访问 -> ilovefishc.com
回复 支持 1 反对 0

使用道具 举报

wyxnogiveup
 楼主| 发表于 2023-1-3 18:00:58 | 显示全部楼层
dolly_yos2 发表于 2023-1-2 18:28
head1 和 head2 需不需要初始化？

呜呜呜，确实是没有初始化的问题，感谢指正
想知道小甲鱼最近在做啥？请访问 -> ilovefishc.com
回复 支持 1 反对 0

使用道具 举报

wyxnogiveup
 楼主| 发表于 2023-1-3 18:03:56 | 显示全部楼层
人造人 发表于 2023-1-3 13:20

太谢谢大佬了！呜呜呜我以后肯定会牢记要释放内存的！那个将sizeof(struct Node)改为sizeof(*p）同样很巧妙，原来还能这样。真的谢谢大佬，细致地帮我指出问题！萌新一定好好改正陋习，不负教导！
想知道小甲鱼最近在做啥？请访问 -> ilovefishc.com
回复 支持 1 反对 0

使用道具 举报

返回列表 发新帖
高级模式
B Color Image Link Quote Code Smilies
您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

小黑屋|手机版|Archiver|鱼C工作室 ( 粤ICP备18085999号-1 | 粤公网安备 44051102000585号)

GMT+8, 2024-12-24 21:28

Powered by Discuz! X3.4

© 2001-2023 Discuz! Team.

快速回复 返回顶部 返回列表