Discussion :

[chaya]

Bonjour,
je rencontre un petit probleme ds mon programme.
Je m'apercois que lorsque je l'execute en boucle il plante au bout de quelques minutes (3-4 tours de programmes) (erreur windows, il a decider de terminer le prog etc..)
Par contre lorsque je le lance sans faire ma boucle , c'est a dire qu'il se deroule une seule fois => tout tourne impeccable.

Le programme a la base est fait pour tourner en boucle (il releve des temps de réponse reseau).


j'ai des soupcons sur une partie de code presente ds ma boucle qui a pour but de convertir un string en char :

Code :

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

1
2
3
4
                       size = affiche.size() + 1;   // la variable 'affiche' est un string
                       buffer = new char[ size ];
                       strncpy( buffer, affiche.c_str(), size );

le fait de repeter cette partie de code ds un while peut elle faire planter mon programme ?

si vous preferez voir le code en entier dite le moi (300 lignes)

[ketchupi]

Ton erreur est peut-être du au fait que tu stockes de la mémoire de plus en plus grande sans libérer celle déjà utilisée !
Mieux vaudrait libérer de la mémoire avec un delete avant de réappeler la fonction new !

[nikko34]

Pour chaque new, il faut un delete.

Sinon, as tu vraiment besoin de faire ça?

[frizou11]

Exactement ! un new doit toujours correspondre à un delete.
En Fac, un prof avait créé un parser pour nos sources et pour chaque fois où un new ne correspondait pas avec un delete on avait un point en moins... je t'assure que ça aide à bien programmer !

[chaya]

oui c'est ce que je pensais pour les deletes, je fait sa je le fait tourner et je vous tiens au courant.

[chaya]

j'ai rajouté les deletes appropriés aux variables que je declarait ds ma boucle. et le programme plante toujours de la meme facon : programme.exe a rencontré une erreur et doit fermer etc ....

des courageux veulent voir le code en entier ? il est assez simple a lire mais fait 300 lignes

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
#include <winsock.h>
#include <windows.h>
#include <iostream>
#include <string.h>
#include <sstream>
#include <mysql/mysql.h>
#include <stdio.h>
#include <stdlib.h>

using namespace std;
//Declaration des fonctions
string ping (char ip[50], int x);
int net_send (char message[200]);
string test_derniere_reponse (string ip);

//debut du programme
int main()
{

    unsigned int  nombres_champs = 0;
    unsigned long *lengths;
    string        update_code    = "";                 
    string        affiche        = "";                 // le resultat de la requete SQL sera stocké au fur et a mesure ds cette variable.
    string        reponse_ping   = "";                 // la variable qui stockera le resultat du ping
    string        requete        = "";
    string        update_requete = "";
    string        temp_requete   = "";
    size_t        size;
    char *        buffer         = "";
    char *        buffer2        = "";
    int           y              = 0;                  // permettra d'avoir un identifiant de boucle ds la boucle de ping.
    
    char  message_net_send[60];                        // notre variable pour faire passer un message a la fonction net send

//declaration des variables pour la bdd qui contient les infos reseaux
    char * host = "localhost";
    char * user = "root";
    char * pass = "manager";
    char * db = "osiris";
    MYSQL *sock ;                                        //Creation d'un string afin de permettre une saisie plus facile
    MYSQL_RES * reponse;                                   //un pointeur sur la reponse SQL
    MYSQL_ROW  ligne;                                      //pour lire le les resultats !

//declaration des variables pour la bdd qui contient les temps de réponse;
    char * update_host = "localhost";
    char * update_user = "root";
    char * update_pass = "manager";
    char * update_db = "osiris";
    MYSQL * update_sock ;    

//Initialisation de la connexion a la base qui stock les infos reseaux;                                                        
    sock = mysql_init(0);                                                     
    if (sock) cout << "Initialisation de la connexion Ok !" << "\n";                  
    else {
         cout << "ERREUR lors de l'initialisation de la connexion : " << mysql_error(sock) << "\n";
         }
//Initialisation de la connexion a la base pour mettre a jour les temps de réponse;
    update_sock = mysql_init(0);
    if (update_sock) cout << "Initialisation de la connexion Ok !" << "\n";
    else
         {
          cout << "Erreur lors de l'initialisation de la connexion : " << mysql_error(update_sock) << "\n";           
         }         
//connection pour recolter les infos reseau
    if (mysql_real_connect(sock, host, user, pass, db, 0, NULL, 0))   // Connection a la BDD avec pour parametres : sock = identificateur de la connexion, bufferhost = adresse IP de la machine, bufferuser = login de connexion a la base, bufferpass= pass de connection a la base, "" = nom de la base de donnée a selectionner (on peut par la suite faire un " USE ma_base; "
         cout << "connection ok, sur la bdd : " << db << "\n";
    else 
    {
         cout << "Echec de connection a la base de donnee : " << db << mysql_error(sock) << "\n";
    }
//connection pour mettre a jour les temps de reponses
    if (mysql_real_connect(update_sock, update_host, update_user, update_pass, update_db, 0, NULL, 0))   // Connection a la BDD avec pour parametres : sock = identificateur de la connexion, bufferhost = adresse IP de la machine, bufferuser = login de connexion a la base, bufferpass= pass de connection a la base, "" = nom de la base de donnée a selectionner (on peut par la suite faire un " USE ma_base; "
         cout << "connection ok, sur la bdd : " << db << "\n";
    else 
    {
         cout << "Echec de connection a la base de donnee : " << db << mysql_error(sock) << "\n";
    }
    

//execution d'une requete
/* cout << "requete ? " ;               
getline( cin, requete );      */
                                
requete = "select site,ip from annuaire_reseau;";
cout << requete.c_str();
while (1)
{
if (mysql_query(sock,requete.c_str())) cout <<"echec de la requete, motif : \n" << mysql_error(sock) << "\n";
    else 
    {
    cout << "requete faite !\n" << endl;       
    reponse = mysql_use_result(sock);
    nombres_champs = mysql_num_fields(reponse);
    while ((ligne = mysql_fetch_row(reponse)))        //Tant qu'on a une ligne de selectionnée
            {
               y++;
               lengths = mysql_fetch_lengths(reponse);
               // selection de la ligne et affichage de ce que l'on va tester
                    cout << "Connexion a : ";
                    affiche = ligne[0];                   // on stock le resultat de notre 1 ere case ds affiche  
                    cout << affiche << " ";                    
                       size = affiche.size() + 1;
                       buffer = new char[ size ];
                       strncpy( buffer, affiche.c_str(), size );                                    
                    affiche = ligne[1];                   // on stock le resultat de notre 2 eme case ds affiche      
                       size = affiche.size() + 1;
                       buffer2 = new char[ size ];
                       strncpy( buffer2, affiche.c_str(), size );
                    cout << "\"" << buffer2 << "\"   ";
               //test reseau
                    reponse_ping = ping (buffer2,y);
                    cout << reponse_ping;
                    if (reponse_ping == "offline")  //si le reseau ne repond pas on va envoyer un message si derniere_reponse = 0 et modifier la base en conséquence
                        reponse_ping = ping (buffer2,y);
                    if (reponse_ping == "offline")
                    {
                                     if(test_derniere_reponse (buffer2)=="0")
                                     {
                                      strcpy(message_net_send,"Le site de : ");
                                      strcat(message_net_send,buffer);
                                      strcat(message_net_send," ayant pour passerelle : ");
                                      strcat(message_net_send,buffer2);
                                      strcat(message_net_send," ne repond plus");                                     
                                      net_send(message_net_send);                                                          
                                      update_requete = "update annuaire_reseau set derniere_reponse = now() where ip = '" + affiche + "';";
                                      mysql_query(update_sock,update_requete.c_str()); 
                                     }
                                     else
                                     {
                                         // on est deja au courant on ne touche pas a la date et on n'envoie pas de messages.
                                     }
                                                                                                        
                    }
                    else                 //si le reseau repond on verifie qu'avant il repondait bien sinon on averti user que le reseau est retabli, la base sera modifié en conséquence
                    {
                                      if(test_derniere_reponse (buffer2)=="0");
                                      else
                                      {
                                       strcpy(message_net_send,"Le site de : ");
                                       strcat(message_net_send,buffer);
                                       strcat(message_net_send," ayant pour passerelle : ");
                                       strcat(message_net_send,buffer2);
                                       strcat(message_net_send," est a nouveau disponible");                                     
                                       net_send(message_net_send);
                                       update_requete = "update annuaire_reseau set derniere_reponse = '0' where ip='"+ affiche +"';";
                                       if (mysql_query(update_sock,update_requete.c_str()))cout <<"echec de la requete, motif : \n" << mysql_error(sock) << "\n";   
                                      }                                       
                    }

               // update du temps de réponse
               update_requete = "update annuaire_reseau set PING = \"";
               temp_requete = "\" where ip= '" + affiche;
               update_requete = update_requete + reponse_ping + temp_requete +"';";
               if (mysql_query(update_sock,update_requete.c_str()))cout <<"echec de la requete, motif : \n" << mysql_error(sock) << "\n";
               cout << endl;
               delete [] buffer;
               delete [] buffer2;
            }  
    }  
}       
//Pause !
    system("PAUSE");
    
//Fermeture de la connection a la BDD
    mysql_close(sock);
    mysql_close(update_sock);

    return EXIT_SUCCESS;
}



/*
Fonction pour ping, qui va renvoyer le temps de reponse de l'@ IP renseignée
Pour ce faire on va faire un ping windows ou l'on va sauvegarder notre resultat 
dans un fichier tempY (avec y = identifiant de ping), ensuite on valire notre 
fichier et y trouver ou pas un temps de reponse et renvoyer ce dernier, si aucun 
temps de réponse n'est retrouvé on renvoie offline
*/
 string ping (char ip[50], int x)
{
        int                 result      = 1;
        bool                booltest    = false;
        char                commande[100];
        char *              bufferping_id;
        char *              fichierbuffer;
        size_t              size;
        size_t              size2;
        string              tests;
        string              ping_id;
        ostringstream       oss;
        FILE *             temp;
 //On convertit notre variable X (qui sert a identifier un process de ping) en string, et par la suite en char
 oss << x;
 ping_id = oss.str();
 size = ping_id.size() + 1;
 bufferping_id = new char[ size ];
 strncpy( bufferping_id, ping_id.c_str(), size );
    
 strcpy (commande,"ping ");
 strcat (commande,ip);
 strcat (commande, " -n 1 -w 800 > temp");
 strcat (commande, bufferping_id);
 system(commande);
 string fichier = "temp";
  size2 = fichier.size() + 1;
  fichierbuffer = new char[ size2 ];
  strncpy( fichierbuffer, fichier.c_str(), size2 );
  strcat  (fichierbuffer, bufferping_id);   
  temp = fopen (fichierbuffer,"r");
  char test[50];
  
 while (!feof (temp))
       {
       fscanf (temp,"%s",test);
       if (booltest == true)
         {
                 booltest = false;
                 tests.assign(test);
                 return tests; 
         }
       if (strcmp (test,"bytes=32") == 0)
         {
                 booltest = true;
         }
       
       if (strcmp (test,"out.") == 0)
          {
                 result = 0;
          }
       }
 fclose(temp);
 tests.assign(test);
 tests = "offline";
 delete [] bufferping_id;
 delete [] fichierbuffer;
 return tests;
}

/*
une fonction qui envoie un message a l'utilisateur, le message est passé en 
parametres sous la form d'un char
*/
int net_send (char message [200])
{
    char commande[250];
    strcpy (commande,"net send moisan.j ");
    strcat (commande,message);
    strcat (commande," > save_net_send.txt");
    system(commande);
    return 0;
    
}

/*
Voici notre fonction qui va rechercher dans notre table annuaire_reseau la valeur 
de la colonne derniere_reponse et la retourner.
*/
string test_derniere_reponse (string ip)
{
    string              affiche         = "";
    string              requete         = "";
    unsigned int        nombres_champs  = 0;
    unsigned long *     lengths;    
    char *              host            = "localhost";
    char *              user            = "root";
    char *              pass            = "manager";
    char *              db              = "osiris";
    MYSQL *             sock ;
    MYSQL_RES *         reponse;                                
    MYSQL_ROW   ligne;   
    //initialisation
    sock = mysql_init(0);                                                      // on initialise la connexion a la base de donnée
    if (sock);                  
    else {
         cout << "ERREUR lors de l'initialisation de la connexion : " << mysql_error(sock) << "\n";
         }
    //connection
    if (mysql_real_connect(sock, host, user, pass, db, 0, NULL, 0)); 
    else 
    {
         cout << "Echec de connection a la base de donnee : " << db << mysql_error(sock) << "\n";
    }
    //la requete
    requete = "select derniere_reponse from annuaire_reseau where ip='"+ip+"';";                      
    //execution de la requete
    if (mysql_query(sock,requete.c_str())) cout <<"echec de la requete, motif : \n" << mysql_error(sock) << "\n";
    else 
    {         
    reponse = mysql_use_result(sock);
    nombres_champs = mysql_num_fields(reponse);
    while ((ligne = mysql_fetch_row(reponse)))        //Tant qu'on a une ligne de selectionnée
            {
                    lengths = mysql_fetch_lengths(reponse);
                    affiche = ligne[0];            // on stock le resultat de notre 1 ere case ds affiche  
            }  
    } 
 mysql_close(sock);
 if (affiche != "0"  )
 {
             return affiche;
 }
else
 { 
             return "0";
 }            
}

[koala01]

Salut,

Ceci dit, pourquoi t'em[oops] à allouer dynamiquement des tableaux de caracteres de l'a taille d'une chaine de type std::string connue

Pourquoi ne pas, tout simplement, utiliser la méthode c_str() des std::string pour récupérer la chaine "C style" quand tu en a besoin

Ainsi, les déclarations

Code :

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

1
2
3
4
5
    char * host = "localhost";
    char * user = "root";
    char * pass = "manager;
    char * db = "osiris";

seraient intelligemment remplacées par leurs homologues:

Code :

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

1
2
3
4
5
    /* const*/ std::string host = "localhost";
    /* const*/ std::string  user = "root";
    /* const*/ std::string  pass = "manager";
    /* const*/ std::string  db = "osiris";

et l'instruction/test (personnellement, j'aurais d'ailleurs préféré une instruction avec récupération de la valeur puis le test (séparé) de cette valeur, mais bon...)

Code :

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

if (mysql_real_connect(sock, host, user, pass, db, 0, NULL, 0))

serait intelligement remplacé par

Code :

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

1
2
3
if (mysql_real_connect(sock, host.c_str(), user.c_str(), pass.c_str(),
                      db.c_str(), 0, NULL, 0))

L'avantage serait qu'il est tout à fait possible d'avoir une code du genre de

Code :

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

1
2
3
4
5
6
7
8
9
10
/* chacune de ces chaines peut etre définie par l'utilisateur ou par
 * programmation ;) */
std::string requete="SELECT";
std::string what="*";
std::string table="matable";
std::string condition="id>100";
/*...*/
std::string final=requete+" "+what+" "+table+" where "+condition/* +... */;
mysql_query(update_sock,final.c_str());

[chaya]

J'ai remanié le programme avec les optimisations que vous m'avez conseillé, j'ai modifié pas mal de char pour les remplacer par des strings et j'ai donc remanié mes fonctions en conséquences.

Resultat => plus de plantage.

je devais surement avoir un probleme avec un char a un moment ou un autre. une taille trop petite pour un message, ce qui faisait planter aléatoirement le programme.

Maintenant avec les strings tout est geré en auto.

[frizou11]

Pense à la balise si tu as trouvé ton bonheur