Discussion :

[Patchanka]

Bonjour à tous,

Je découvre Visual C++ aujourd'hui avec un programme qui n'avait aucun soucis de compilation sous Linux (compilateur g++). Visual a visiblement un problème avec l'allocation d'un tableau (in_file) qui doit permettre d'accéder à une série de fichiers.

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
// Juste pour info ---
char file_name[1024];
char** file_name_in;
file_name_in = (char**) malloc(max_index * sizeof(char*));
for (ind = 0; ind < max_index; ind++)
  {
    file_name_in[ind] = (char*) malloc(1024 * sizeof(char));
    if (!file_name_in[ind])
      printf("allocation failure for filename array %s",file_name_in[ind]);
      fscanf(fp_feature, "%s", file_name_in[ind]);
  }
//---
FILE * in_file[max_index]; // ligne 538 qui pose problème
for (ind = 0; ind < max_index; ind++)
  {
    sprintf(file_name,"%s", file_name_in[ind]);
    if ((in_file[ind] = fopen(file_name, "rb")) == NULL) // ligne 545
      printf("Could not open input file: %s", file_name);
  }

Sachant que :
- max_index est un int lu prédédemment dans un fichier
- feature_filename est un fichier qui contient les noms complets vers des fichiers de "features"

Quand je compile, j'obtiens les erreurs suivantes :

(538) : error C2057: expression constante attendue
(538) : error C2466: impossible d'allouer un tableau de taille constante 0
(538) : error C2133: 'in_file'*: taille inconnue

Du coup, j'ai tenté une autre façon d'allouer mon tableau in_file :

Code :

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

1
2
FILE * in_file;
in_file = (FILE *) malloc(max_index * sizeof(FILE));

ce qui me génère ces nouvelles erreurs :

(545) : error C2679: '=' binaire*: aucun opérateur trouvé qui accepte un opérande de partie droite de type 'FILE *' (ou il n'existe pas de conversion acceptable)
1> c:\program files\microsoft visual studio 8\vc\include\stdio.h(68): peut être '_iobuf &_iobuf::operator =(const _iobuf &)'
1> lors de la tentative de mise en correspondance de la liste des arguments '(FILE, FILE *)'

(583) : error C2664: 'fseek'*: impossible de convertir le paramètre 1 de 'FILE' en 'FILE *'
1> Aucun opérateur de conversion définie par l'utilisateur disponible qui puisse effectuer cette conversion, ou l'opérateur ne peut pas être appelé
1>e:\svm\src\pspsvm.cpp(584) : error C2664: 'fread'*: impossible de convertir le paramètre 4 de 'FILE' en 'FILE *'
1> Aucun opérateur de conversion définie par l'utilisateur disponible qui puisse effectuer cette conversion, ou l'opérateur ne peut pas être appelé

La ligne 583 est la suivante :

Code :

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

fseek(in_file[ind],sizeof(float)*i,SEEK_SET);

Quelqu'un saurait me dire comment faire pour que ça compile correctement ?

[Patchanka]

Finalement, j'ai modifié mon code pour ne plus avoir à utiliser file_in comme un tableau, et ça compile. Ca s'exécute même plutôt correctement !

Par contre, je ne m'explique pas pourquoi ça fonctionnait tel quel sous Linux...

Enfin, désolée du dérangement !

[Médinoc]

La réponse est simple: On ne peut pas faire un tableau de FILE, vu que rien n'utilise les FILE par valeur.
fopen() retourne un FILE* et non pas un FILE, donc si tu veux un tableau, tu dois utiliser un tableau de FILE*.
...de plus, le nombre de fichiers ouvrables simultanément par un processus est limité.

[3DArchi]

La réponse est même encore plus simple. Les tableaux statiques en Visual nécessitent que la taille soit connue à la compilation :

Code :

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

1
2
3
4
5
 
MonType mon_tableau[3]; // OK
 
int ma_taille(recuperer_ma_taille());
MonType mon_tableau[ma_taille]; // KO

La solution : passer par un std::vector ou utiliser l'allocation dynamique.

[Médinoc]

Pas seulement pour Visual, tu sais: C'est le standard C++ (ainsi que C90) qui veut ça.

Ensuite, il n'est peut-être pas normal que tu compiles en C++, car ce code ressemble fort à un code C (qui nécessiterait le standard C99 en raison des déclarations de variable en milieu de bloc)...

[3DArchi]

Pas seulement pour Visual, tu sais: C'est le standard C++ (ainsi que C90) qui veut ça.

Sans l'option -pedantic, gcc accepte ce genre de tableau.

Ensuite, il n'est peut-être pas normal que tu compiles en C++, car ce code ressemble fort à un code C (qui nécessiterait le standard C99 en raison des déclarations de variable en milieu de bloc)...

Je ne suis pas le PO. Effectivement, tous dans ce code ressemble à du C

[Patchanka]

si tu veux un tableau, tu dois utiliser un tableau de FILE*.

Les tableaux statiques en Visual nécessitent que la taille soit connue à la compilation. La solution : passer par un std::vector ou utiliser l'allocation dynamique.

C'est bien ce que je me disais, vu les messages d'erreur que j'obtenais !

Effectivement, tous dans ce code ressemble à du C

Et, euh... c'est grave si j'utilise un code construit quasi comme en C, mais avec une extension C++ ? Pour info, mon code utilise un autre code qui, lui, est vraiment du C++ ^^"

[3DArchi]

Et, euh... c'est grave si j'utilise un code construit comme en C, mais avec une extension C++ ?

Mélanger les deux langage à au moins deux conséquences à mon avis néfastes :
1/ Le risque de bug est très accru.
2/ La maintenance est rendue plus ardu.
Dans un contexte professionnel, je te demanderais de choisir : C ou C++.
La seule raison valable est le cas où tu dois interfacer du code C++ (resp. C) avec une bibliothèque offrant une interface en C (resp C++). Auquel cas, j'isolerais le mélange dans une enveloppe adéquate.

[Patchanka]

Ok, je vais tâcher de faire attention, alors. J'utilise une bibliothèque C++ avec mon code. De toute façon, je vais devoir remanier pas mal mon code, alors je vais essayer de faire ça proprement en tenant compte des spécificités du C++.

Merci pour toutes vos précisions en tout cas !