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.