C++: Permutation a-variables

Question

Possible en double:
Why do I get a segmentation fault when writing to a string?

J'ai le programme suivant:

#include <iostream> 
using namespace std; 

void reverseString(char* first, char* last) 
{ 
    while(first < last) 
    { 
     cout << *first << " " << *last << endl; //for debugging; prints 'H' and 'o' then crashes 
     char temp = *last; 
     *last = *first; //this line crashes the program 
     *first = temp; 
     first++; 
     last--; 
    } 
} 

int main() 
{ 
    char* s = "Hello"; 
    reverseString(s, s + strlen(s) - 1); 
    cout << s << endl; 
} 

Cependant, je vais avoir du mal à échanger la les valeurs auxquelles les pointeurs pointent. Je pensais que * p = * p1 devrait simplement mettre la valeur pointée sur p à la valeur pointée de p1, mais quelque chose semble buggé. Merci d'avance pour votre aide!

Answer 1

Le code me semble bien. Le problème le plus probable est que le compilateur est autorisé à supposer que les littéraux de chaîne ne sont pas modifiés, il peut donc les mettre en mémoire en lecture seule. Essayez plutôt

char s[] = "Hello"; 

dans main(), ce qui crée une copie inscriptible de la chaîne littérale.

Answer 2

Le fichier d'en-tête pour strlen() est manquant. En second lieu, il lance un avertissement - Conversion obsolète de la chaîne de caractères en char *, la solution de @ j_random_hacker semble prendre en charge ce problème.

Answer 3

Une solution alternative à @j_random_hacker:

char* buffer = new char[32]; 
strcpy(buffer, "Hello"); 
reverseString(buffer, buffer + strlen(buffer) - 1); 

... rest of your program ... 

delete[] buffer; 

Cette mémoire pour correctement alloue une chaîne C de style qui peut ensuite être modifié par une fonction. Bien sûr, vous devez inclure <string.h> en-tête pour accéder à strcpy et strlen.