Discussion :

[boubainfo]

Type_A to Type_B

The compiler encountered an illegal implicit conversion.

The ANSI C++ language differs from ANSI C in the treatment of void*. ANSI C allows an implicit conversion from a:

Pointer to void > to a pointer > to another object type (but not to a pointer to function type)

In C++, a void* cannot be assigned to an object of any type other than void* without an explicit cast.

The example below is legal ANSI C, but is not accepted in C++.

pour ce programme:

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
 
ifstream * ouvrirFichierEntree( int argc, char **argv);
 
ifstream * ouvrirFichierEntree( int argc, char **argv)
{
   string fichierEntree;
  ifstream fichier;
  if(argc == 1)
  {			
     cout<<"Donnez le nom du fichier: ";
     cin>>fichierEntree;
  }
  else if(argc == 2 || argc == 3  )
  {
  	 fichierEntree = argv[1];
 
  }
  else 
  {
  	 cerr << "Erreur : vous n'avez pas respecte le nombre de parametre..." << endl;	
  	 return NULL ;
  }
 
	fichier.open(fichierEntree.c_str());
 
	if (!fichier)
    {
        cerr << "Erreur : impossible d'ouvrir le fichier en lecture..." << endl;
        return NULL;
    }
 
  return fichier;
}

[Jean-Marc.Bourguet]

Tu déclares que ta fonction renvoie un pointeur vers ifstream et tu essaies de renvoyer un objet de type ifstream et non un pointeur vers ifstream. Le message d'erreur n'est pas trop clair parce que le compilateur regarde les conversions implicites. Hors il y a une conversion implicite de ifstream vers void* (celle dont tu t'es servi pour vérifier que le fichier avait bien été ouvert). Mais ce void* ne peux pas être converti implicitement non plus en pointeur vers ifstream. Le message d'erreur est rédigé dans l'optique où, plus familier avec le C qu'avec le C++, tu pensais qu'une telle conversion implicite existait.

Il y a une balise code (# dans l'éditeur pour l'introduire), utilise-là, c'est plus facile à lire.

Renvoyer un pointeur vers fichier me semble une mauvaise idée: il va falloir par après s'assurer qu'il est toujours libéré. Mais on ne peut pas retourner par valeur des streams, donc il vaut mieux écrire une fonction qui prend un stream par référence et l'initialise comme tu veux.