1

我是 c 的新手，我处于一个奇怪的情况：我正在尝试更新我作为字符串输入的日期，我的代码是

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;
}

在这里这个函数工作得很好但是当我在另一个函数中调用它时

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

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

现在它打印一些任意文本..：/请有人帮助我

4

2 回答 2

2

在第一个代码中

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

“d[20]”在堆栈上。您正在返回指向堆栈上数据的指针。一旦 update_date() 返回，它的所有局部变量现在都是无效的。

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

strok() 可能最好避免。它会修改您传入的字符串并保持静态状态，这两者都是严重的陷阱并且经常导致细微的错误。

这里有几个替代实现。（当然，在生产代码中，您应该首先避免编写代码来解析日期/时间。有操作系统和库函数可以做到这一点。有很多微妙之处）。

// 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 '/'
    }
}
于 2012-11-04T04:13:27.613 回答
1

我知道这与您的问题无关，但请注意，而不是这个：

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);

你可以简单地这样做：

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

如果您想安全，请使用snprintf：

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

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

于 2012-11-04T05:27:10.230 回答