魔术师发牌问题

Sally2121

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

#define CardNumber 13

typedef struct node
{
        int data;
        struct node* next;
}sqlist, *linklist;


void main()
{
        void Magician(linklist L);
        linklist creat();
        linklist rs;
        /*int i;*/    //如果把这一句注释掉，换到下面两行
        rs=creat();
        Magician(rs);
        int i;        //换到这里，编译就就会报错，如图片所示，用的visual C++编译器，这是什么原因呢.
        
        for(i=1;i<=CardNumber;i++)
        {
                printf("%d\n",rs->data);
                rs=rs->next;
        }
}
linklist creat()
{
        linklist head=NULL;
        linklist p;
        linklist r;
        int i;
        for(i=1;i<=CardNumber;i++)
        {
                p=(linklist)malloc(sizeof(sqlist));
                p->data=0;

                if(head==NULL)
                {
                        head=p;
                }
                else
                {
                        r->next=p;
                }
                r=p;
        }
        r->next=head;
        return head;
}


void Magician(linklist L)
{
        linklist p;
        int j;
        int Countnumber = 2;
        p=L;
        L->data=1;
        
        
        while(1)
        {
                for(j=1;j<=Countnumber;j++)
                {
                        p=p->next;
                        if(p->data!=0)
                        {
                                j--;
                        }
                }
                if(p->data==0)
                {
                        p->data=Countnumber;
                        Countnumber++;
                        
                        if(Countnumber==14)
                                break;
                }
        }
}

qiuyouzhi

Dev-C++没有报错？你换成别的（Dev-C++）试试，或许是开发环境（编译器）的问题

永恒的蓝色梦想

我这里无论怎样都会报错，因为你的46行的 r 没初始化内存。
环境：VISUAL STUDIO 2019

FayeHao

之前使用的是VC++6.0，该编译环境不允许在定义变量前面进行函数运算等语句，定义的变量必须放在开头处。所以你的int i；不能放在后面。其他的编译器优化了此操作。不过对于编程习惯而言，一般规定是把定义的变量放在前面，放后面在其他的编程平台虽然不能称之为错的，只是不规范而已。你的这个程序我在code::block中运行正常。

这个程序我发现还有些不太规范化的操作，在第15行16行应该为函数声明，应该放在main函数之上而非main函数之内。

FayeHao

永恒的蓝色梦想 发表于 2020-5-17 17:40
我这里无论怎样都会报错，因为你的46行的 r 没初始化内存。
环境：VISUAL STUDIO 2019

用的不是VS环境。想和你讨论一下：我觉得r没初始化内存也没关系吧。上面已经定义了r的地址(虽然没空间)，然后对地址进行指向操作，之后又把p赋给了r，此时的r就有了内存空间。如果报错就是r的问题，那应该是编译环境不同的问题吧。

永恒的蓝色梦想

FayeHao 发表于 2020-5-18 15:32
用的不是VS环境。想和你讨论一下：我觉得r没初始化内存也没关系吧。上面已经定义了r的地址(虽然没空间)， ...

是的，我觉得没关系，编译器觉得有关系

Sally2121

应该是的，谢谢两位了