Discussion :

[Linu6]

voici le code qui est la cause de mes maux de tête .. depuis 3 jours.

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
#include <fstream>
#include <iostream>
#include <memory>
using namespace std;
 
class file_error {};
 
class entier_f {
private:
  int * a;
  char * fn;
  ifstream input;
 
public:
  entier_f(char * nom, int val=0) : fn(nom) {
    a = new int[1000000];	// Allocation d'un tableau de 1 Mo !!!
    *a= val;
    input.open(fn);
    if (! input) { 
      throw(file_error());
    };
  };
  ~entier_f() {
    delete a;
    a=NULL;
    input.close();
  };
};
 
class entier_f1 {
private:
  auto_ptr<int> a;
  char * fn;
  ifstream input;
 
public:
  entier_f1(char * nom, int val=0) : fn(nom) {
    a = new int[1000000];	// Allocation d'un tableau de 1 Mo !!!
    *a= val;
    input.open(fn);
    if (! input) { 
      throw(file_error());
    };
  };
  ~entier_f1() {
    input.close();
  };
};
 
main() {
  try {
    entier_f I2("toto");
  } 
  catch(file_error) {
    cout << "erreur de lecture de fichier" << endl;
  };
 
  int j=0;
  for (int i=0;i<100;i++) {	
    j++;
    try {
      entier_f1 I2("toto");
    } catch (file_error) {
      //      cout << "ERR" << endl;
      continue;
    } catch (...) {
      break;                 // Pas de fuite de memoire
    };
  };
  cout << "boucle executee " << j << " fois" << endl; 
 
  j=0;
  for (int i=0;i<100;i++) {	
    j++;
    try {
      entier_f I2("toto");
    } catch (file_error) {
      //      cout << "ERR" << endl;
      continue;
    } catch (...) {
      break;                 // ca va planter (fuite de memoire)
    };
  };
  cout << "boucle executee " << j << " fois" << endl;
};

[ram-0000]

Et c'est quoi le problème ? compilation, exécution, crash, ...

[Montag]

Salut,

Bon alors plusieurs remarques :
1) ton code ne compile pas (c'est quand même gênant)
2) le prototype de la fonction main c'est :

Code :

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

int main()

ou bien :

Code :

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

int main(int, char **)

3) tu convertis des char const * en char * : c'est moche.

Bon ceci dit le problème vient du fait que le destructeur de auto_ptr fait un delete sur le pointeur qu'il gère. Or toi tu lui passes un tableau et je te rappelle que dans ce cas la libération de la mémoire doit se faire avec un delete[]. Ton programme a un comportement indéterminé.

PS : si tu es sous Linux un outil comme valgrind permet de détecter ce genre de problème.

[Médinoc]

Chaque fois que j'ai utilisé l'opérateur = pour initialiser un auto_ptr<> à partir d'un pointeur brut, j'ai eu des problèmes (sous Visual).
Je conseille d'utiliser sa fonction membre reset() à la place.

[Médinoc]

Ce code marche sous Visual 2005, avec un fichier "toto" vide :

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
//problème d'allocation d'auto_ptr
// par Linu6
//http://www.developpez.net/forums/showthread.php?p=3396870#post3396870
 
#include <fstream>
#include <iostream>
#include <memory>
#include <exception>
#include <string>
using namespace std;
 
class file_error : public std::exception
{
public:
	//NOTE: Ceci marche sous Visual car le constructeur
	//qui prend seulement un char const * en paramètre est garanti dupliquer la chaîne.
	//Le code risque de devoir être modifié sous d'autres compilos,
	//car je ne sais pas ce que dit le standard sur le(s) constructeur(s) de std::exception
	file_error(char const *fileName)
	 : std::exception(std::string("File error (").append(fileName).append(")").c_str())
	{ }
};
 
class entier_f {
private:
	int * a; //Lifetime managed by this
	char const * fn; //Lifetime NOT managed by this
	ifstream input;
 
public:
	entier_f(char const * nom, int val=0) : fn(nom) {
		a = new int[1000000];	// Allocation d'un tableau de 1 Mo !!!
		a[0] = val;
		input.open(fn);
		if (!input) { 
			throw(file_error(nom));
		};
	};
	~entier_f() {
		delete[] a;
		a=NULL;
		input.close();
	};
};
 
 
class entier_f1 {
private:
	auto_ptr<int> a;
	char const * fn; //Lifetime NOT managed by this
	ifstream input;
 
public:
	entier_f1(char const * nom, int val=0) : fn(nom) {
		a.reset(new int[1000000]);	// Allocation d'un tableau de 1 Mo !!!
		a.get()[0] = val; //J'emploie la syntaxe tableau, puisque c'est sur un tableau qu'on pointe
		input.open(fn);
		if (!input) { 
			throw(file_error(fn));
		};
	};
	~entier_f1() {
		input.close();
	};
};
 
 
int main(void) {
	try {
		entier_f I2("toto");
	} 
	catch(exception const &e) {
		cout << "erreur de lecture de fichier: " << e.what() << endl;
	};
 
	int j=0;
	for (int i=0;i<100;i++) {	
		j++;
		try {
			entier_f1 I2("toto");
		} catch (exception const &) {
			//      cout << "ERR" << endl;
			continue;
		} catch (...) {
			break;                 // Pas de fuite de memoire
		};
	};
	cout << "boucle executee " << j << " fois" << endl; 
 
	j=0;
	for (int i=0;i<100;i++) {	
		j++;
		try {
			entier_f I2("toto");
		} catch (file_error) {
			//      cout << "ERR" << endl;
			continue;
		} catch (...) {
			break;                 // ca va planter (fuite de memoire)
		};
	};
	cout << "boucle executee " << j << " fois" << endl;
	return 0;
};

[Linu6]

re

Pardon, pour avoir oublié de mentionner quelques détails.
Ainsi: En ce moment je travaille sous Windows (et non linux)
ensuite le problème est un problème de compilation. qui se trouve dans

class entier_f1 -> public: -> allocation (Ligne2)

et voici le message d'erreur.

Code :

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

1
2
3
4
5
 In constructor `entier_f1::entier_f1(char*, int)': 
no match for 'operator=' in '((entier_f1*)this)->entier_f1::a = (int*)operator new [](4000000u)' 
\include\c++\3.4.2\memory:223 candidates are: std::auto_ptr<_Tp>& std::auto_ptr<_Tp>::operator=(std::auto_ptr<_Tp>&) [with _Tp = int] 
include\c++\3.4.2\memory:223    
std::auto_ptr<_Tp>& std::auto_ptr<_Tp>::operator=(std::auto_ptr_ref<_Tp>) [with _Tp = int]

[Montag]

As-tu lu les messages de Médinoc et mon post ?

[Médinoc]

@Montag: En effet, je n'ai pas pris en compte ton post dans mon code.
Sous l'implémentation Microsoft actuelle, ça marche parce que int n'a pas de destructeur, mais ce n'est absolument pas garanti marcher.

En pratique, on peut dire que gérer un tableau avec auto_ptr<> est impossible. C'est sans doute pour ça qu'existe la classe vector<>...

[Linu6]

re

une dernière question (histoire de compléter le topic )
Comment on détruit un tableau alloué avec reset()?

merci.

[Médinoc]

reset() n'alloue rien, il sert juste à affecter un pointeur brut à un auto_ptr<>.

Comme Montag l'a dit, auto_ptr<> n'offre aucun moyen de désallouer proprement un tableau: Tu ne peux désallouer qu'un seul objet avec.

[Linu6]

ok

merci à vous.