Discussion :

[BBric]

Bonjour, est-ce qu'on peut créer un tableau de char de taille définie dans une fonction, le remplir par indices et renvoyer un pointeur.
Ce code compile mais donne une erreur de segmentation à l'exécution.

Code :

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

1
2
3
4
5
6
7
const char * f()
{
	char * s[2] = {0};
	*s[0] = 'a'; // erreur de segmentation
	*s[1] = '\0';
	 return *s;
}

J'ai essayé d'autres possibilités mais rien ne fonctionne, problème de conversion, d'initialisation, ou retour de l'adresse d'une variable locale.

[Iradrille]

Hello,

Utilises des vector (ou des string)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
std::vector<char> f() {
   std::vector<char> tab(2);
   tab[0] = 'a';
   tab[1] = '\0';
   return tab;
}
 
// ou vu que tu sembles utiliser des chaines de caractères
std::string f() {
   return "a";
}

[imperio]

Si tu veux a tout prix rester dans du C-like :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
char * f()
{
	char *s = new char[2];
	s[0] = 'a';
	s[1] = '\0';
	return s;
}
 
void tralala()
{
  char *s = f();
  delete[] s;
}

Cependant je pense que la solution proposee par Iradrille est plus adaptee.

[BBric]

Avec cette technique il faut effacer le pointeur de l'extérieur aussi...
Merci pour les réponses.

[Astraya]

Je te déconseille de faire ce genre de fonction... Tu ne sais jamais si ton delete sera bien appelé après ton new.
je vois trois possibilitée:
1 - Tu encapsules ton tableau dans une classe. Cette classe alloue et delete le contenu automatiquement; cf : RTTI

Code :

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

1
2
3
4
5
6
7
8
 
class s
{
     char * data;
 
     s(size_t size): data(new (size)){}
     ~s(){ delete data; }
};

Mais ici tu dois faire attention lorsque tu copies ta classe s. (Interdire la copie pourrais être une bonne idée)
2 - Passe les informations nécessaires à ta fonction f():

Code :

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

1
2
3
4
5
void f(char* data, size_t size)
{
    data[0] = 'a'; // erreur de segmentation
    data[1] = '\0';
}

Et la tu demandes au client de fournir un char* alloué avec la taille nécessaire. C'est lui qui new c'est lui qui delete, pas de 50/50.

3- Utilise les std::string (Celle que je ferais si j'étais toi)

Code :

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

1
2
3
4
5
6
7
 
void f(std::string& str)
{
    str.reserve(2);
    str.push_back('a');
    str.push_back('\0');
}

[BBric]

La seconde est la plus simple, j'avais remarqué que passer la cible était une solution fréquente aussi. Pour les chaînes je garde ça sous le coude pour le moment.
Merci.