在C中反转字符串[关闭]

Question

#include<stdio.h>
#include<string.h>
#include<malloc.h>
char *str_rev(char s[]){
 static char *st = NULL;
 static char *l;
 int i = 0,c = 0;
 st = malloc((strlen(s) * sizeof(*s))+1);
 *l = st;
 if(s == NULL){
  return "INVALID PARAMS";
 }
 for(i=0;s[i]!='\0';i++){
 ;
 }
 for(c=i;c >=0;c--){
  *l++ = s[c];
 }
 l = '\0';

 return st;
}
int main(){
 char a[]="Angus Declan R";
 printf("\n %s \n",str_rev(a));
 return 0;
}

如何释放在 func str_rev() 中使用 malloc() 分配的内存，因为我需要重新运行反转的字符串。

Answer 1

(1):l 中的第一个内存是\0，原因如下：

在你的循环之后

for(i=0;s[i]!='\0';i++){
 ;
 }

s[i] 变为 \0 并且您分配 c=i 并且在接下来的第二个循环中您在 l 的第一个点分配 \0。

for(c=i;c >=0;c--){
  *l++ = s[c];  // you are assigning `\0` for first time
 }

你返回return l; 并且在l 的第一个位置\0 是这样 所以在 printf 语句中使用%s

printf("\n %s \n",str_rev(a));    

什么都不打印。

建议：

for(c=i-1;c >=0;c--){
     // ^ c should be i-1 initially 
  *l++ = s[c];
 }

(2):问题代码中至少有两个编译错误。你在两个位置忘记了;

 return "INVALID PARAMS"; 
                        ^ 

下一个

char a[]="Angus Declan R";
                         ^

第三个严重错误

您返回的内存地址无效！
你在做什么，在st分配内存，然后分配给l，然后free(st)并返回l：（读取cmets）

st = malloc((strlen(s) * sizeof(*s))+1);   // allocation 
l = st;          // assign to l

// code in between

free(st);    // st and l become invalid after free
return l;    // returning invalid memory 

建议：你是否在不调用 free() 的情况下使用 l 并返回 st。

(4)：

这不是错误，但是为什么这个无用的循环？

while(c > 0){
  l--;
  c--;
 }

(5)：忘记了l前面的*

for(c=i;c >=0;c--){
  *l++ = s[c];
 }
  l = '\0';
 ^ forgot *   it should be *l = '\0';