关于链表，小弟我再一次遇到恶心有关问题了

关于链表，我再一次遇到恶心问题了
为什么每次都不能正常删除元素啊？

typedef struct LNode{
    int data;
    LNode *next;
}*LinkList;

void ListDelete(LinkList &L,int i,int e)
{
    LinkList p = L;
    int j = 0;
    while (p->next && j<i-1) {//每次都会停在这，说是EXC_BAD_ACCESS，这什么问题啊？？
        p = p->next;
        ++j;
    }
    
    if(!p->next&&j>i-1){cout<<"error3.."<<endl;}
    LinkList q = p->next;
    p->next = q->next;
    e = q->data;
    free(q);
}
//创建链表没什么问题，亲测可用，问题应该出在删除的函数那儿
void CreateList(LinkList &L,int n)
{
    L = (LinkList)malloc(sizeof(LNode));
    L->next = NULL;
    for (int i = n; i>0; --i) {
        LinkList p = (LinkList)malloc(sizeof(LNode));
        scanf("%d",&p->data);
        p->next = L->next;
        L->next = p;
    }
}
//打印也没问题
void PrintList(LinkList &L)
{
    L = L->next;
    for (int i = 0; L; ++i) {
        cout<< L->data<<" "<<flush;
        L = L->next;
    }
    cout<<endl;
}

int main(int argc, const char * argv[])
{    
    LinkList listA;
    CreateList(listA, 4);
    PrintList(listA);

    
    int e = 0;
    ListDelete(listA, 1, e);
    PrintList(listA);
    
    
    return 0;
}

运行每次打印第一遍之后，到删除那就中断了
链表

删除元素
[解决办法]
内存访问越界了吧
[解决办法]
是你的创建链表的函数出了问题，需要在创建链表的for循环后面对链表尾的->next = NULL;才行
[解决办法]
仅供参考

//假设带表头结点的单向链表头指针为head，试编写一个算法将值为5的结点插入到连接表的第k个结点前，删除第k个节点，并对该链表进行排序。
#include <stdio.h>
#include <stdlib.h>
#include <malloc.h>
#include <time.h>
struct NODE {
    int          data;
    struct NODE *next;
} H,*head,*p,*q,*s1,*s2,*s3,*s4,*s;
int i,j,k,n,t;
int main() {
    srand(time(NULL));

    //填写头节点数据
    H.data=-1;
    H.next=NULL;
    head=&H;

    //创建10个节点的单链表
    p=head;
    for (i=0;i<10;i++) {
        q=(struct NODE *)malloc(sizeof(struct NODE));
        if (NULL==q) return 1;
        q->data=rand()%100;//填写0..99的随机值
        q->next=NULL;
        p->next=q;
        p=q;
    }

    //输出整个单链表
    s=head->next;
    while (1) {
        if (NULL==s) {
            printf("\n");
            break;
        }
        printf("%02d->",s->data);
        s=s->next;
    }

    //将值为5的结点插入到单链表的第k个结点前
    k=3;
    n=0;
    p=head;
    while (1) {
        if (NULL==p) { 
 
            break;
        }
        n++;
        if (k==n) {
            q=(struct NODE *)malloc(sizeof(struct NODE));
            if (NULL==q) return 1;
            q->data=5;
            q->next=p->next;
            p->next=q;
            break;
        }
        p=p->next;
    }

    //输出整个单链表
    s=head->next;
    while (1) {
        if (NULL==s) {
            printf("\n");
            break;
        }
        printf("%02d->",s->data);
        s=s->next;
    }

    //删除第k个节点
    k=5;
    n=0;
    p=head;
    while (1) {
        if (NULL==p) {
            break;
        }
        n++;
        if (k==n) {
            q=p->next;
            if (q) {
                p->next=q->next;
                free(q);
            }
            break;
        }
        p=p->next;
    }

    //输出整个单链表
    s=head->next; 
 
    while (1) {
        if (NULL==s) {
            printf("\n");
            break;
        }
        printf("%02d->",s->data);
        s=s->next;
    }

    //从小到大排序
    for (p=head;p!=NULL && p->next!=NULL;p=p->next) {
        for (q=p->next;q!=NULL && q->next!=NULL;q=q->next) {
            if (p->next->data > q->next->data) {

                //交换data
//              printf("swap %02d %02d\n",p->next->data,q->next->data);
//              t=p->next->data;p->next->data=q->next->data;q->next->data=t;

                //或者

                //交换next
//              printf("swap %02d %02d\n",p->next->data,q->next->data);
                s1=p->next;
                s2=p->next->next;
                s3=q->next;
                s4=q->next->next;

                if (s2!=s3) {
                     p->next=s3;
                    s3->next=s2;
                     q->next=s1;
                    s1->next=s4;
                } else { 
 
                     p->next=s3;
                    s3->next=s1;
                           q=s3;
                    s1->next=s4;
                }

                //输出整个单链表
//              s=head->next;
//              while (1) {
//                  if (NULL==s) {
//                      printf("\n");
//                      break;
//                  }
//                  printf("%02d->",s->data);
//                  s=s->next;
//              }
//              getchar();
            }
        }
    }

    //输出整个单链表
    s=head->next;
    while (1) {
        if (NULL==s) {
            printf("\n");
            break;
        }
        printf("%02d->",s->data);
        s=s->next;
    }

    //释放所有节点
    p=head->next;
    while (1) {
        if (NULL==p) { 
 
            break;
        }
        q=p->next;
        free(p);
        p=q;
    }

    return 0;
}
//29->82->10->22->71->05->13->60->59->39->
//29->82->05->10->22->71->05->13->60->59->39->
//29->82->05->10->71->05->13->60->59->39->
//05->05->10->13->29->39->59->60->71->82->
//

[解决办法]
我刚才仔细看了下你的代码，原来你创建的是循环链表，我回答错了哈，不好意思。。。
[解决办法]
崩溃的时候在弹出的对话框按相应按钮进入调试，按Alt+7键查看Call Stack里面从上到下列出的对应从里层到外层的函数调用历史。双击某一行可将光标定位到此次调用的源代码或汇编指令处。

[解决办法]
不是你的删除函数的问题，是你的打印函数除了问题
你的打印函数经过循环后，把链表的那个指针最后指向了NULL，所以再到删除函数里面去再使用->当然就出错了，建议你在打印函数里面声明一个临时的结构指针变量来操作，并在删除函数里面将for循环里面的p->next改为p，这样就不会出错了。
[解决办法]

void ListDelete(LinkList &L,int i,int e)
{
    LinkList p = L;
    int j = 0;
    if(L == NULL)
        return;
    while (p->next && j<i-1) {//每次都会停在这，说是EXC_BAD_ACCESS，这什么问题啊？？
        p = p->next;
        ++j;
    }
     
    if(!p->next&&j>i-1){cout<<"error3.."<<endl;}
    LinkList q = p->next;
    p->next = q->next;
    e = q->data;
    free(q);
}

判断下边界条件。
[解决办法]
while (p!=NULL && p->next && j<i-1) {

查看更多 下一篇