return 指针 结果不对,大家帮忙看看

nxlinux

int main(void){
        char *pwd="iloveyou";
        char *strrev(char *str);
        //strrev(pwd);
        printf("%s\n",strrev(pwd));
}

char *strrev(char *str){
        int len=strlen(str);
        int i,n;
        char returnstr[len];
        for(i=len-1,n=0;str[i]!='\0';i--,n++){
                returnstr[n]=str[i];
        }
        returnstr[n+1]='\0';
        return returnstr;
        //printf("%s\n",returnstr);
}

我要的结果时uoyevoli 但是结果确实乱码,如果我不return直接strrev(pwd);就是对的.

nxlinux

有人在吗
自己顶起来

hbmhalley

static char returnstr[MAXLEN];

hijzhang

char *strrev(char *str){
        int len=strlen(str);
        int i,n;
        char returnstr[len];
        for(i=len-1,n=0;str[i]!='\0';i--,n++){
                returnstr[n]=str[i];
        }
        returnstr[n+1]='\0';
        return returnstr;
        //printf("%s\n",returnstr);
}

第一，不应该返回一个局部变量
第二，你的循环结束条件错误，会内存越界
第三，returnstr应该申明len+1长度
第四，翻转字符串的算法没必要另开一个内存

nxlinux

回复 3# hbmhalley


    出现这样的错误

    storage size of 'returnstr' isn't constant.

hbmhalley

回复 5# nxlinux


   

static char returnstr[MAXLEN];

    #define MAXLEN xxx

nxlinux

回复 4# hijzhang


    你最后说的翻转没有必要再开内存时什么意思,小弟糊涂了

hijzhang

回复 7# nxlinux


    你直接在pwd上交换就行了
0和len-1交换,1和len-2交换。

lbzhung

hijzhang 发表于 2012-09-08 00:33
回复 7# nxlinux

一般都是char *strrev(const char *str)这种格式，返回一个新的字符串，没必要在原来字符串上改。

hijzhang

strrev - Reverse String

Syntax




#include <string.h>
char *strrev(char *string);


Description

strrev reverses the order of the characters in the given string. The ending null character (\0) remains in place.

Returns

strrev returns a pointer to the altered string. There is no error return value.

strrev返回修改后的字符串，而不是新串。

另外，const 是C++才有的东西，而strrev显然应该设计成是一个c函数
在函数内部分配内存返回给调用者也是要尽力避免的事情。