Adaptation Windows vers linux

Bonjour, je souhaite adapter mon programme pour qu'il puisse etre compatible pour linux.


Code Windows:

Code:

---

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 39

#include <windows.h> #include <stdlib.h> #include <stdio.h> #include <iostream> #include <time.h> #include "multithreadimpl.h" #include <cstdlib> #include <fstream> using namespace std; #include <winsock2.h> #include <string> #include <limits> #include <MYSQL/mysql.h> #define __WIN__ #include <sstream>   using namespace std;   int main(int argc, char* argv[]) //fonction principale du serveur acceptant les connections et activant les threads qui effectuent la réception { WSADATA Wsa; // int erreur = WSAStartup(MAKEWORD(2,0), &Wsa);//active le module permettant l'utilisation des sockets if(erreur!=0)//gestion d'erreur { cout<<"WSA n'a pas pu démarrer : erreur "<<erreur<<endl; return EXIT_FAILURE; } sockaddr_in mysock;//structure définissant l'IP, le port et le protocole de ce serveur SOCKET Socket;//le socket servant à la reception int size=(int)sizeof(mysock);//taille nécessaire pour la suite Socket = socket (AF_INET, SOCK_STREAM, 0);//ouvre une socket pour la reception if(Socket==INVALID_SOCKET)//gestion d'erreur { cout<<"Erreur d' initialisation de la socket!"<<endl; return EXIT_FAILURE; }   return 0; }

---

Malheureusement je n'y arrive pas, sa fait une semaine que je suis dessus.
Je vous poste le code windows car le code linux ne vaut rien.


Pouvez vous m'aider s'il vous plait?

Cordialement

Bonjour,

Je ne crois pas que winsock2 existe sous linux.
A moins qu'il existe dans wine(qui sert justement a émuler windows sous linux).
Sinon il va falloir utiliser une autre bibliothèque réseau. Perso j'utilise boost.asio qui marche très bien.

Citation:

---

Envoyé par Nogane

Perso j'utilise boost.asio qui marche très bien.

---

Et Boost::Asio est portable sous Windows et sous Linux. Donc le problème de portabilité est réglé

J'ai pas lu le code, mais un truc m'a sauté aux yeux : C'est généralement une très mauvaise idée d'avoir des #include après un using namespace.

Citation:

---

Envoyé par JolyLoic

J'ai pas lu le code, mais un truc m'a sauté aux yeux : C'est généralement une très mauvaise idée d'avoir des #include après un using namespace.

---

Je pense qu'il s'agit d'une erreur de copier/coller parce qu'il y a même deux "using namespace std;", ce qui n'a pas bcp de sens...

Comme il a été dit, adapter du code "natif" vers une autre plate-forme (ce qu'est bel et bien du code Windows basé sur Winsock), c'est rarement possible sans passer par une couche "lourde", type Wine ou Cygwin, qui simule / émule un environnement plus ou moins complet.

En général, la meilleure manière de parvenir à porter un tel code, c'est soit :

• Le compiler tel quel pour un environnement d'émulation, donc Wine dans ton cas.
• Réécrire le code pour utiliser des éléments déjà connus comme étant portables, ce qui sous-entend d'utiliser des fonctions de librairies portables exclusivement et non plus des fonctions natives.

Salut,
Ton code risque de susciter beaucoup de commentaire... Cependant le portage n'est pas si colossal qu'on pourrait le penser. Donc allons à l'essentiel.
Tu as les points suivants qui vont te poser problème dans le portage :
1/ Les en-têtes

Code:

---

1 2	#include <windows.h> #include <winsock2.h>

---

Ces deux en-têtes sont spécifiques à la windows. Donc il faut ne les inclure que sous directive de compilation :

Code:

---

1 2 3 4 5 6 7 8

#ifdef WIN32 #include <windows.h> #endif // et plus loin : #ifdef WIN32 #include <winsock2.h> #endif

---

2/ Socket :
Une solution sans soucis : -> Passer sur Boost.Asio
Une solution par rapport à ton code :

Code:

---

1 2	WSADATA Wsa; // int erreur = WSAStartup(MAKEWORD(2,0), &Wsa);//active le module permettant l'utilisation des sockets

---

et

Code:

---

WSACleanup();//fonction symbolique (car normalement à un WSASartup doit être associé un WSACleanup mais bon celui là ne sera jamais utilisé :x...)

---

Ceux sont aussi des appels spécifiques à Microsoft :

Code:

---

1 2 3 4 5

#ifdef WIN32 WSADATA Wsa; // int erreur = WSAStartup(MAKEWORD(2,0), &Wsa);//active le module permettant l'utilisation des sockets #endif

---

et

Code:

---

1 2 3 4

#ifdef WIN32 WSACleanup();//fonction symbolique (car normalement à un WSASartup doit être associé un WSACleanup mais bon celui là ne sera jamais utilisé :x...) #endif

---

En revanche, tous les appels suivants (socket, recv, send, etc...) m'ont l'air d'être du socket BSD. Donc à priori tu peux les garder tels quels. Je ne sais pas quels sont les en-têtes sous plateforme linux pour les sockets BSD. Peut être ceux-ci :

Code:

---

1 2	#include <sys/types.h> #include <sys/socket.h>

---

3/ La création du thread :
Rien de standard n'existe (pour l'instant) sur les threads. Donc CreateThread ne compilera pas sous linux.
Je te conseille de commencer par séparer les parties thread spécifiques d'une plateforme du traitement que tu souhaites faire. Vu que ton code est en C avec une légère teinte C++, je vais reprendre ta démarche.
Je commencerais par créer une fonction :

Code:

---

1 2 3 4 5

unsigned long TraitementServeur(ServeurMulti*serveur) { // tout le code actuel de ServeurThread est recopié ici. }

---

Une fois ceci fait :
Une solution sans soucis : passer par Boost.Thread
Une solution possible avec ton code :

ServeurThread devient :

Code:

---

1 2 3 4 5 6 7

#ifdef WIN32 DWORD WINAPI ServeurThread(LPVOID para) { return TraitementServeur(reinterpret_cast<ServeurMulti*>(serveur)); } #endif

---

Sur linux, je ne sais pas ce qu'il existe pour les threads. Tu peux te baser sur les threads posix

Code:

---

1 2 3 4 5 6 7 8

#ifndef WIN32 void *pth_ServeurThread(void*para) { TraitementServeur(reinterpret_cast<ServeurMulti*>(serveur)); return NULL; } #endif

---

Ensuite, l'appel à CreateThread est remplacé :

Code:

---

1 2 3 4 5 6 7

#ifdef WIN32 CreateThread(NULL,NULL,ServeurThread, (LPVOID) pointeur, NULL, NULL);//démarrage du thread de reception des données #else pthread_t pth_thread; pthread_create( &pth_thread, NULL, pth_ServeurThread, (void*)pointeur); #endif

---

Les en-têtes doivent être :

Code:

---

1 2 3 4 5 6

#ifdef WIN32 #include <winsock2.h> #else #include <pthread.h> #endif

---

4/MySQL : je ne connais pas donc je ne sais pas s'il existe des problèmes de portabilité.

Ces éléments sont une piste pour porter rapidement ton code. Notes que le code que tu nous montres risque de poser beaucoup de problèmes et qu'en entreprise, beaucoup de chef de projet pourrait te reprocher un tel code. Si tu as du temps, il serait intéressant de le refactorer si possible en choisissant une approche plus C++ ou alors complètement C - là tu as le c.l entre 2 chaises et ça peut irriter.
Hope it helps !

Merci a tous de votre aide.


Surtout a toi 3DArchi.

Je viens de faire se que tu ma conseillé, mais quelques erreurs subsistent.

Citation:

---

expected primary-expression before ',' token

---

Code:

---

pthread_create( &pth_thread, NULL, pth_ServeurThread, (void*)pointeur);

---

Citation:

---

'serveur' was not déclared in this scope

---

Code:

---

TraitementServeur(reinterpret_cast<ServeurMulti*>(serveur));

---

Cordialement

Citation:

---

Envoyé par thanks33

Code:

---

pthread_create( &pth_thread, NULL, pth_ServeurThread, (void*)pointeur);

---

---

As tu déclaré (prototype) pth_ServeurThread ?
Si tu écris comme ci-dessous, quelle ligne est en erreur ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#ifdef WIN32 CreateThread(NULL,NULL,ServeurThread, (LPVOID) pointeur, NULL, NULL);//démarrage du thread de reception des données #else pthread_t pth_thread; pthread_create( &pth_thread, NULL, pth_ServeurThread, reinterpret_cast<void*>(pointeur) ); #endif

---

Citation:

---

Envoyé par thanks33

Code:

---

TraitementServeur(reinterpret_cast<ServeurMulti*>(serveur));

---

---

Erreur de copier/coller de ma part (mais tu aurais du pouvoir corriger :?)

Code:

---

1 2 3 4 5 6 7

#ifndef WIN32 void *pth_ServeurThread(void*para) { TraitementServeur(reinterpret_cast<ServeurMulti*>(para)); return NULL; } #endif

---

Code:

---

1 2 3 4 5 6 7 8 9

CreateThread(NULL,NULL,ServeurThread, (LPVOID) pointeur, NULL, NULL);//démarrage du thread de reception des données #else pthread_t pth_thread; pthread_create( &pth_thread, NULL, Erreur est ici>>>>> pth_ServeurThread, reinterpret_cast<void*>(pointeur) );

---

C'est vrai que la seconde erreur j'aurai pu la trouvé>>> aucune excuse =)


Cordialement

Citation:

---

Envoyé par thanks33

Code:

---

1 2 3 4 5 6 7 8 9

CreateThread(NULL,NULL,ServeurThread, (LPVOID) pointeur, NULL, NULL);//démarrage du thread de reception des données #else pthread_t pth_thread; pthread_create( &pth_thread, NULL, Erreur est ici>>>>> pth_ServeurThread, reinterpret_cast<void*>(pointeur) );

---

---

Citation:

---

Envoyé par 3DArchi

As tu déclaré (prototype) pth_ServeurThread ?

---

Utiliser une lib portable de plus au niveau aurait été beaucoup plus simple et propre, non ? Ce que vous faites là, c'est réinventer la roue.
Le jour où tu voudras porter ton code sous une autre plateforme, tu seras bon pour recommencer.

Citation:

---

Envoyé par Florian Goo

Utiliser une lib portable de plus au niveau aurait été beaucoup plus simple et propre, non ? Ce que vous faites là, c'est réinventer la roue.
Le jour où tu voudras porter ton code sous une autre plateforme, tu seras bon pour recommencer.

---

On est entièrement d'accord. Les pistes proposées le sont que dans le cadre de son existant 'à minima'. Mais, Boost.Asio et Boost.Thread seraient de bons points de départs pour du plus long terme.

oui je l'ai déclaré comme tu ma dit :

Code:

---

1 2 3 4 5

void *pth_ServeurThread(void*para) { TraitementServeur(reinterpret_cast<ServeurMulti*>(serveur)); return NULL; }

---

Cordialement

Nan. Là tu l'as défini. Mais il faut la déclarer avant de l'utiliser. Comme tu as du le faire pour ServeurThread. Qu'il y a-t-il dans "multithreadimpl.h" ?

Puisque c'est aussi le sujet, j'ai mis quelques liens de librairies portables dans ce message.
Notamment une librairie pthread sous Win32, qui marche plutôt bien.

j'ai mis dans le .h ceci :

Code:

---

1 2 3 4 5 6

struct ServeurMulti{ //La structure envoyée comme paramètre à SOCKET myfarsocket; //le socket qui a demandé la connection sockaddr_in sin; //la structure définissant l'addresse IP, le port, le protocole du socket distant };   DWORD WINAPI ServeurThread(LPVOID para);//prototype permettant le multi-thread de reception des données

---

Voila j'ai tous redéfini ya plus erreur.

mais voici se que le compilateur dit :

Code:

---

undefined reference to 'pthread_create'

---

Cordialement

Puisqu'il semble qu'il faut te tenir le stylo :? :

Code:

---

1 2 3 4 5 6 7 8 9 10

struct ServeurMulti{ //La structure envoyée comme paramètre à SOCKET myfarsocket; //le socket qui a demandé la connection sockaddr_in sin; //la structure définissant l'addresse IP, le port, le protocole du socket distant }; #ifdef WIN32 DWORD WINAPI ServeurThread(LPVOID para);//prototype permettant le multi-thread de reception des données #else void *pth_ServeurThread(void*para); #endif

---

Citation:

---

Envoyé par thanks33

j'ai mis dans le .h ceci :

Code:

---

1 2 3 4 5 6

struct ServeurMulti{ //La structure envoyée comme paramètre à SOCKET myfarsocket; //le socket qui a demandé la connection sockaddr_in sin; //la structure définissant l'addresse IP, le port, le protocole du socket distant };   void *pth_ServeurThread(void*para);//prototype permettant le multi-thread de reception des données

---

Voila j'ai tous redéfini ya plus erreur.

mais voici se que le compilateur dit :

Code:

---

1 2 3 4 5 6 7

pthread_t pth_thread; pthread_create( &pth_thread, NULL, pth_ServeurThread, reinterpret_cast<void*>(pointeur) Erreur est ici>>>>> );

---

Citation:

---

undefined reference to 'pthread_create'

---

Cordialement

---

Merci encore pour ton aide

Cordialement

As-tu rajouté #include <pthread.h> ?

oui il y est, c'est la première chose que j'ai regardé.


je ne comprend pas pourquoi il me sort cette erreur.

C'est un problème à l'édition de lien, plutôt, non ?
Il manque pas un -lpthread dans la ligne de compilation ?

C'est une erreur de link . Il faut que tu ajoutes dans les options de link la librairie pthread. Sous MinGW, il s'agit libpthread.a. Cette librairie dépend de ton compilateur et la façon de l'ajouter de ton environnement de développement. Je suis sur que tu vas t'en sortir ;)
[Grilled:aie:]

merci a tous de votre aide.

effectivement c'est le link qui manquer



Cordialement

j'ai un nouveau problème.


Cette fois si sur le build de ma soket.

Code:

---

1 2 3 4 5 6 7 8 9 10 11

sockaddr_in mysock;//structure définissant l'IP, le port et le protocole de ce serveur SOCKET Socket;//le socket servant à la reception int size=(int)sizeof(mysock);//taille nécessaire pour la suite Socket = socket (AF_INET, SOCK_STREAM, 0);//ouvre une socket pour la reception if(Socket==INVALID_SOCKET)//gestion d'erreur { cout<<"Erreur d' initialisation de la socket!"<<endl; return EXIT_FAILURE; }   }

---

Je ne trouve pas pourquoi sa marche pas, en plus je n'ai aucun message erreur a la compilation.

Cordialement

Code:

---

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 39 40 41 42

sockaddr_in mysock;//structure définissant l'IP, le port et le protocole de ce serveur #ifdef WIN32 SOCKET Socket;//le socket servant à la reception #else int Socket;//le socket servant à la reception #endif int size=(int)sizeof(mysock);//taille nécessaire pour la suite Socket = socket (AF_INET, SOCK_STREAM, 0);//ouvre une socket pour la reception #ifdef WIN32 if(Socket==INVALID_SOCKET)//gestion d'erreur #else if(Socket==-1)//gestion d'erreur #endif { cout<<"Erreur d' initialisation de la socket!"<<endl; switch(errno) { case EPROTONOSUPPORT : std::cout<<"Le type de protocole, ou le protocole lui-même n'est pas disponible dans ce domaine de communication. "<<std::endl; break; case ENFILE : std::cout<<"La table des descripteurs par processus est pleine. "<<std::endl; break; case EMFILE : std::cout<<"La table des fichiers est pleine. "<<std::endl; break; case EACCES : std::cout<<"La création d'une telle socket n'est pas autorisée. "<<std::endl; break; case ENOBUFS ou ENOMEM : std::cout<<"Pas suffisament d'espace pour allouer les buffers nécessaires. "<<std::endl; break; case EINVAL : std::cout<<"Protocole inconnu, ou famille de protocole inexistante. "<<std::endl; break; default : std::cout<<"Erreur n° "<<errno<<" hexa : "<<std::hex<<errno<<std::endl; break; } return EXIT_FAILURE; }

---

Ca donne quoi comme trace ?

en faite je viens de résoudre le problème mais merci de ton aide 3DArchi.

Par contre quand je reçois l'adresse ip du programme serveur elle s'avère erroné.

Code:

---

cout<<"Connection d'un ordinateur au serveur a l'adresse : "<<inet_ntoa(Oaaddr.sin_addr)<<endl;

---

Je reçois 1.0.0.0 au lieu de 192.168.0.1.

Cordialement