Discussion :

[sushi_]

Bonjour !

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

Code :

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

char var[50] ;

J'ai essayé avec

Code :

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

this->var = new char[50] ;

et

Code :

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

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 !

[loufoque]

new char[50] retourne un char*, pas un char[50].

[sushi_]

C'est ce que j'avais cru comprendre... Y a t-il un moyen de retourner un char[50] ?...

[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.

[sushi_]

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 :

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
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 :

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
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 ;
	
}

[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.

[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 :

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

1
2
strncpy(this->villeDepart, "", 50);
strncpy(this->villeArrivee, "", 50);

Et pour l'autre constructeur, il te faudra là encore des strcpy() :

Code :

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

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.

[loufoque]

strncpy n'existe pas dans C++.
Par contre il y a std::strcpy dans <cstring>

[Jean-Marc.Bourguet]

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.

[dj.motte]

Bonjour,

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

Salut.

[loufoque]

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

Dans l'espace de nom global ?

[sushi_]

Merci pour votre aide, tout marche comme il faut !!

[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

[Laurent Gomila]

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

Encore un vilain prof.

[koala01]

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

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