以下交换字符串的程序到底哪里错了

[hujiiori]

帮我看看以下交换字符串的程序到底哪里错了,编译通过,运行时出现错误.

#include<iostream.h>
#include<string.h>

int stringswp(char * i,char * j){
char temp[100];
strcpy(temp,i);
strcpy(i,j);//好像这句出的错
strcpy(j,temp);
return 0;
}

int main(){
char *p="how are you";
char *q="how you";
stringswp(p,q);
while(*p!='\0'){
cout<<*p;
p++;
}
cout<<endl;
while(*q!='\0'){
cout<<*q;
q++;
}
return 0;
}

[peter9606]

int main(){
//char *p="how are you";
char p[] ="how are you";
//char *q="how you";
char q[] ="how you";
stringswp(p,q);
while(*p!='\0'){
cout<<*p;
p++;
}
cout<<endl;
while(*q!='\0'){
cout<<*q;
q++;
}
return 0;
}

如果楼主的方法 那么两个指针都没有被分配空间~

[ma100]

p和q都是常量，不能改变

[真相重于对错]

int stringswp(char * i,char * j){
char *temp;
temp = strdup(i)
j=strdup(i);
i=strdup(j)
return 0;
}

[antijpn]

不对的地方多呢，二楼改的也不对，越界访问了
int stringswp(char * i,char * j){
char temp[100];
strcpy(temp,i);
strcpy(i,j);//好像这句出的错
strcpy(j,temp);
return 0;
}
改成
int stringswp(char** i,char** j)
{
char* temp;
temp=*i;
*i=*j;
*j=temp;
return 0;
}

[peter9606]

汗~

[antijpn]

int stringswp(char * i,char * j){
char *temp;
temp = strdup(i)
j=strdup(i);
i=strdup(j)
return 0;
}



错的,而且内存泄漏了
就算是这样,也要用char**i,char**j为参数

[一叶还真不知秋]

就C的特性决定了这样做一定会错。
既然用了C＋＋，何不用std中的string类，本身就有交换功能。

[hujiiori]

有这么多错啊……

[Andy84920]

用string不好吗?

[Andy84920]

#include<iostream>
#include<cstring>
using namespace std;

int stringswp(const char * i,const char * j)
{
char temp[100];

strcpy(temp,i);
strcpy(const_cast<char *>(i),j);
strcpy(const_cast<char *>(j),temp);

return 0;
}

int main()
{
const char p[]="how are you";
const char q[]="how you";

stringswp(p,q);

const char *pp=p;
const char *pq=q;

while((*pp!='\0')&& (cout << *pp++));
cout<<endl;

while((*pq!='\0')&&(cout << *pq++));
cout << endl;
}

还有一点错误,自己改改...

[antijpn]

楼上的程序和二楼有什么区别吗？看不出来，越界访问了了-_-bbb（修正了旧的错误，引入了新的错误，是不是M$的程序员？）

[pwtitle]

对指针进行引用就可以了
int stringswp(char *&i,char *&j){
char temp[100];
strcpy(temp,i);
strcpy(i,j);//好像这句出的错
strcpy(j,temp);
return 0;
}

你的程序可以优化为：

#include<iostream>

using namespace std;

void stringswp(char *& i,char *& j){
char *temp = i;
i = j;
j = temp;
}

int main(){
char *p="how are you";
char *q="how you";
stringswp(p,q);

cout<<"*p="<<p<<endl;
cout<<"*q="<<q<<endl;

system("PAUSE");
return 0;

}

[pwtitle]

以上程序已经测试通过，环境Dev-C++ 4.9.10

[Andy84920]

呵,楼上的只是换了一下指针的指向,不过也行...-_-

[Andy84920]

可能楼主是想训练strcpy,不然为什么不用string是吧?我想....楼上的程序只
是一个变通的做法...

我还没看到我的程序哪里越界了,我知道是越界了.^_^

[Andy84920]

哦,知道了....初始化数组大小....

[pwtitle]

一样啊，把指针更换改成strcpy也一样啊。
但直接调换指针效率要高的多吧？

[antijpn]

不一样，char some[]="str"只是保证了'\0'的空间，而不保证更多的空间给strcpy用