我必须承认,我不知道如何使用指针,但我尝试了非常少。我的程序的问题是它反向显示字符串,除了缺少的第一个字母,整个字符串向前移动一个空格,第一个元素为空。 例如,在输入“hello”时显示“olle”。
#include <iostream>
#include <string>
using namespace std;
string reverse(string word);
int main()
{
char Cstring[50];
cout<<"enter a word: ";
cin>>Cstring;
string results = reverse(Cstring);
cout <<results;
}
string reverse(string word)
{
char *front;
char *rear;
for (int i=0;i< (word.length()/2);i++)
{
front[0]=word[i];
rear[0]=word[word.length()-i];
word[i]=*rear;
word[word.length()-i]=*front;
}
return word;
}
新代码完美无缺。将字符串更改为cstrings。问题技术问cstrings但我发现字符串更容易,所以我使用字符串,然后进行必要的更改,使其成为字符串。想出来也可以初始化后部和前部。
#include <iostream>
#include <cstring>
using namespace std;
string reverse(char word[20]);
int main()
{
char Cstring[20];
cout<<"enter a word: ";
cin>>Cstring;
string results = reverse(Cstring);
cout <<results;
}
string reverse(char word[20])
{
char a='a';
char b='b';
char *front=&a;
char *rear=&b;
for (int i=0;i< (strlen(word)/2);i++)
{
front[0]=word[i];
rear[0]=word[strlen(word)-1-i];
word[i]=*rear;
word[strlen(word)-1-i]=*front;
}
return word;
}
答案 0 :(得分:4)
char *front;
char *rear;
然后再
front[0]=word[i];
rear[0]=word[strlen(word)-1-i];
不好。取消引用未初始化的指针会调用未定义的行为。
除此之外,你的代码过于复杂,它在每次迭代(甚至多次)中调用strlen()
,这是多余的,并且交换逻辑也不必要地复杂。尝试使用两个指针,你的代码会更清晰:
void rev_string(char *str)
{
char *p = str, *s = str + strlen(str) - 1;
while (p < s) {
char tmp = *p;
*p++ = *s;
*s-- = tmp;
}
}
然而,事实是,在C ++中,使用原始指针很少有充分的理由。如何使用std::reverse()
呢?
string s = "foobar";
std::reverse(s.begin(), s.end());
答案 1 :(得分:1)
inline void swap(char* a, char* b)
{
char tmp = *a;
*a = *b;
*b = tmp;
}
inline void reverse_string(char* pstart, char* pend)
{
while(pstart < pend)
{
swap(pstart++, pend--);
}
}
int main()
{
char pstring[] = "asfasd Lucy Beverman";
auto pstart = std::begin(pstring);
auto pend = std::end(pstring);
pend -= 2; // end points 1 past the null character, so have to go back 2
std::cout << pstring << std::endl;
reverse_string(pstart, pend);
std::cout << pstring << std::endl;
return 0;
}
答案 2 :(得分:0)
你也可以这样做:
#include <iostream>
#include <cstring>
using namespace std;
string reverse(char word[20]);
int main()
{
char Cstring[20];
cout<<"enter a word: ";
cin>>Cstring;
string results = reverse(Cstring);
cout <<results;
}
string reverse(char word[20])
{
char a='a';
char b='b';
char *front=&a;
char *rear=&b;
for (int i=0;i< (strlen(word)/2);i++)
{
*front=word[i];
*rear=word[strlen(word)-1-i];
word[i]=*rear;
word[strlen(word)-1-i]=*front;
}
return word;
}
它成功地在我的系统上工作,即。在 Windows 7
上的 emacs + gcc 上答案 3 :(得分:-1)
摘自C《如何编程Deitel和Deitel第8版》:
void reverse(const char * const sPtr)
{
if (sPtr[0] == '\0')
return;
else
reverse(&sPtr[1]);
putchar(sPtr[0]);
}