Constructeur char[50]

Version imprimable

01/11/2006, 14h24
sushi_

Constructeur char[50]

Bonjour !

Je veux créer un constructeur par defaut (sans paramètres) pour la variable suivante :

Code:

char var[50] ;

J'ai essayé avec

Code:

this->var = new char[50] ;

et

Code:

this->var = new char ;

mais le compilateur me dit "error: incompatible types in assignment of 'char*' to 'char [50]' "

Quelle méthode faut-il utiliser ?

Merci d'avance !
01/11/2006, 14h41
loufoque

new char[50] retourne un char*, pas un char[50].
01/11/2006, 14h50
sushi_

C'est ce que j'avais cru comprendre... Y a t-il un moyen de retourner un char[50] ?...
01/11/2006, 15h09
Laurent Gomila

Tu as déclaré un tableau de 50 caractères, il n'y a rien à allouer ensuite avec new. Si tu veux spécifier la taille à l'exécution alors déclare un char* et utilise new char[50].

Mais char* c'est crade.

En faite, voila ce qu'on me demmande de faire (c'est un TP pour les cours) (IUT Info) :

On me donne la description de classe suivante :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
class Billet
{
	private :
		char villeDepart[50];
		char villeArrivee[50];
		float longueur;
		float prixAuKm;
 
	public :
		Billet();
			// Constructeur par defaut sans parametres
 
		Billet(char villeDepart [], char villeArrivee [], float longueur, float prixAuKm);
			// Constructeur avec les parametres
 
		float prix();
			// Retourne le prix du billet : longueur * prixAuKm
 
		void afficher (char separateur = '\n');
			// Affiche le message suivant :
						// Le billet va de <villeDepart> à <villeArrivee> et a pour prix <prix> <separateur>
 
		void saisir (char message[] = "Entrez un billet : ");
			// Affichage du message et saisie de tous les attributs du billet
 
};

et je dois écrire le ".cc" correspondant...

J'ai écrit ceci :
(J'ai mis en rouge les erreures du compilateur)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
#include "billet.h"
#include <iostream>

using namespace std;


Billet::Billet()
	// Constructeur par defaut sans parametres
{
	this->villeDepart = new char[50] ; // error: incompatible types in assignment of 'char*' to 'char [50]'
	this->villeArrivee = new char[50] ; // error: incompatible types in assignment of 'char*' to 'char [50]'
	this->longueur = 0.0 ;
	this->prixAuKm = 0.0 ;
}

Billet::Billet(char villeDepart[50], char villeArrivee[50], float longueur, float prixAuKm)
	// Constructeur avec les parametres
{
	this->villeDepart = villeDepart ; // error: incompatible types in assignment of 'char*' to 'char [50]'
	this->villeArrivee = villeArrivee ; // error: incompatible types in assignment of 'char*' to 'char [50]'
	this->longueur = longueur ;
	this->prixAuKm = prixAuKm ;
}

float prix()
	// Retourne le prix du billet : longueur * prixAuKm
{
	return this->longueur * this->prixAuKm ; // error: invalid use of 'this' in non-member function

}

void afficher (char separateur);
	// Affiche le message suivant :
				// Le billet va de <villeDepart> à <villeArrivee> et a pour prix <prix> <separateur>
{  // error: expected unqualified-id before '{' token

	cout << "Le billet va de " << villeDepart << " à " << villeArrivee << " et a pour prix " << prix() << separateur << endl ;
}

void saisir (char message[]); 
	// Affichage du message et saisie de tous les attributs du billet
{  // error: expected unqualified-id before '{' token

	cout << message << endl ;
	cout << "Entrez la ville de depart : " ;		cin >> this->villeDepart ;
	cout << "Entrez la ville d'arrivee : " ;		cin >> this->villeArrivee ;
	cout << "Entrez la longueur du trajet : " ;		cin >> this->longueur ;
	cout << "Entrez le prix au Km du billet : " ;	cin >> this->prixAuKm ;
	
}

01/11/2006, 15h53
Laurent Gomila

Qu'est-ce que tu n'as pas compris à ma réponse ?

Si tu veux mettre quelque chose par défaut dans ton tableau tu peux mettre un '\0' dans le premier élément, ça te fera une chaîne vide.
01/11/2006, 15h54
Médinoc
Tu confonds sans doute avec java.

Tu peux carrément supprimer les deux lignes, car les deux tableaux de caractères existent déjà "en dur" dans l'objet.
Par contre, tu peux les initialiser si tu veux, comme ceci par exemple:
Code:

1 2 strncpy(this->villeDepart, "", 50); strncpy(this->villeArrivee, "", 50);
Et pour l'autre constructeur, il te faudra là encore des strcpy() :
Code:

1 2 3 4 strncpy(this->villeDepart, villeDepart, 50); this->villeDepart[50-1] = '\0'; strncpy(this->villeArrivee, villeArrivee, 50); this->villeArrivee[50-1] = '\0';
Pour le reste, tu as oublié billet:: pour le prix et tu as laissé le point-virgules pour les fonctions suivantes.
01/11/2006, 17h12
loufoque

strncpy n'existe pas dans C++.
Par contre il y a std::strcpy dans <cstring>
01/11/2006, 17h34
Jean-Marc.Bourguet

Citation:

Envoyé par loufoque

strncpy n'existe pas dans C++.

Table 47 p 406: <cstring> synopsis montre un strncpy.

Mais strncpy est piègeux: il ne force pas un '\0' terminal s'il ne copie que n caractères d'une part, il complète avec des '\0' si la source comporte moins de n caractères.
01/11/2006, 17h45
dj.motte

Bonjour,

Ma foi s' il faut persiter dans les char[] autant les initialier avec memset ...

Salut.
01/11/2006, 18h07
loufoque

Citation:

Table 47 p 406: <cstring> synopsis montre un strncpy.

Dans l'espace de nom global ?
01/11/2006, 19h48
sushi_

Merci pour votre aide, tout marche comme il faut !!
02/11/2006, 17h37
koala01
Salut,

Ceci dit, juste une petite question pour t'insiter à une autre réflexion...

Pourquoi est-ce que tu t'enquiquines avec des tableaux de caractères, alors que villedepart, villearrivee et message sont clairement des chaines?

La gestion des chaines de caractères est bien plus facile si l'on utilise la classe string, disponible dans l'espace de nommage std (avec inclusion de <string>)...

Les string auront, en plus, les avantages de
1. ne pas limiter le nom à 50 caractères (meme si les villes dont le nom fait plus de 50 caractères ne sont pas légion
2. ne pas monopoliser d'office 50 caractères alors que la ville peut n'en contenir que beaucoup moins (Spa, en Belgique, par exemple)
Et, si, vraiment, à un moment donné, tu devais disposer d'une chaine "C style" (tableau de caractères terminé par '\0'), tu pourrais toujours utiliser la metode c_srt() ;)

Je ne fais que donner mon avis sur la question... à toi d'y réfléchir et de décider, ou non, de t'y ralier ;)
02/11/2006, 20h19
Laurent Gomila

Parce que c'est un TP, et que la définition de la classe était donnée.

Encore un vilain prof.
02/11/2006, 20h55
koala01

Citation:

Envoyé par Laurent Gomila

Parce que c'est un TP, et que la définition de la classe était donnée.

Encore un vilain prof.

ca serait moi, je lui parlerais de la classe string... le fait de m'adresser à un prof ne m'a jamais empeché... de lui expliquer ma manière de penser ;):P

Et ca a même porté ses fruits dans certains cas :P