下面的C程序将打印最短和最长的字符串作为t[0]
和t[n-1]
。但是,当我运行此代码时,它说存在内存问题。我的代码有什么问题?
问题是最后两行,带有"strcpy"。
#include <stdio.h>
#include <string.h>
void fx (char* t[], int n);
int main(void)
{
char* t[] = {"horse", "elephant", "cat", "rabbit"};
int n;
n = sizeof( t )/ sizeof( t[0] );
fx(t, n);
printf("shortest is %s, longest is %sn", t[0], t[n-1]);
}
void fx (char* t[], int n)
{
char st[50], lt[50];
strcpy(lt,t[0]);
strcpy(st,t[0]);
for (int i=0; i<n; i++)
{
if (strlen(t[i]) < strlen(st))
strcpy(st,t[i]);
if (strlen(t[i]) > strlen(lt))
strcpy(lt,t[i]);
}
strcpy( t[0], st);
strcpy( t[n-1], lt);
}
两个 strcpy((s,
strcpy( t[0], st);
strcpy( t[n-1], lt);
错了! t[i]
指向 const 字符串文本 - 不可修改,这会导致运行时出现未定义的行为。
char* t[] = {"horse", "elephant", "cat", "rabbit"};
声明指向字符串文本的指针数组。 字符串文本可以放在只读内存中,并且不能修改。 fx
中的最后 strcpy
行正在尝试写入只读内存。
当你这样做时
char *ptr = "string";
之后,如果你这样做
ptr[0]='S';
这会给你错误。不过它会编译。原因是,"字符串"被放置在内存的数据段或文本部分并且是恒定的。这使得 ptr 成为指向常量字符串的指针,因此不允许修改。
同样,这里所有指向字符串的指针,即horse
、elephant
等都是常量,不应更改。