字符串反向程序抛出异常

Question

// ExampleCodes.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#include<stdio.h>
#include<iostream>
using namespace std;    

char* stringReverse(char* s)    
{    

    char temp, *p,*q;    
    q = s;    
    while( *(++q));    
    for( p = s; p &lt; --q; p++)    
    {    
        temp = *p;    
        *p = *q;     
        *q = temp;    
    }    
    return s;    
}    


int _tmain(int argc, _TCHAR* argv[])    
{    

    stringReverse("StringReverse");    
    return 0;    
}    

Answer 1

你不能修改常量字符串。

stringReverse("StringReverse");

您可以改用字符数组：

char str[] = "StringReverse";

Answer 2

字符串字面量，如"StringReverse"，不允许在C中修改。使用数组：

char str[] = "StringReverse";
stringReverse(str);

请注意，如果您向 stringReverse() 函数提供长度为零的字符串，则它的行为未定义。

Answer 3

您从未为 p 分配任何内存。你可以试试p = new char[sizeof(s)];

Answer 4

也许你应该这样写：

// in - string to reverse
// out - buffer for reversed string
// l - size of in

StringReverse(char* in, char* out, int l) {
  for (int i=0;i<l;i++) {
    out[l-(i+1)] = in[i];
  }
}

*[] 和 *(p++) / *(p--) 之间几乎没有速度差异

Answer 5

好吧，我本来打算清理 OP 中糟糕的源代码，但我会发布一个更简洁的版本：

#include <stdio.h>
#include <string.h>

void reverse_string(char *s)
{
    char *q = s + strlen(s);
    for (char *p = s; p < --q; p++)
    {
        char temp = *p;
        *p = *q;
        *q = temp;
    }
}

int main()
{
    char s[] = "StringReverse";
    reverse_string(s);
    puts(s);
    return 0;
}

看在你的份上，我希望你还是个学生。

更新0

这实际上是纯 C，所以要学习并敬畏它的性能，但不要像这样开始编写 C++。