反转C字符串函数崩溃了吗?

时间:2013-05-17 17:58:10

标签: c string return reverse

我正在尝试编写一个C函数来反转传入的C样式字符串(即char *)并返回反向字符串的char指针。但是当我在VS2012中运行它时,终端中没有打印任何内容,并且“main.exe已停止工作”,msg显示出来。

#include <stdio.h>
#include <string.h>
char * rrev_str(char * str )
{
    char *revd_str=""; //I tried char revd_str []="" error: stack around "revd_str" is corrupted
    int i,r;
    int str_len=strlen(str);
    for (i = str_len-1, r=0; i >=0; i--,r++)
    {
        revd_str[r]= str[i];
    }
    return revd_str;
}

int main(int argc, char* argv[])
{
   char str1 [] ="STEETS";
   char str2 [] ="smile everyday!";

   //reverse "chars" in a C string and return it
   char * rev_string=rrev_str(str1);
}

3 个答案:

答案 0 :(得分:2)

这里的问题是三折。首先,您没有为反向字符串分配足够的空间,其次,您在rrev_str()中返回指向局部变量的指针,第三,您正在修改字符串文字。您需要在堆上为revd_str分配空间:

char * rrev_str(char * str )
{   
    int i,r;
    int str_len=strlen(str);

    char *revd_str=malloc(str_len + 1); 
    memset(revd_str, 0, str_len + 1);

    for (i = str_len-1, r=0; i >=0; i--,r++)
    {
        revd_str[r]= str[i];
    }
    return revd_str;
}

答案 1 :(得分:1)

问题:

您正在访问无效的内存地址 revd_str指向长度为1的文字常量字符串,并且您将访问超出无效长度的字符串。

<强>解决方案:

  • 创建需要长度的char数组(静态或动态)。
  • 反转给定的字符串。
  • 将第二个参数作为目标字符串传递
    语法:char * rrev_str(char * src, char *dest);

反转给定的字符串

char * rrev_str(char * str )
{
   int start = 0;
   int end = strlen(str) - 1;
   char temp;

    for (; start < end; start++ ,end--)
    {
        temp = str[start];
        str[start] = str[end];
        str[end] = temp;
    }
    return str;
}

int main(int argc, char* argv[])
{
   char string [] ="smile";

   //reverse "chars" in a C string and return it
   char * rev_string = rrev_str(string);

   printf("%s",rev_string);
}


将第二个参数作为目标字符串

char * rrev_str(char * src, char *dest)
{
   int srcLength = strlen(src);
   int destLength = strlen(dest);
   int i;
   // Invalid destination string
   if (srcLength > destLength)
   {
        return NULL;
   }

   dest[srcLength] = '\0';
   srcLength--;
    for (i=0; srcLength >= 0;i++, srcLength--)
    {
        dest[i] = src[srcLength];
    }

 return dest;
}

int main(int argc, char* argv[])
{
   char string [] ="smile";
   char revString[20];  

   //reverse "chars" in a C string and return it
   char * rev_string = rrev_str(string, revString);

    printf("%s",rev_string);
}

答案 2 :(得分:0)

什么!你在做..

char *revd_str=""; // Creating String Literal which can't be modified because they are read only  
char *revd_str[]=""; // Creating Char Array of Size Zero.

所以解决方案是

要么参考你的字符串

char *revd_str = strdup(str);

或创建动态字符数组

char *revd_str = (char*) malloc (strlen(str)+1);  

你的程序运行正常。逻辑不正确的反转所以修改它。下面给出了一个样本解决方案

char * rrev_str(char * str )
{
    char *revd_str=strdup(str);
    int i;  // no need for extra 'int r'
    int str_len=strlen(str);
    for (i = 0; i < str_len/2; i++)
    {
        char temp = revd_str[i];
        revd_str[i]= revd_str[str_len - 1 -i];
        revd_str[str_len - 1 -i] = temp;
    }
    return revd_str;
}