声明Int变量会导致分段错误？

Question

嘿家伙我在我的程序中出现了一个非常奇怪的分段错误。这个程序假设自动化纸牌游戏“战争”，到目前为止，我已经能够为两个玩家构建两个随机化的半套牌。这似乎表明排队工作正确。我也能够将所有值排队，并且它们以正确的顺序出现。但是，如果我每次取消注释主程序段错误中的整数声明，主内部。我不能为了我的生活找出简单的声明可能会导致错误。请注意，这是我使用队列的唯一第二项任务。

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

typedef struct node 
{ 
    int value; 
    int suit; 
    char*sname; 
    char*txt; 
    struct node *next; 
} node; 
int isempty(node *base){ 
    if (base==NULL) 
     {return 1;} 
    else 
     return 0; 
} 
void printlist(node *base){ 
    node *current=base;         
    if(base==NULL) 
     { 
      printf("The List is empty!\n"); 
      return; 
     } 
    else 
     { 
      printf("Content: \n"); 
      int count=0; 
      while(current!=NULL){ 
       count++; 
       printf("%s \tof \t%s\n",current->txt,current->sname); 
       current=current->next; 
      }              
      printf("\nCount:%d\n",count); 
     } 
    }  
char* valname(int n){          
    char *name;           
    switch(n)            
    {              
     case 0:name="two";break;       
     case 1:name="three";break;      
     case 2:name="four";break;      
     case 3:name="five";break;       
     case 4:name="six";break;       
     case 5:name="seven";break;      
     case 6:name="eight";break;      
     case 7:name="nine";break;       
     case 8:name="ten";break;       
     case 9:name="Jack";break; 
     case 10:name="Queen";break; 
     case 11:name="King";break; 
     case 12:name="Ace";break; 
     default:printf("Broken\n");exit(1);    
    }             
    return(name);           
} 
char* suitname(int n){ 
    char *name; 
    switch(n){ 
     case 0:name="Hearts";break; 
     case 1:name="Spades";break; 
     case 2:name="Clubs";break; 
     case 3:name="Diamonds";break; 
     default:printf("Broken\n");exit(1); 
    } 
    return(name); 
} 
void enqueue(node **base,int item){ 
node *nn,*current=*base; 
nn=malloc(sizeof(node)); 
    if(*base==NULL) 
    { 
     *base=nn; 
    } 
    else 
    { 
     while(current->next!=NULL){ 
      current=current->next; 
     } 
     current->next=nn; 
    } 
    nn->value=item; 
    nn->txt=valname(item%13); 
    nn->sname=suitname(item/13); 
    nn->next=NULL; 
} 
int dequeue(node **base){ 
node *current=*base,*temp; 
    if (isempty(*base)==0){ 
     int giveback=current->value; 
     if(current->next==NULL) 
     { 
      free(*base); 
      *base=NULL; 
     } 
     else 
     { 
      temp=current->next; 
      free(current); 
      *base=temp; 
     } 
     return giveback; 
    }else{return -1;} 
} 
void createdecks(node **deck1,node **deck2){ 
int i=0; 
int thenumber=0; 
int deck[52]={0}; 
for(i=0;i<26;i++){ 
     thenumber=rand()%52; 
     if(deck[thenumber]==0){ 
      //add to list 
      enqueue(deck1,thenumber); 
      deck[thenumber]=1; 
     } 
     else 
     { 
      i--; 
     } 
    } 
for(i=0;i<26;i++){ 
     thenumber=rand()%52; 
     if(deck[thenumber]==0){ 
      //add to list 
      enqueue(deck2,thenumber); 
      deck[thenumber]=1; 
     } 
     else 
     { 
      i--; 
     } 
    } 
} 
int main(void){ 
    node *d1,*d2,*warholder; 
    //int c1=0,c2=0;     //THIS LINE!!!!!!!!!!! 
    srand(time(NULL)); 
    createdecks(&d1,&d2); 
    //printlist(d1); 
    //printlist(d2); 
    int i=0; 
    for(i=0;i<26;i++) 
     printf("%d ",dequeue(&d1)); //return testing 
    printf("\n"); 
    printlist(d1); 
} 

教授的例子功能

char * namenum(int num) 
{ 
    char * name; 
    switch(num) 
    { 
     case 0: 
      name = "zero"; break; 
     case 1: 
      name = "one"; break; 
     case 2: 
      name = "two"; break; 
     case 3: 
      name = "three"; break; 
     case 4: 
      name = "four"; break; 
     case 5: 
      name = "five"; break; 
     case 6: 
      name = "six"; break; 
     case 7: 
      name = "seven"; break; 
     case 8: 
      name = "eight"; break; 
     case 9: 
      name = "nine"; break; 
     default: 
      printf("Invalid Number generated\n"); 
      exit(1); 
    } 
    return name; 
} 

Answer 1

我简单地看了一下代码，它看起来像你有一个未初始化变量的问题。您在main()声明如下：

node *d1,*d2,*warholder; 

然后你把它传递给createdecks()，进而调用enqueue()。 enqueue()假定指针已初始化。

尝试在main()初始化d1和d2：

node *d1,*d2,*warholder; 
d1 = d2 = warholder = NULL; 

Answer 2

它不是由于声明INT。在使用它之前，您还没有将内存分配给char *名称。例如

char* valname(int n){          
    char *name;  

上面的代码后使用的名字前使用malloc（）分配内存

char* suitname(int n){ 
    char *name; 

一错再错

如果你想避免这种情况使用数组，而不是指针