関数は文字列を正しくリバースしません

Question

#include <iostream> 
#include <cstring> 
using namespace std; 




void reverseString(char s[]) 
{ 
    int length = strlen(s); 
    for (int i = 0; s[i] != '\0'; i++) { 
     char temp = s[i]; 
     s[i] = s[length - i - 1]; 
     s[length - i - 1] = temp; 

     cout << s[i]; //this ends up printing "eooe" instead of reversing the whole string 
    } 
} 

int main() 
{ 
    char a[] = "Shoe"; 
    reverseString(a); 

    return 1; 
} 

アルゴリズムがどこで狂っているのか、それを修正するために何ができるのだろうと思っています。何かを見落としました。正しく動作します。

Answer 1

以下のコードを使用します。あなたのアルゴは正しいですが、少し修正が必要

#include <stdio.h> 

void strrev(char *p) 
{ 
    char *q = p; 
    while(q && *q) ++q; 
    for(--q; p < q; ++p, --q) 
    *p = *p^*q, 
    *q = *p^*q, 
    *p = *p^*q; 
} 

int main(int argc, char **argv) 
{ 
    do { 
    printf("%s ", argv[argc-1]); 
    strrev(argv[argc-1]); 
    printf("%s\n", argv[argc-1]); 
    } while(--argc); 

    return 0; 
} 

Answer 2

を、あなたはlength/2回のアルゴリズムを実行する必要があります。あなたの文字列が内容を交換することを防ぎます。i = 2s = eohsにありますが、hはoで再び交換されます。ブレークポイントを挿入してさらに理解してください。私は少しあなたの機能を変更します。

char* reverseString(char s[]) 
{ 
    int length = strlen(s); 
    for (int i = 0; i<length/2; i++) 
    { 
     char temp = s[i]; 
     s[i] = s[length - i - 1]; 
     s[length - i - 1] = temp; 
     //cout << s[i]; //this ends up printing "eooe" instead of reversing the whole string 
    } 

    return s; 
} 

int main() 
{ 
    char a[] = "Shoe"; 
    cout<<reverseString(a); 
    system("pause"); 
    return 1; 
}