我正在尝试永久更改字符串，但这段代码没有这样做

Question

    #include<stdio.h>
    #include<conio.h> //**Just To Add getch() function**
    int length(char *p){
        int i; //**I know That these variable are not the same as they are in other function**
        for(i=0;*(p+i)!='\0';i++);
        return i;
    }
    void strrev(char *p){
        int i,len;
        len=length(p);    
        char cpy[len]; //**Already Tried to change it to some fixed value**
        for(i=0;i<len;i++){
            cpy[i]=*(p+len-i);
        }
        for(i=0;i<len;i++){
            *(p+i)=cpy[i];
        }
    }
    int main(){
        char str[20]="computer";
        strrev(str);
        printf("%s",str);
        getch(); //**to Stop The Screen**
        return 0;
    }

我尝试将数组大小更改为固定值，我也尝试更改变量，但我的语法没有错误。

Answer 1

根据@Yunnosch 的推荐，这是我的评论作为答案。

在您的函数 strrev 中，您迭代整个字符串，即 i 从零到 len / 2 的迭代您正确地抓取了字符，但剩余的迭代只是再次撤消此操作。

因此，只需从零迭代到 len >> 1。位移确保整数除法。

void strrev(char* const str)
{
   const size_t len = strlen(str);
   for(size_t i = 0; i < (len >> 1u); ++i)
   {
     const size_t j = len - 1u - i;
     char c = str[i];
     str[i] = str[j];
     str[j] = c;
   }
}

Answer 2

使用这个cpy[i]=*(p+len-i); 意味着您将\0 终止符放在字符串cpy 的开头。在这种情况下，您的两个字符串都将以 \0 开头，因此 printf 将什么也不做。

所以将cpy[i]=*(p+len-i); 更改为cpy[i] = *(p + len - i-1);。