问题在C

Question

我是新的C和IM在一个奇怪的情况下使用char型日期：我试图更新我曾作为字符串输入日期和我的代码是

typedef struct Employee 
{ 
char fname[20]; 
char lname[20]; 
int eme_id; 
int emr_id; 
char department[20]; 
int age; 
char join_date[20]; 
float bsal; 
float pol_value; 
char pol_start_date[20]; 
char pol_end_date[20]; 
float premium; 
float pre_payment; 
char pre_pay_date[20]; 
int pre_status;/* 0 then not paid 1 then paid*/ 
float bonus; 
}Employee; 


char *update_date(char *dat) 
{ 

char *result = NULL; 
printf(dat); 
result = strtok(dat, "/"); 
int date[3]; 
int i=0; 
while(result != NULL) { 
    printf("result is \"%s\"\n", result); 
    date[i] = atoi(result); 
    printf("%d\n", date[i]); 
    i++; 
    result = strtok(NULL, "/"); 
} 

if(date[1]!=12) 
{ 
    date[1]++; 
} 
else 
{ 
    date[1]=1; 
    date[2]++; 
} 

char a[20]; 
char b[20]; 
char c[20]; 
char d[20]; 
sprintf(a, "%d", date[0]); 
sprintf(b, "%d", date[1]); 
sprintf(c, "%d", date[2]); 

strcpy (d,a); 
strcat (d,"/"); 
strcat (d,b); 
strcat (d,"/"); 
strcat (d,c); 

printf(d); 
return d; 
} 

这里此功能工作得很好，但是当IM调用它的侧像

while(fread(&eme,recsize_eme,1,fq)==1) 
      { 

       char *hell; 
       hell = update_date(eme.pre_pay_date); 
       printf("%s",hell); 
      } 

其他功能，现在它打印的任意文本..：/ PLZ有人帮我出

Answer 1

在FI第一个代码

char d[20]; 
... ... 
return d; 

“d [20]”在栈上。 您正在返回一个指向堆栈上数据的指针。 只要update_date（）返回的所有局部变量现在都是无效的。

在返回之前打印d []的值，所以没有问题。

strok（）可能是最好的避免。 它修改你传入的字符串，并保持静态，这两个都是严重的陷阱，并且经常会导致微小的错误。

这里有几个替代实现。 （当然，在生产代码中，你应该避免编写代码来解析日期/时间，有OS和库函数来实现这一点，有许多微妙之处）。

// Scanf can do parsing for you 
int date[3]; 
int n; 
n = sscanf(dat, "%d/%d/%d", &date[0], &date[1], &date[2]); 
if (n == 3) 
{ 
    // OK, we got 3 integers... 
} 


// atoi() stops on non-digits, use it instead of strtok 
char *result = dat; 
int date[3]; 
int i = 0; 
while (i < 3 && result) 
{ 
    date[i++] = atoi(result); 
    result = strchr(result, '/'); 
    if (result) 
    { 
     ++result; // Skip the '/' 
    } 
} 

Answer 2

我知道这是不是与你的问题，但要注意的是，除了这一点：如果你想

char d[20]; 
sprintf(d, "%d/%d/%d", date[0], date[1], date[2]); 

：

char a[20]; 
char b[20]; 
char c[20]; 
char d[20]; 
sprintf(a, "%d", date[0]); 
sprintf(b, "%d", date[1]); 
sprintf(c, "%d", date[2]); 

strcpy (d,a); 
strcat (d,"/"); 
strcat (d,b); 
strcat (d,"/"); 
strcat (d,c); 

你可以简单地这样做请安全使用snprintf：

char d[20]; 
snprintf(d, sizeof d, "%d/%d/%d", date[0], date[1], date[2]); 

这样做，你摆脱了临时缓冲区，并消除了使用strcat的低效率。请注意，无论何时使用strcat，函数都必须从字符串的开头开始，找到结尾，然后复制源字符串。