Я что-то упустил в своей функции?

Question

Это то, что у меня есть, и я получаю сообщение об ошибке. Любая помощь?

void ReverseString(char* string) {
    int len = strlen(string);
    for(int i = 0; i < len; i++)
    {
         string[i] = string[len-i];
    }
}

Answer 1

• Когда i равно 0, вы получите доступ string[len] что неверно как действительный индекс в массиве длины len являются [0,len-1]

Если я правильно понимаю ваше намерение, вы пытаетесь изменить строку, но я вижу, что пропущено несколько вещей:

• Вы не обмениваетесь.
• Также обмен должен произойти для одна половина массива, не для весь массив.

Следующий фрагмент исправляет эти проблемы:

int len = strlen(string);
for(int i = 0; i < len/2; i++) {
    swap(string[len-i-1],string[i]);
}

Answer 2

Прежде всего, вы получите сообщение об ошибке в строке 6.

Измените { на }. Затем попробуйте снова.

Answer 3

Это тег C ++, делайте это C ++ ...

std::string ReverseString(std::string str) 
{
  std::reverse(str.begin(), str.end());
  return str;
}

Answer 4

Помимо двух уже упомянутых ошибок:

Вы сделаете палиндром из исходной строки.Первая половина станет равной второй половине перевернутой.Однако второй тайм останется прежним.Это не то, что объявляет имя функции.

Answer 5

должно быть string[i] = string[len-i-1];

// added (untested):  

void ReverseString(char * string) {   
    int len = strlen(string);  
    for(int i = 0; i < len / 2; i++)  
    {  
         string[i] ^= string[len-i-1];  
         string[len-i-1] ^= string[i];  
         string[i] ^= string[len-i-1];  
    }  
}