SERVEUR MULTITHREAD SOUS LINUX/WINDOWS[MYSQL][VC++/DEVC++/G++]

Commentaire de NitRic le 25/10/2006 17:58:22

CBridge::CBridge(int NbClients)
{
  // On utilise « this », pas le nom de la classe ...
  // j'ai vu cette « chose » dans plusieurs de tes classes ...
  CBridge::NbClients = NbClients;
}
// modifié
CBridge::CBridge(int NbClients) : this->NbClients(NbClients)
{
  // nothing
}

pour faire vraiment C++ et non un mélange de C et de C++, pourquoi ne pas utiliser les *stream pour les fichiers?

dans ton fichier CConfiguration.h, c'est quoi ca?
    public:
        CConfiguration::CConfiguration();

pourquoi mettre les attributs de tes classes en public? autant y aller en C avec des structure, aucune différence ... dans ton cas, l'OO n'a pas lieux d'être, l'encapsulation/etc. => au poubelle dans ton cas ...

*** Exemple
class Object
{
  public:
    Object() : m_data(0)
    {
      // nothing
    }
    ~Object()
    {
      // nothing
    }

    int getData() const
    {
      return this->m_data;
    }
    void setData(const int data) // « const » est optionnel(c'est par copie de toute facon)
    {
      this->m_data = data;
    }

  private:
    int m_data;
};

tu devrais utiliser <> pour les headers « standards » et ceux du OS, tels que <windows.h>/<string.h>/etc. et garder les "" pour tes headers. au moins avec les <> on se pose pas trop de question :: c'est un header « standard/OS » ou un perso. !? :: réponse: std/OS, c'est automatique quand on le voit ...

en passant, en C++, c'est pas string.h/stdio.h/etc. mais plutôt cstring/cstdio/cstdlib/etc. => namespace

sous Windows, utilises plutôt <winsock2.h> et non celui de <windows.h> car celui de <windows.h> c'est la version 1(qui est préhistorique maintenant), alors préfères la version 2, pour être à jour ...

#include <winsock2.h> // inclut aussi <windows.h> et ce dernier devient donc optionnel
#include <windows.h> // optionnel si on utilise/inclut <winsock2.h>

tu peux aussi définir WIN32_LEAN_AND_MEAN avant l'inclusion de <windows.h> pour éviter l'inclusion de la version 1 de winsock mais avec <winsock2.h>, t'as rien à faire, sauf inclure ce fichier !

dans ceci: bool CDataQuery::GetAuthentification(char* User, char* Pass, CConnection* pClient)
plutôt que d'avoir 10,000 else, pourquoi ne pas utiliser une variable(bool)? tu retournes plus souvent false que true alors mets false par défaut(à l'initialisation) et changes pour true si besoin, comme ca, t'aura juste UN ->Free(...) au lieux de 20(même si le compilateur devrait pouvoir s'arranger pour mettre ca « correcte » derrière) ...
  bool retval = false;

  if ( cond1 )
  {
    if ( cond2 )
    {
      if ( cond3 )
      {
        retval = true;
      }
    }
    else {
      retval = true;
    }
    ->Free(...);  // juste UN Free()
  }
  return retval;

et tu pourrais utiliser cette méthode dans plusieurs de tes classes/méthodes/...

std::string ne t'interesse pas? très utile pourtant ...

en passant, en utilisant les *stream, dans ton CLog tu pourrais même surcharger l'opérateur << pour écrire dans ton fichier, CLog l;  l << "ceci";  // par exemple
y'a pleins de choses que tu pourrais faire mais que t'as pas fait :P

en C++, autant y aller en C++/OO plutôt que C++/C
le C++ est un C OO alors utilises l'OO sinon retournes en C :}

les librairies, ajoutes les dans ton makefile ou dans les options du projet mais laisses tomber le #pragma, pour connaître les librairies utilisés/liés/.. à ton projet, faut voir dans les fichiers sources/options/etc., un endroit c'est plus ... simple ... pour ce qui est de la configuration(comme pour les libs/...) on fait pas ca dans les fichiers .c/.cpp/.h/.hpp/etc. mais dans les makefiles/options du projet/etc., c'est global au projet et pas spécifique a un fichier(il y a quand même des exceptions parfois) ...

c'est ce que j'ai remarqué des fichiers que j'ai regardé :}

bref voilà, ce sont mes commentaires, dans l'ensemble ca l'air d'être un bon projet(je l'ai pas testé/essayé), bonne continuation :}