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.