Discussion :

[coyotte507]

bonjour,
voila mon code, tres simple:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#include <iostream>
 
void fonct(char *r)
{
    std::cout << r[0] << std::endl; //affiche a
    r[0] = 'a'; // bugge ici, segfault
}
 
int main()
{
	std::cout << "Hello world!" << std::endl;
    char *chaine;
    chaine = "acdcsc";
    fonct(chaine);
 
	return 0;
}

Pourquoi une segfault sur r[0] = 'a';?

Si jamais je fais: r = "abc";
Alors il n'y a aucun problème.

Merci de votre aide.

[David Fleury]

les chaines littérales sont constantes en C++

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
char chaine[] = "acdcsc";

pour éviter le problème

[Laurent Gomila]

Et surtout, en C++ les chaînes de caractères c'est std::string, surtout pas char*.

[koala01]

Et surtout, en C++ les chaînes de caractères c'est std::string, surtout pas char*.

+1

L'avantage d'une std::string, c'est qu'elle permet tout ce que peut permettre un char*, mais qu'elle évite de nombreux désagrément, tout en fournissant, de plus, certaines fonctions intéressantes propres à leur fonctionnement...

Si vraiment, tu dois avoir recours à un char* (pour l'utiliser comme nom de fichier à ouvrir, par exemple, ou pour le passer comme argument à une fonction de type C), tu pourras toujours l'obtenir grace à machaine.c_str()

[Startux]

Pour répondre à ton problème de manière plus scolaire :

tu déclares un pointeur sur une chaîne de caractères.
tu ne lui alloue pas de mémoire
résultat tu fais une segmentation fault.

solution 1 : tu utilises en C++ string comme le dit laurent.
solution 2 : plus orienté C que C++, tu alloues de la mémoire avec

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

variable = (char*)malloc(sizeof(char)*Nombre de caractères voulus dans ta chaîne)

Un travail n'a d'importance que si son résultat est très attendu

[koala01]

Pour répondre à ton problème de manière plus scolaire :

tu déclares un pointeur sur une chaîne de caractères.
tu ne lui alloue pas de mémoire
résultat tu fais une segmentation fault.

solution 1 : tu utilises en C++ string comme le dit laurent.
solution 2 : plus orienté C que C++, tu alloues de la mémoire avec

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

variable = (char*)malloc(sizeof(char)*Nombre de caractères voulus dans ta chaîne)

Un travail n'a d'importance que si son résultat est très attendu

ou, un peu mieu quand meme, car plus dans l'optique du C++, en utilisant l'operateur new pour l'allocation

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char *variable=new char[nombre de caractères voulus];

avec, pour la libération, appel à delete[] variable

EDIT:Quitte à donner des trucs bancaux, autant essayer qu'ils ne le soient pas trop quand meme