请问这样写程序错了吗？ - 未名空间MITBBS历史存档

国际科技财经博客移民网络热点娱乐民生时事公众号

Redian新闻

>未名空间

>JobHunting - 待字闺中

请问这样写程序错了吗？

请问这样写程序错了吗？# JobHunting - 待字闺中

q*i2010-11-04 07:11

1 楼

int main()
{
char *pString1 = "AAA";
char *pString2 = "BBBBB";
int len = strlen(pString2);
char *NewpString1 = pString1 ;

for(int i=0;i{
*pString1++ = *pString2++;
}
}
return 1;
}
运行之后是 Access violation 。难道因为pString1 是 string literal，不能写？
要怎么改呢？谢谢

s*n2010-11-04 07:11

2 楼

because you are writing in constant TEXT segment.

【在 q*******i 的大作中提到】

: int main()
: {
: char *pString1 = "AAA";
: char *pString2 = "BBBBB";
: int len = strlen(pString2);
: char *NewpString1 = pString1 ;
:
: for(int i=0;i: {
: *pString1++ = *pString2++;

y*o2010-11-04 07:11

3 楼

"AAA" is a constant char* not char*

【在 q*******i 的大作中提到】

: int main()
: {
: char *pString1 = "AAA";
: char *pString2 = "BBBBB";
: int len = strlen(pString2);
: char *NewpString1 = pString1 ;
:
: for(int i=0;i: {
: *pString1++ = *pString2++;

c*22010-11-04 07:11

4 楼

1) You are doing double loops: i++ and *p++
2) You did not check whether *p still points to something

【在 q*******i 的大作中提到】

: int main()
: {
: char *pString1 = "AAA";
: char *pString2 = "BBBBB";
: int len = strlen(pString2);
: char *NewpString1 = pString1 ;
:
: for(int i=0;i: {
: *pString1++ = *pString2++;

c*22010-11-04 07:11

5 楼

This program works fine:

d*t2010-11-04 07:11

6 楼

2楼正解~

q*i2010-11-04 07:11

7 楼

那这个程序怎么改对呢？

【在 s*****n 的大作中提到】

: because you are writing in constant TEXT segment.

q*i2010-11-04 07:11

8 楼

这个程序我这里VC也不能运行，错误一样。难道是编译器不一样，有些情况“AAA”在
text字段，有些情况不在？

【在 c***2 的大作中提到】

: This program works fine:

j*k2010-11-04 07:11

9 楼

二楼应该是这个题目的正解.
但是如果稍微改一下题目, 把pString1,2的定义改为:
char *pString1, *pString2;
pString1 = (char *)malloc(3*sizeof(char));
pString2 = (char *)malloc(21*sizeof(char));
strcpy(pString1,"AAA");
strcpy(pString2,"BBBBBBBBBBBBBBBBBBBB");
这样在for循环里走过第三步的时候pString1就已经到头了, 但为什么接下来的循环都
不会出错? 虽然说c++没有边界检测, 但程序在修改pString1越界指向的内容, 这难道
不该是Access violation吗?

s*f2010-11-04 07:11

10 楼

for(int i=0;i{
pString1 = pString2;
}
}

c*22010-11-04 07:11

11 楼

I also use VC compiler, but only in cmd, not in dde.
Did you include the headers?
stdio/string

【在 q*******i 的大作中提到】

: 这个程序我这里VC也不能运行，错误一样。难道是编译器不一样，有些情况“AAA”在
: text字段，有些情况不在？

c*22010-11-04 07:11

12 楼

this is not the issue. you are just using heap memory instead of stack
memory
Also don't forget to free

【在 j****k 的大作中提到】

: 二楼应该是这个题目的正解.
: 但是如果稍微改一下题目, 把pString1,2的定义改为:
: char *pString1, *pString2;
: pString1 = (char *)malloc(3*sizeof(char));
: pString2 = (char *)malloc(21*sizeof(char));
: strcpy(pString1,"AAA");
: strcpy(pString2,"BBBBBBBBBBBBBBBBBBBB");
: 这样在for循环里走过第三步的时候pString1就已经到头了, 但为什么接下来的循环都
: 不会出错? 虽然说c++没有边界检测, 但程序在修改pString1越界指向的内容, 这难道
: 不该是Access violation吗?

c*22010-11-04 07:11

13 楼

this loop won't do anything.
if you want p1 point to p2, one assignment is enough.
I guess you want to do:
pString1[i] = pString2[i];
in that case, you want to check whether p1 has enough memory to hold all of
p2.
otherwise, random memory overwrite may happen, will cause crash.

【在 s***f 的大作中提到】

: for(int i=0;i: {
: pString1 = pString2;
: }
: }

s*f2010-11-04 07:11

14 楼

careless mistake, yes you are right.
just pString1 = pString2; is enough

of

【在 c***2 的大作中提到】

: this loop won't do anything.
: if you want p1 point to p2, one assignment is enough.
: I guess you want to do:
: pString1[i] = pString2[i];
: in that case, you want to check whether p1 has enough memory to hold all of
: p2.
: otherwise, random memory overwrite may happen, will cause crash.

m*n2010-11-04 07:11

15 楼

不知道你这个程序在干嘛。
你的每次for loop，strlen(pString2) 都要重算一遍，每次都会变。
这种程序对于读你的code的人来说简直是噩梦。
如果你在面试中写出这种程序，就直接挂了。

【在 q*******i 的大作中提到】

: int main()
: {
: char *pString1 = "AAA";
: char *pString2 = "BBBBB";
: int len = strlen(pString2);
: char *NewpString1 = pString1 ;
:
: for(int i=0;i: {
: *pString1++ = *pString2++;