Discussion :

[stan314]

Salut tout le monde,

J'ai un probleme de saisie lorsque j'ai des char*...
En effet je ne sais pas comment faire un cin d'un char* parce que quand je le fais ca plante...
Comme par ex:

char* nom;
cin>>nom;

La ca plante...
Est ce que quelqu'un pourrait me dire comment faire ?
Merci d'avance
stan314

[Swoög]

Salut !

pense aux balises code la prochaine fois

en C++, pour manipuler les chaînes de caractères, utilise plutôt les string (tu trouveras de nombreux tutos dans la rubrique adéquate de ce site )

De plus, en faisant saisire une chaîne de caractère (que ce soit un char* ou un string), garde à l'esprit que le texte sera coupé après le premier espace (tu ne récupérera pas ce qu'il y a après le premier espace, si tu veux une ligne en entier, il vaut mieux te tourner vers getline )

sinon, pour le problème actuel, c'est parce que tu n'alloue ton char*, c'est à dire que tu déclares un pointeur, mais sans lui réserver, d'espace mémoire...

tu as deux choix : demander une taille fixe de mémoire, en changeant la déclaration :

Code :

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

char nom[100];

par exemple, oublie pas qu'il faut le nombre de caractères attendus + 1

soit faire une allocation dynamique :

Code :

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

char* nom = new char[100];

ici aussi, pense au nombre de caractère + 1...

mais dans ce cas, il te faudra penser à faire un

Code :

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

delete[] nom;

pour libérer l'espace mémoire demander...

au fait, tout ceci fait réellement partie des bases en C//C++...

Il vaudrait peut-être mieux que tu lises un petit tuto

[Laurent Gomila]

Ton pointeur pointe nulle part, il faut allouer de la mémoire avant de saisir quoique ce soit dans ta chaîne (ce qui implique notamment de connaître la longueur de la chaîne à l'avance).

Mais les char* c'est mal en C++

Code :

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

1
2
3
4
#include <string>
 
std::string chaine;
std::cin >> chaine;

Voir la FAQ C++ pour plus de détails.

[Pragmateek]

Il faut déclarer la chaine ainsi:

Code :

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

char s[100]="";

Mais comme cela a été dit en C++ les string sont mieux même si plus lentes.

[diogene]

Laurent Gomila

Mais les char* c'est mal en C++

Voila un position très dogmatique! Les char *, c'est pas plus mal qu'autre chose si on sait s'en servir et si on s'en sert à bon escient. Les string ne sont pas par axiome une nécessité absolue .
Swoög

sinon, pour le problème actuel, c'est parce que tu n'alloue ton char*, c'est à dire que tu déclares un pointeur, mais sans lui réserver, d'espace mémoire...

Soyons plus précis dans nos affirmations:

Code :

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

1
2
3
 
char* nom;
cin>>nom;

Le char * (autrement dit nom) est parfaitement alloué et l'espace mémoire nécessaire est bien réservé pour ce char *. Ce qui n'est pas correct, c'est que nom a pour vocation de contenir l'adresse d'une zone mémoire dédiée au stockage d'un ou plusieurs char , que cette zone mémoire (pour stocker les char) n'est pas allouée et que nom n'est pas initialisé avec cette hypothétique zone mémoire.
Pour allouer cette zone, Swoög propose qq possibilités qui résoudront le problème.

[Matthieu Brucher]

Les char* sont à éviter dans ce cas, les string proposent tout ce qu'on serait obligé de faire à côté pour lire sur cin, et plus encore. Et quand on a des arguments pour ne pas utiliser string, on devrait utiliser vector<char>. Resource Acquisition Is Initialization.

[Swoög]

diogene : tout d'abord excuse moi pour ce léger écart de langage, il est vrai que je me suis imparfaitement exprimé.
Cependant, permet-mio de te dire que pour quelqu'un qui est aussi précis, tu me semble en même temps très permissif !

Les string de la STL C++ sont parfaitement portables et standards, ils permettent de se séparer de syntaxe lourdes et peu claires, ainsi que de se détacher de l'allocation mémoire, En outre, ils sont certainement étés optimisés au maximum par des personnes qui ont infiniment plus de connaissances et d'expériences que toi ou moi en la matière, je te met au défi de dire le contraire (et de le prouver)

Ils font parties des Différences Majeures entre le C et le C++ (qui sont deux langages différents ne l'oubliont pas !)
si pour des raisons de compatibilté (et d'utilité dans certains cas bien précis) les pointeurs ont été conservés en C++, ce n'est pas une raison pour les utiliser à tort et à travers : certe savoir les utiliser et un atout. Mais comme tu le dis toi même, il faut les utiliser à bonne escient... et soit dit en passant, je pense qu'utiliser des pointeurs sur des types de base en C++ ne se fait jamais à bonne escient, il y a les références et les conteneurs standards pour les remplacer.
Cependant, dans un cas aussi simple que celui-ci, je pense qu'on peut dire : les char* c'est mal.

en effet, pourquoi utiliser des pratiques dépréciées et plantogènes alors qu'il existent des pattern optimisés et sûrs ?

[Laurent Gomila]

Voila un position très dogmatique! Les char *, c'est pas plus mal qu'autre chose si on sait s'en servir et si on s'en sert à bon escient. Les string ne sont pas par axiome une nécessité absolue .

J'ai rarement (jamais) vu un cas où un char* était préférable un std::string. Le fait est que la classe de chaînes de caractères en C++ est std::string, tout comme les tableux dynamiques sont std::vector.