Discussion :

[azrael88370]

Bonjour, je veux ouvrir un fichier en lecture, et j'ai vu qu'on pouvait le faire avec CFile, CStdioFile, et ifstream.
Comme je suis avec VC et les MFC, je me suis dis qu'il serai mieux de se servir de CFile plutot que ifStream, mais la, probleme !

Voila mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
bool CBulletinDlg::SvgDonnees(CString sFichier)
{   
    bool ok = false;
    //ouverture du fichier
    CFile myFile;
 
    if ( myFile.Open( sFichier, CFile::modeRead ))
    {
        ok = true;
    }
    return ok;
}

et voila l'erreur que le debuggeur me met :

Code :

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

1
2
 
Error	1	error C2440: '=' : cannot convert from 'CString' to 'char *'

Alors, que faire ?

[farscape]

salut,
tu es sur que l'erreur rapportée correspond au code montré ?
je ne vois pas d'affectation dans ton code .

[GP77]

Essaie :

Code :

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

1
2
 
if ( myFile.Open( &sFichier, CFile::modeRead ))

La fonction demande un LPCTSTR...

[azrael88370]

C'est au niveau de :

Code :

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

1
2
 
if ( !myFile.Open( sFichier, CFile::modeRead ))

la variable sFichier, il ne veut pas la convertir en char* et quand je le fait manuellement avant
(char*)sFichier
,il me met la meme erreur, c'est null

[azrael88370]

je viens de mettre la petite esperluette '&', et la, voila l'erreur :

Code :

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

1
2
 
error C2664: 'CFile::Open' : cannot convert parameter 1 from 'CString *__w64 ' to 'LPCTSTR'

[azrael88370]

J'ai trouvé la solution, il fallait que je convertisse mon CString en char* :

Code :

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

1
2
3
4
5
6
7
8
 
    char* nom_fic = (char*)LPCTSTR(strFichier); 
    CFile myFile;
 
    if ( !myFile.Open( nom_fic, CFile::modeRead ))
        MessageBox("impossible d'ouvrir le fichier");
    else
        MessageBox("Ouverture ok");

merci pour le coupe de main

[farscape]

ce n'est absolument pas normal ,Open accepte en argument un objet CString .

Code :

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

1
2
3
4
5
6
7
8
9
 
bool ok = false;
    //ouverture du fichier
    CFile myFile;
	CString sFichier="toto";
    if ( myFile.Open( sFichier, CFile::modeRead ))
    {
        ok = true;
    }

ce code compile sans probleme.
tu dois etre en Unicode ....

[Médinoc]

Non, on ne convertit pas en char *.
En char const peut-être, mais le char * viole la sécurité de la classe.

D'ailleurs, CFile::Open() prend un LPCTSTR en paramètre, donc un TCHAR const *



Si ça se trouve, ton erreur résidait uniquement dans le fait que tu aies fait quelque part un char* nom_fic = strFichier.

Pour éviter ce genre de problèmes, conseil n°1: Apprenez à bien utiliser const!
et conseil n°2 : Utiliser les opérateurs de conversion du C++! Si vous avez besoin d'un const_cast, c'est très souvent parce que vous avez fait une erreur.