Discussion :

[GLDavid]

Bonjour à tous

Voici mon problème. Il y a quelques temps, j'avais écrit une librairie C++ sous Linux qui compilait sans erreurs ni warnings avec g++.
Il m'a été demandé de recompiler ce code sous Windows x86 avec Visual C++ Express 2005. Autant, j'ai de l'expérience avec g++, autant je vous avoue que coder sous Visual C++ tient pour moi de l'aventure.
Donc, j'importe mon projet sous Visual C++ et je lance la compilation. Gasp ! Que d'erreurs et warnings !!!
Par exemple, considérons la fonction suivante (qui plus est c'est une librairie faisant appel à JNI pour Java) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
JNIEXPORT void JNICALL Java_com_bioxpr_linkers_BeaconPCR_getLine(JNIEnv *env, jobject object, jint a, jobjectArray string){
    //Ceci avait été écrit avant et fonctionnait sous Linux
//const char *args[a];
//Modification pour Win mais sans doute fausse
    const char *args[] = new char[a];
    for (int i=0; i<a; i++){
        jstring jstr = (jstring)env->GetObjectArrayElement(string, i);
        args[i] = env->GetStringUTFChars(jstr, 0);
    }
    FindBeacon(a, args);
}

Dans cette méthode, je me choppe les erreurs suivantes :

Code :

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

1
2
3
 
error C2440: 'initializing' : cannot convert from 'char *' to 'const char *[]'
There are no conversions to array types, although there are conversions to references or pointers to arrays

Quelqu'un peut-il m'aider à trouver la voie ?
Merci d'avance de votre aide.

@++

[Jean-Marc.Bourguet]

Code :

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

1
2
3
4
 
JNIEXPORT void JNICALL Java_com_bioxpr_linkers_BeaconPCR_getLine(JNIEnv *env, jobject object, jint a, jobjectArray string){
    //Ceci avait été écrit avant et fonctionnait sous Linux
//const char *args[a];

Pourquoi ne pas utiliser des std::vector et des std::string?

Les VLA (des tableaux C avec une taille determinee dynamiquement), c'est du C99. Ce n'est pas du C++ meme si g++ l'offre en extension (du moins offre quelque chose de semblable).

Code :

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

1
2
//Modification pour Win mais sans doute fausse
    const char *args[] = new char[a];

Code :

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

char const** args = new char*[a];

[Jan Rendek]

Le message d'erreur est pourtant juste, car cette ligne est fausse.

Code :

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

const char *args[] = new char[a];

args est déclaré comme un tableau de pointeurs sur des chaîne de caractères (const char*[]) et tu veux lui affecter un tableau de caractères (char[])
J'imagine que ce que tu souhaites faire c'est:

Code :

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

const char *args[] = new char*[a]

Ton erreur de portage vient du fait que g++, sans autre précision te permet de déclarer des tableaux de manière 'statique' avec un nombre d'items non précisé à la compilation.
Ceci est licite en C99, mais pas en C++. Compile ta bibliothèque avec g++ et les options --pedantic --ansi, et tu verras que lui aussi trouvera une erreur dans le code original.
D'ailleurs avant de porter le code, je te conseille vivement de tout recompiler sous linux avec ces options et de corriger les eventuelles erreurs

EDIT: grillé par JMB (rapide le lundi matin ;-) )

[GLDavid]

Ok les gars.
Merci bien.
Avec vos nouvelles options pour g++, je vais reprendre mon code Linuxien et tâcher de recorriger mon code.
C'est très intéressant et je vous remercie.

@++

[Jean-Marc.Bourguet]

Code :

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

const char *args[] = new char*[a]

Ce n'est pas du C++ valide. Il faut utiliser les pointeurs explicitement en dehors des prototypes de fonctions.

[Jan Rendek]

Autant pour moi, j'ai répondu un peu vite. Cependant il me semble que dans ce cas,
c'est plutôt

Code :

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

char *const *args = new char *[a]

[loufoque]

À mon avis tu te trompes VoidSeer.

[Jean-Marc.Bourguet]

Autant pour moi, j'ai répondu un peu vite. Cependant il me semble que dans ce cas,
c'est plutôt

Code :

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

char *const *args = new char *[a]

args est alors un pointeurs vers des pointeurs constant vers des char*. De sort que j'ai comme l'impression que tu ne pourras jamais initialiser la memoire que tu alloues.

[Jan Rendek]

Vivi, désolé. J'ai lu le pb en diagonale. C'est juste que la soluce de JM ne compile pas, et que je n'ai pas pris le temps de regarder le pouquoi.
La soluce exacte est:

Code :

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

char const ** = new const char * [a];

La soluce de JM ne compile pas car il est impossible d'affecter à un char** à un const char**.
La mienne est fausse pour la raison exposée par JM.
Enfin, tout ça pour dire que les vector et les string c'est mieux.

[Jean-Marc.Bourguet]

C'est juste que la soluce de JM ne compile pas, et que je n'ai pas pris le temps de regarder le pouquoi.

Ne jamais poster sans compiler (Mais il faut avouer que le format des forums web n'aide pas , pour les newsgroups, j'ai une macro qui fait ca toute seule dans Gnus et je m'en sers pas toujours )

[Jan Rendek]

Ne jamais poster sans compiler

Je prend ça pour moi aussi, sur ce coup là...