Discussion :

[darkwall_37]

Bonjour,

Je me fais insulter par le compilateur parce que je fais quelque chose de travers mais je ne comprends pas pourquoi.

Voici le code

File.h

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
#ifndef DEF_FILE
#define DEF_FILE
 
#include <iostream>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
class File
{
    public:
 
    File( const char *path );
    FILE *open( const char *path, char *mode);
    unsigned int getFileLength(const char *path);
 
 
    private:
 
    FILE *file;
    char *path;
};
 
#endif

File.cpp

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
#include "File.h"
 
 
using namespace std;
 
 
File::File(const char *_path)
{
    strcpy( path, _path);
}
 
 
FILE *File::open(const char *path, char *mode)
{
    FILE *ptr = fopen( path, mode ) ;
 
    if ( ptr != NULL )
    {
        return ptr;
    }
}
 
unsigned int File::getFileLength(const char *path)
{
    char character;
    unsigned int counter;
 
    file.open(path, "r");
 
    while ( fscanf(file, "%c", &character) != EOF )
    {
        counter++;
    }
 
    fclose(file);
 
    return counter;
}

Il braille ici ...

Code :

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

file.open(path, "r");

... m'insultant de ceci :

Code :

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

request for member 'open' in '((File*)this)->File::file', which is of non-class type 'FILE*'|

Pourtant la méthode open est bien définie pour l'attribut file dans la classe File. Quelque chose m'échappe

Merci d'avance.

[skeud]

file est un pointeur tu dois donc acceder a ces methode comme ceci:

Code :

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

file->open(path,"r");

Edit: pan mauvaise réponse j'ai regarder l'erreur et tellement habituer a avoir cette erreur que je donne une solution pour l'erreur mais qui en génere une autre

[gl]

Code :

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

file.open(path, "r");

... m'insultant de ceci :

Code :

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

request for member 'open' in '((File*)this)->File::file', which is of non-class type 'FILE*'|

Pourtant la méthode open est bien définie pour l'attribut file dans la classe File. Quelque chose m'échappe

L'attribut file est un FILE* pas un File (ni un File*). Or la méthode open que tu as fait est un méthode de File pas de FILE. Tu ne peux donc pas appeler file.open(), par contre il faut bel et bien récupérer le résultat de open() dans file. Je suppose donc que tu voulais plutôt écrire file=open().

A part ça, deux-trois remarques:

• Dans la méthode open() en cas d'échec (fopen() renvoie NULL), le retour de la fonction vaut n'importe quoi et tu vas utiliser un pointeur invalide ensuite.
• J'ai du mal à voir l'intérêt d'utiliser FILE plutôt que les flux C++ ici.
• En C++, les headers C sont niommé cXXXX pas XXXX.h (cstdio à la place de stdio.h, etc.).
• Tu te trimballes plusieurs headers inutiles dans File.h, par contre il en manque dans File.cpp.

[3DArchi]

Salut,

A part ça, deux-trois remarques:

Aux remarques pertinentes de gl, je rajouterais :

• l'utilisation de std::string plutôt que char *
• ça évitera de copier des données dans path sans l'avoir alloué
• Pourquoi avoir un membre path et le passer en paramètre de toutes tes fonctions ?
• éviter les noms commençant par souligné '_' : c'est un schéma de nom laissé aux compilateurs
• si ce n'est pas un exercice, utilises les flux de la STL
• même si c'est un exercice, utilises std::string car une classe ne doit gérer qu'une ressource et à ce moment ne faire que ça. Or tu as deux ressources : char* et FILE*.

[darkwall_37]

Ah oui exact ^^ Merci pour l'aide et les remarques, le code est grossier je le sais désolé pour les puristes ^^

En fait je continue de coder en C en parallèle alors je préfère continuer à manipuler les chaines ainsi et pour le reste aussi car je ne veux pas oublier.

[oodini]

Très mauvaise démarche...