IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

C++ Discussion :

accés Mysql en VC++ : problème de montée de mémoire


Sujet :

C++

  1. #1
    Membre à l'essai
    Profil pro
    Inscrit en
    Novembre 2006
    Messages
    17
    Détails du profil
    Informations personnelles :
    Localisation : France, Bouches du Rhône (Provence Alpes Côte d'Azur)

    Informations forums :
    Inscription : Novembre 2006
    Messages : 17
    Points : 10
    Points
    10
    Par défaut accés Mysql en VC++ : problème de montée de mémoire
    Bonjour

    Pour un de mes projets en VC++ j'utilise une classe cBD de ma fabrication pour gerer le SQL envoyé à MySql (cf la classe ci-dessous)

    Or chaque appel (en particulier des methodes query_one et query_N) de la memoire (dans le gestionnaire de taches) est consommée ... Ca augmente tout le temps jusqu'a atteindre plusieurs giga. je dois redémarrer l'appli (qui est un serveur en service) régulierement

    Je suis convaincu d'avoir une coquille lors de mes appels à mysql dans cette classe mais je ne trouve pas.

    Est ce qu'un pro pourrait jeter un oeil a mes deux methodes de ma classe et me dire ou est le pb ?


    codé en VC++ 6.0 convertir en VS2010
    MySql : 5.0.67

    Dans l'attente de votre aide..
    JFL



    -------------------------------------------------------------------
    cBD.cpp
    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
    20
    21
    22
    23
    24
    25
    26
    27
    28
    29
    30
    31
    32
    33
    34
    35
    36
    37
    38
    39
    40
    41
    42
    43
    44
    45
    46
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
    75
    76
    77
    78
    79
    80
    81
    82
    83
    84
    85
    86
    87
    88
    89
    90
    91
    92
    93
    94
    95
    96
    97
    98
    99
    100
    101
    102
    103
    104
    105
    106
    107
    108
    109
    110
    111
    112
    113
    114
    115
    116
    117
    118
    119
    120
    121
    122
    123
    124
    125
    126
    127
    128
    129
    130
    131
    132
    133
    134
    135
    136
    137
    138
    139
    140
    141
    142
    143
    144
    145
    146
    147
    148
    149
    150
    151
    152
    153
    154
    155
    156
    157
    158
    159
    160
    161
    162
    163
    164
    165
    166
    167
    168
    169
    170
    171
    172
    173
    174
    175
    176
    177
    178
    179
    180
    181
    182
    183
    184
    185
    186
    187
    188
    189
    190
    191
    192
    193
    194
    195
    196
    197
    198
    199
    200
    201
    202
    203
    204
    205
    206
    207
    208
    209
    210
    211
    212
    213
    214
    215
    216
    217
    218
    219
    220
    221
    222
    223
    224
    225
    226
    227
    228
    229
    230
    231
    232
    233
    234
    235
    236
    237
    238
    239
    240
    241
    242
    243
    244
    245
    246
    247
    248
    249
    250
    251
    252
    253
    254
    255
    256
    257
    258
    259
    260
    261
    262
    263
    264
    265
    266
    267
    268
    269
    270
    271
    272
    273
    274
    275
    276
    277
    278
    279
    280
    281
    282
    283
    284
    285
    286
    287
    288
    289
    290
    291
    292
    293
    294
    295
    296
    297
    298
    299
    300
    301
    302
    303
    304
    305
    306
    307
    308
    309
    310
    311
    312
    313
    314
    315
    316
    317
    318
    319
    320
    321
    322
    323
    324
    325
    326
    327
    328
    329
    330
    331
    332
    333
    334
    335
    336
    337
    338
    339
    340
    341
    342
    343
    344
    345
    346
    347
    348
    349
    350
    351
    352
    353
    354
    355
    356
    357
    358
    359
    360
    361
    362
    363
    364
    365
    366
    367
    368
    369
    370
    371
    372
    373
    374
    375
    376
    377
    378
    379
    380
    381
    382
    383
    384
    385
    386
    387
    388
    389
    390
    391
    392
    393
    394
    395
    396
    397
    398
    399
    400
    401
    402
    403
    404
    405
    406
    407
    408
    409
    410
    411
    412
    413
    414
    415
     
    //pragma warning(disable: 4005 4244)
    #include "cBd.h"
     
    cBd::cBd()
    {
    	this->server	= "localhost";
    	this->user		= "root";
    	this->mdp		= "";
    	this->base		= "terranota";
    }
     
    cBd::cBd(char* server,char* user,char* mdp,char* base)
    {
    	this->server = server;
    	this->user   = user;
    	this->mdp    = mdp;
    	this->base   = base;
    }
     
    cBd::~cBd()
    {
    }
     
     
    void cBd::connect(void)
    {
    	string strMessageErreur;
    MYSQL* conn;
     
    	this->connexion = mysql_init(NULL); 
    	conn = mysql_real_connect(this->connexion,this->server,this->user,this->mdp,this->base,0,NULL,0);
    	if ( conn == NULL) 
    	{ 
    		strMessageErreur.append("Probleme pour se connecter à la base de données : ");
    		strMessageErreur.append(this->base);
    		strMessageErreur.append(" (Server:");
    		strMessageErreur.append(this->server);
    		strMessageErreur.append("    user:");
    		strMessageErreur.append(this->user);
    		strMessageErreur.append(") ");
    //		strMessageErreur.append(mysql_errno(this->connexion));
    		strMessageErreur.append(":");
    		strMessageErreur.append(mysql_error(this->connexion));
    		strMessageErreur.append("\n");
     
     
    		*myLog<<'e'<<strMessageErreur<<".\n";
            ::SetEvent(hTerminateEvent);
    		throw BdException(strMessageErreur);
    	} 
    	else 
    	{ 
    		*myLog<<'s'<<"Connecté à la base '"<<this->base<<"' sur '"<<this->server<<"' avec le user '"<<this->user<<"'.\n";
    		this->serverInfos = mysql_get_server_info(this->connexion); 
     
    		if (this->serverInfos != NULL) 
    		{ 
    			*myLog<<'s'<<"Information sur le serveur:"<<this->serverInfos<<".\n";
    		} 
    		else 
    		{ 
    			strMessageErreur.append("Impossible de recuperer les informations du serveur.");
    			*myLog<<'e'<<strMessageErreur<<"\n";
    			throw BdException(strMessageErreur);
    		}
    	}
    }
     
    int cBd::query_one(CString CstrSql,MYSQL_ROW *row,my_ulonglong *id)
    {
    	MYSQL_RES*	res=NULL;
     
    	string strSql = (LPCSTR) CstrSql;
    	string strTmpSql = "";
    	string typeRequete = strSql.substr(0,6);
     
    	if(this->connexion == NULL)	this->connect();
    	CSingleLock singlelock(&mutexBD);
    	singlelock.Lock();
     
    	if(typeRequete == "INSERT")
    	{
    		// ajout dans la liste des champs
    		int intPosVALUES = strSql.find("VALUES");
    		strTmpSql = strSql.substr(0,intPosVALUES - 2);
    		strTmpSql.append(", user_creation , date_creation ");
    		strTmpSql.append(strSql.substr(intPosVALUES - 2));
    		strSql = strTmpSql;
     
    		// ajout dans les valeurs
    		int intPosFin = strSql.find_last_of(")");
    		strTmpSql = strSql.substr(0,intPosFin);
    		strTmpSql.append(", 'terraNota', now() ");
    		strTmpSql.append(strSql.substr(intPosFin));
    		strSql = strTmpSql;
    	}
    	else
    	{
    		if(typeRequete == "UPDATE")
    		{
    			int intPosSET = strSql.find("SET");
    			strTmpSql = strSql.substr(0,intPosSET + 3);
    			strTmpSql.append(" user_modification = 'terraNota', date_modification = now(), ");
    			strTmpSql.append(strSql.substr(intPosSET + 3));
    			strSql = strTmpSql;
    		}
    	}
    	*myLogSql<<'i'<<"QUERY_ONE : "<<strSql.c_str()<<"\n";
     
    	try
    	{
    		int result = mysql_query(this->connexion, (char*)strSql.c_str());
    		if(result == TELPASS_SQL_OK)
    		{
    			res = mysql_store_result(this->connexion);
    			int intNbRows = mysql_affected_rows(this->connexion);
    			*myLogSql<<'i'<<"QUERY_ONE : SUCCESS : AffectedRow : "<<intNbRows<<"\n";
    			if(intNbRows == -1)
    			{
    				// erreur dans la requete
    				char* tmp = "";
    				int intErreurNum = mysql_errno(this->connexion);
    				string strErreurMessage = mysql_error(this->connexion);
    				*myLogSql<<'e'<<"QUERY_ONE : ERREUR :"<<intErreurNum<<":"<<strErreurMessage;
    				singlelock.Unlock();
    				mysql_free_result(res);
    				return TELPASS_SQL_NOK;
    			}
    			if(intNbRows == 0)
    			{
    				// aucune ligne affectee
    				(*row) = NULL;
    				singlelock.Unlock();
    				mysql_free_result(res);
    				return TELPASS_SQL_AUCUN;
    			}
    			else
    			{
    				// N lignes affectées
    				if(typeRequete == "SELECT")
    				{
    					MYSQL_ROW tmp = mysql_fetch_row(res);
    					(*row) = tmp;
    				}
    				else
    				{
    					(*row) = NULL;
    					if(typeRequete == "INSERT")
    					{
    						 my_ulonglong tmp = mysql_insert_id(this->connexion);
    						(*id) = tmp;
    					}
    				}
    				singlelock.Unlock();
    //				mysql_free_result(res);
    				return TELPASS_SQL_OK;
    			}
    		}
    		else
    		{
    			char* tmp = "";
    			int intErreurNum = mysql_errno(this->connexion);
    			string strErreurMessage = mysql_error(this->connexion);
    			*myLogSql<<'i'<<"QUERY_ONE : ERREUR :"<<intErreurNum<<":"<<strErreurMessage;
    			(*row) = NULL;
    			singlelock.Unlock();
    			mysql_free_result(res);
    			cErreur* erreur = new cErreur();
    			switch(intErreurNum)
    			{
    				case 2013 :     *myLogSql<<'e'<<"QUERY_ONE : "<<strErreurMessage<<" : "<<strSql<<"\n";
    								erreur->mailErreur(TELPASS_ERRORMAIL_SQL,"",CstrSql,0);
    								break;
    				case 2006 :     *myLog<<'e'<<"QUERY_ONE : "<<strErreurMessage<<"\n";
    								*myLogSql<<'e'<<"QUERY_ONE : "<<strErreurMessage<<"\n";
    								::SetEvent(hTerminateEvent); 
    //								delete(erreur);
    								throw BdException(strErreurMessage);
    								break;
    			}
    //			delete(erreur);
    			return TELPASS_SQL_NOK;
    		}
    	}
    	catch(exception& e)
    	{
    		singlelock.Unlock();
     
    		throw e;
    	}
    	singlelock.Unlock();
    }
     
    int cBd::query_N(CString CstrSql,mysql_rows *rows,my_ulonglong *id)
    {
    	MYSQL_RES*	res		=	NULL;
     
    	string strSql = (LPCSTR) CstrSql;
    	string strTmpSql = "";
     
    	CSingleLock singlelock(&mutexBD);
    	singlelock.Lock();
     
    	if(this->connexion == NULL)	this->connect();
     
    	*myLogSql<<'i'<<"QUERY_N : "<<strSql.c_str()<<"\n";
    	try
    	{
    		int result = mysql_query(this->connexion, (char*)strSql.c_str());
    		if(result == 0)
    		{
    			res = mysql_store_result(this->connexion);
    			int intNbRows = mysql_affected_rows(this->connexion);
    			*myLogSql<<'i'<<"QUERY_N : SUCCESS : AffectedRow : "<<intNbRows<<"\n";
    			if(intNbRows == -1)
    			{
    				// erreur dans la requete
    				char* tmp = "";
    				int intErreurNum = mysql_errno(this->connexion);
    				string strErreurMessage = mysql_error(this->connexion);
    				*myLogSql<<'e'<<"QUERY_N : ERREUR :"<<intErreurNum<<":"<<strErreurMessage;
    				singlelock.Unlock();
    				return TELPASS_SQL_NOK;
    			}
    			if(intNbRows == 0)
    			{
    				// aucune ligne affectee
    				(*rows).RemoveAll();
    				singlelock.Unlock();
    				return TELPASS_SQL_AUCUN;
    			}
    			else
    			{
    				MYSQL_ROW row;
    				while((row = mysql_fetch_row(res)) != NULL)
    				{
    					(*rows).Add(row);
    				}
    				singlelock.Unlock();
    				return TELPASS_SQL_OK;
    			}
    		}
    		else
    		{
    			char* tmp = "";
    			int intErreurNum = mysql_errno(this->connexion);
    			string strErreurMessage = mysql_error(this->connexion);
    			*myLogSql<<'i'<<"QUERY_N : ERREUR :"<<intErreurNum<<":"<<strErreurMessage<<":"<<strSql;
    			(*rows).RemoveAll();
    			singlelock.Unlock();
    			return TELPASS_SQL_NOK;
    		}
    	}
    	catch(exception& e)
    	{
    		singlelock.Unlock();
    		throw e;
    	}
    }
    int cBd::query(CString CstrSql,int *NbRowsAffected)
    {
    	MYSQL_RES*	res		=	NULL;
     
    	string strSql = (LPCSTR) CstrSql;
    	string strTmpSql = "";
     
    	CSingleLock singlelock(&mutexBD);
    	singlelock.Lock();
     
    	if(this->connexion == NULL)	this->connect();
     
    	*myLogSql<<'i'<<"QUERY : "<<strSql.c_str()<<"\n";
    	try
    	{
    		int result = mysql_query(this->connexion, (char*)strSql.c_str());
    		if(result == 0)
    		{
    			res = mysql_store_result(this->connexion);
    			(*NbRowsAffected) = mysql_affected_rows(this->connexion);
    			*myLogSql<<'i'<<"QUERY : SUCCESS : AffectedRow : "<<(*NbRowsAffected)<<"\n";
     
    			singlelock.Unlock();
    			if((*NbRowsAffected) == -1)
    			{
    				// erreur dans la requete
    				char* tmp = "";
    				int intErreurNum = mysql_errno(this->connexion);
    				string strErreurMessage = mysql_error(this->connexion);
    				*myLogSql<<'e'<<"QUERY : ERREUR :"<<intErreurNum<<":"<<strErreurMessage;
    				return TELPASS_SQL_NOK;
    			}
    			if((*NbRowsAffected) == 0)
    			{
    				// aucune ligne affectee
    				return TELPASS_SQL_AUCUN;
    			}
    			else
    			{
    				return TELPASS_SQL_OK;
    			}
    		}
    		else
    		{
    			char* tmp = "";
    			int intErreurNum = mysql_errno(this->connexion);
    			string strErreurMessage = mysql_error(this->connexion);
    			*myLogSql<<'i'<<"QUERY : ERREUR :"<<intErreurNum<<":"<<strErreurMessage;
    			singlelock.Unlock();
    			return TELPASS_SQL_NOK;
    		}
    	}
    	catch(exception& e)
    	{
    		singlelock.Unlock();
    		throw e;
    	}
    }
     
    void cBd::test(void)
    {
    	if(this->connexion == NULL)	this->connect();
    	int retour = mysql_query(this->connexion,"select * from t_parametres");
    	fprintf(stdout,(char*) mysql_affected_rows(this->connexion));
    }
     
    CString cBd::setForSql(time_t field,time_format format)
    {
    	CString strSql = "";
    	if(field == 0)
    		strSql = "NULL";
    	else
    		strSql.Format("'%s'", MyTime::getOneTime(format,field).c_str());
    	return strSql;
    }
     
    CString cBd::setForSql(string field)
    {
    	CString strSql = "";
    	if(field.c_str() == NULL)
    		strSql ="NULL";
    	else
    	{
    		stringReplace(field,"'","''");
    		strSql.Format("'%s'",field.c_str());
    	}
    	return strSql;
    }
     
    CString cBd::setForSql(CString field)
    {
    	CString strSql = "";
    	if(field == "")
    		strSql ="NULL";
    	else
    	{
    		field.Replace("'","''");
    		strSql = "'"+field+"'";
    	}
    	return strSql;
    }
     
    CString cBd::setForSql(int field)
    {
    	CString strSql = "";
    	if(field == NULL_INT)
    		strSql ="NULL";
    	else
    		strSql.Format("%d",field);
    	return strSql;
    }
    CString cBd::setForSql(long field)
    {
    	CString strSql = "";
    	if(field == NULL_LONG)
    		strSql ="NULL";
    	else
    		strSql.Format("%l",field);
    	return strSql;
    }
    time_t cBd::getFromSql_datetime(char* field)
    {
    	if(field==NULL)
    		return NULL;
    	if(strcmp(field,"")==0)
    		return NULL;
    	if(strcmp(field,"0000-00-00 00:00:00")==0)
    		return NULL;
    	else
    		return	MyTime::setOneTime(mysql_datetime,field);
    }
    string cBd::getFromSql_string(char* field)
    {
    	return field;
    }
    CString cBd::getFromSql_CString(char* field)
    {
    	return (LPCSTR)field;
    }
    int cBd::getFromSql_int(char* field)
    {
    	return atoi(field); 
    }
    int cBd::getFromSql_int(CString field)
    {
    	return atoi(field); 
    }
    bool cBd::getFromSql_bool(char* field)
    {
    	return (atoi(field) == 1); 
    }
    bool cBd::getFromSql_bool(CString field)
    {
    	return (atoi(field) == 1);  
    }


    -------------------------------------------------------------------
    cBD.h
    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
    20
    21
    22
    23
    24
    25
    26
    27
    28
    29
    30
    31
    32
    33
    34
    35
    36
    37
    38
    39
    40
    41
    42
    43
    44
    45
    46
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
    #ifndef TELPASS_CBD_H
    #define TELPASS_CBD_H
    #define TELPASS_SQL_OK	0
    #define TELPASS_SQL_NOK	1
    #define TELPASS_SQL_AUCUN	2
     
    #define NULL_INT INT_MAX 
    #define NULL_LONG LONG_MAX 
     
    #include <afxmt.h>
    #include "Utils/Logger.h"
    #include <winsock2.h>
    #include <mysql.h>
    #include <mysqld_error.h>
    #include <errmsg.h>
    #include "Exceptions/BdException.h"
    #include "Utils/fonctions.h"
    #include "cErreur.h"
    #include <string>
    using namespace std;
     
    extern Logger* myLog;
    extern Logger* myLogSql;
    extern CMutex mutexBD;
    extern HANDLE hTerminateEvent;
     
     
    /** Format de la date
    */
    typedef enum sql_format_ {
    	SQL_DATETIME,
    	SQL_TIME,
    	SQL_DATE,
    	SQL_VARCHAR,
    	SQL_INT,
    } sql_format;
     
    typedef CArray<MYSQL_ROW, MYSQL_ROW&> mysql_rows;
     
    class cBd
    {
    	public:
    				cBd::cBd();
    				cBd::cBd(char* server,char* user,char* mdp,char* base);
    				cBd::~cBd();
    		void	cBd::connect(void);
    		int		cBd::query_one(CString strSql,MYSQL_ROW *row,my_ulonglong *id);
    		int		cBd::query_N(CString strSql,mysql_rows *rows,my_ulonglong *id);
    		int		cBd::query(CString CstrSql,int *NbRowsAffected);
    		void	cBd::test(void);
    		CString cBd::setForSql(time_t field,time_format format);
    		CString cBd::setForSql(string field);
    		CString cBd::setForSql(CString field);
    		CString cBd::setForSql(int field);
    		CString cBd::setForSql(long field);
    		time_t	cBd::getFromSql_datetime(char* field);
    		string	cBd::getFromSql_string(char* field);
    		CString	cBd::getFromSql_CString(char* field);
    		int		cBd::getFromSql_int(char* field);
    		int		cBd::getFromSql_int(CString field);
    		bool	cBd::getFromSql_bool(char* field);
    		bool	cBd::getFromSql_bool(CString field);
    	private:
    		const char* server;
    		const char* base;
    		const char* user;
    		const char* mdp;
     
    		MYSQL*	connexion;
    		const char*	serverInfos;
     
    		CMutex mutexBD;
    };
    #endif

  2. #2
    Expert éminent sénior
    Homme Profil pro
    Développeur informatique
    Inscrit en
    Février 2005
    Messages
    5 171
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 52
    Localisation : France, Val de Marne (Île de France)

    Informations professionnelles :
    Activité : Développeur informatique
    Secteur : Conseil

    Informations forums :
    Inscription : Février 2005
    Messages : 5 171
    Points : 12 293
    Points
    12 293
    Par défaut
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    //				mysql_free_result(res);

  3. #3
    Membre à l'essai
    Profil pro
    Inscrit en
    Novembre 2006
    Messages
    17
    Détails du profil
    Informations personnelles :
    Localisation : France, Bouches du Rhône (Provence Alpes Côte d'Azur)

    Informations forums :
    Inscription : Novembre 2006
    Messages : 17
    Points : 10
    Points
    10
    Par défaut trop simple, mais pas bon comme solution
    Je l'attendais celle là !!

    Si je décommente cette ligne, ma variable "row" semble ne plus rien contenu et me fait exploser l'exe ...
    Je suis conscient que cette ligne doit bien etre là et décommentée, mais alors comment faire la récupération des infos de la bases correctement pour ne rien perdre ?

  4. #4
    Expert éminent sénior
    Homme Profil pro
    Développeur informatique
    Inscrit en
    Février 2005
    Messages
    5 171
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 52
    Localisation : France, Val de Marne (Île de France)

    Informations professionnelles :
    Activité : Développeur informatique
    Secteur : Conseil

    Informations forums :
    Inscription : Février 2005
    Messages : 5 171
    Points : 12 293
    Points
    12 293
    Par défaut
    On ne résout pas un problème en le planquant.
    C'est quoi, le problème initial, avant de le planquer à coup de commentaire ?

  5. #5
    Membre émérite
    Avatar de white_tentacle
    Profil pro
    Inscrit en
    Novembre 2008
    Messages
    1 505
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Novembre 2008
    Messages : 1 505
    Points : 2 799
    Points
    2 799
    Par défaut
    Quand tu libères res, ça libère tous les objets sous-jacents. C'est donc normal que "row" soit vide, surtout que ça plante, il est libéré par le mysql_free_result.

    La bonne solution, c'est soit :
    - copier le contenu de row dans une structure à part, et renvoyer cette structure.
    - t'assurer que res est libéré après que row ait été utilisé par le client.

  6. #6
    Expert éminent sénior
    Avatar de Mat.M
    Profil pro
    Développeur informatique
    Inscrit en
    Novembre 2006
    Messages
    8 395
    Détails du profil
    Informations personnelles :
    Localisation : France, Rhône (Rhône Alpes)

    Informations professionnelles :
    Activité : Développeur informatique

    Informations forums :
    Inscription : Novembre 2006
    Messages : 8 395
    Points : 20 502
    Points
    20 502
    Par défaut
    Salut JF Léost,

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
     
    CSingleLock singlelock(&mutexBD);
    singlelock.Lock();
    peut-être il faudrait déclarer un CSingleLock en variable globale parce que le programme doit à chaque appel de fonction demander des réallocations mémoires et ressources à l'OS.
    Est-ce vraiment nécessaire d'appeler ce genre de verrouillage à chaque fois ?
    Avec SQL-Server il me semble qu'on n'ait pas à gérer ce genre de problème , SQL-Server gère en natif les verrouillages et accès concurrentiels.
    C'est ça la différence entre SQL-Server payant et MY SQL gratuit

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    extern HANDLE hTerminateEvent;
    comment gères-tu les Events ?
    Tu as peut-être un problème à ce niveau-là.
    Il ne faut pas perdre de vue que le multithreading consomme plus de ressources

  7. #7
    Expert éminent sénior
    Avatar de Mat.M
    Profil pro
    Développeur informatique
    Inscrit en
    Novembre 2006
    Messages
    8 395
    Détails du profil
    Informations personnelles :
    Localisation : France, Rhône (Rhône Alpes)

    Informations professionnelles :
    Activité : Développeur informatique

    Informations forums :
    Inscription : Novembre 2006
    Messages : 8 395
    Points : 20 502
    Points
    20 502
    Par défaut
    Citation Envoyé par jfleost Voir le message
    Je l'attendais celle là !!

    Si je décommente cette ligne, ma variable "row" semble ne plus rien contenu et me fait exploser l'exe ...
    Je suis conscient que cette ligne doit bien etre là et décommentée, mais alors comment faire la récupération des infos de la bases correctement pour ne rien perdre ?
    alors tu as un problème de synchronisation des Mutexes....
    quand tu fais un mysql_free_result(res) tu as un plantage parce que le free pointe sur une zone mémoire ou un process qui est verrouillé.

    Avec SQL_Server je doute que tu aies tous ces problèmes
    MySQL c'est bien pour les petites bases de données pas pour les bases d'entreprise

  8. #8
    Membre à l'essai
    Profil pro
    Inscrit en
    Novembre 2006
    Messages
    17
    Détails du profil
    Informations personnelles :
    Localisation : France, Bouches du Rhône (Provence Alpes Côte d'Azur)

    Informations forums :
    Inscription : Novembre 2006
    Messages : 17
    Points : 10
    Points
    10
    Par défaut
    Citation Envoyé par white_tentacle Voir le message
    Quand tu libères res, ça libère tous les objets sous-jacents. C'est donc normal que "row" soit vide, surtout que ça plante, il est libéré par le mysql_free_result.
    C'est bien ce qu'il me semblait

    Citation Envoyé par white_tentacle Voir le message
    La bonne solution, c'est soit :
    - copier le contenu de row dans une structure à part, et renvoyer cette structure.
    - t'assurer que res est libéré après que row ait été utilisé par le client.
    En effet je voudrais recopier ce que me renvoit mysql (ici tmp) dans ma variable de retour row, afin de liberer tranquille avec mysql_free_result(res);
    Mais là j'atteinds mes limites. comment recopier dans row mon tmp de type MYSQL_ROW (cad char**, tableau de tableau de caracteres) , ceci proprement et sans fuite de memoire ?

  9. #9
    Expert éminent sénior
    Avatar de Mat.M
    Profil pro
    Développeur informatique
    Inscrit en
    Novembre 2006
    Messages
    8 395
    Détails du profil
    Informations personnelles :
    Localisation : France, Rhône (Rhône Alpes)

    Informations professionnelles :
    Activité : Développeur informatique

    Informations forums :
    Inscription : Novembre 2006
    Messages : 8 395
    Points : 20 502
    Points
    20 502
    Par défaut
    Citation Envoyé par jfleost Voir le message
    En effet je voudrais recopier ce que me renvoit mysql (ici tmp) dans ma variable de retour row, afin de liberer tranquille avec mysql_free_result(res);
    Mais là j'atteinds mes limites. comment recopier dans row mon tmp de type MYSQL_ROW (cad char**, tableau de tableau de caracteres) , ceci proprement et sans fuite de memoire ?
    pourquoi pas recopier le résultat du curseur dans un CArray de classes comme le suggère White_tentacle ?
    Si par exemple tu as une table client avec nom , prenom , adresse et que tu fasses une requête SQL alors il suffit de déclarer une classe client et de recopier les résultats dans une instance de classe et de placer cela dans un CArray

    ici au lieu de passer un mysql_rows *rows
    dans
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
     
    int cBd::query_N(CString CstrSql,mysql_rows *rows,my_ulonglong *id)
    il faut passer une classe voire mieux un template C++
    Et le remplir avec
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
     
    MYSQL_ROW row;
    while((row = mysql_fetch_row(res)) != NULL)
    {
    }

  10. #10
    Membre à l'essai
    Profil pro
    Inscrit en
    Novembre 2006
    Messages
    17
    Détails du profil
    Informations personnelles :
    Localisation : France, Bouches du Rhône (Provence Alpes Côte d'Azur)

    Informations forums :
    Inscription : Novembre 2006
    Messages : 17
    Points : 10
    Points
    10
    Par défaut
    ma classe cBD est générique.
    Elle est utilisée par toutes les classes de mon projet qui veulent exécuter une requête sur mysql.
    Je ne peux donc pas faire dans cette classe la transformation car je sais pas ce que contient tmp (retour du select)

    Voila pourquoi je veux renvoyer (et je faisais) le MYSQL_ROW, qui sera interpréter différemment dans chacune de mes classes.

    Il faut donc bien que je recopie le MYSQL_ROW tmp , resultat de mysql_query dans un autre que je renverrai a ma classe appelante....
    Si quelqu'un peut me dire comment ecrire ca, je pense que je solutionnerai mon pb.

  11. #11
    Membre émérite
    Avatar de white_tentacle
    Profil pro
    Inscrit en
    Novembre 2008
    Messages
    1 505
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Novembre 2008
    Messages : 1 505
    Points : 2 799
    Points
    2 799
    Par défaut
    Citation Envoyé par jfleost Voir le message
    C'est bien ce qu'il me semblait

    En effet je voudrais recopier ce que me renvoit mysql (ici tmp) dans ma variable de retour row, afin de liberer tranquille avec mysql_free_result(res);
    Mais là j'atteinds mes limites. comment recopier dans row mon tmp de type MYSQL_ROW (cad char**, tableau de tableau de caracteres) , ceci proprement et sans fuite de memoire ?
    Je n'ai plus l'api mysql sous la main, et ma mémoire est faillible, mais je suppose que tu sauras transposer.

    En première approche (je suppose que ton appli n'est pas critique en perfs, et donc, ça pourra très bien rester comme ça), je parcourerai en parallèle :
    - la description du row renvoyé par mysql (elle contient la liste des en-têtes de colonnes)
    - la row elle-même

    et je remplirai une map<std::string,std::string>, c'est à dire un tableau associatif nom_du_champ/valeur, qui serait passée par référence en paramètre de la fonction.

    Une fois la copie faite dans la map, tu peux libérer le result_set.

    Autre chose, je vois que tu utilises l'api C, il y a aussi une api C++, elle est peut-être un poil plus simple d'approche (par contre, jamais utilisée, donc je ne sais pas s'il faut te la conseiller ).

    Sinon, Mat.M, la pub pour SQL Server, avec de mauvais argument, comment dire...

  12. #12
    Membre à l'essai
    Profil pro
    Inscrit en
    Novembre 2006
    Messages
    17
    Détails du profil
    Informations personnelles :
    Localisation : France, Bouches du Rhône (Provence Alpes Côte d'Azur)

    Informations forums :
    Inscription : Novembre 2006
    Messages : 17
    Points : 10
    Points
    10
    Par défaut
    Citation Envoyé par white_tentacle Voir le message
    je remplirai une map<std::string,std::string>, c'est à dire un tableau associatif nom_du_champ/valeur, qui serait passée par référence en paramètre de la fonction.
    Oui c'est ce que j'aurai fait (comme je peux le faire en PHP)
    Mais plus simplement, dans le cadre de ce projet, pour ne pas avoir a refaire toutes les classes appelantes, faire une copie mémoire d'un MYSQL_ROW vers un MYSQL_ROW c'est faisable ?

  13. #13
    Membre émérite
    Avatar de white_tentacle
    Profil pro
    Inscrit en
    Novembre 2008
    Messages
    1 505
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Novembre 2008
    Messages : 1 505
    Points : 2 799
    Points
    2 799
    Par défaut
    Ca ne me semble pas une bonne idée. Je ne connais pas suffisamment l'api mysql pour te dire de manière catégorique si ça marchera ou pas (globalement, ça va dépendre de la manière dont sont implémentées certaines choses), mais je pense ça ne va pas marcher.

    Et même si ça marche, tu n'as aucune garantie qu'une mise à jour ne casse pas le fait que ça marche, car tu utilises l'api d'une manière pour laquelle elle n'a pas été conçue.

    Désolé, mais je pense qu'il faut vraiment revoir cette partie de fond en comble.

Discussions similaires

  1. problème d'accès à Mysql (sur windows server 2003)
    Par laure07 dans le forum Administration
    Réponses: 3
    Dernier message: 07/09/2011, 09h34
  2. Accès mysql : problème de performance
    Par vincenlnx dans le forum Requêtes
    Réponses: 2
    Dernier message: 17/07/2007, 11h31
  3. Lenteur d'accès à MySQL via le réseau
    Par fredouille31 dans le forum Requêtes
    Réponses: 1
    Dernier message: 10/02/2006, 08h39
  4. [MySQL] Requetes imbriquées, problème de groupage
    Par cdelamarre dans le forum Langage SQL
    Réponses: 2
    Dernier message: 07/02/2006, 21h16
  5. [mysql] Toujours ce problème d'index !!
    Par LE NEINDRE dans le forum Requêtes
    Réponses: 8
    Dernier message: 12/10/2005, 17h05

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo