Discussion :

[crossbowman]

Je voulais cr�er une liste d'obets se modifiant facilement et j'ai pens� � cete classe :

class List_objets
{
public:
List_objets(int id)
{
list[2000];
}
int get_free_id()
{
int done = 0;
int i = 0;
while(!done && i<2000){if(list_check[i]) {i++;}else {done=true;break; return i;}}
return 0;
}
void new_objet(Object_sort &sort, int x, int y)
{
int id = get_free_id();
list_check[id] = true;
list[id] = &Objet(id, sort, x,y); //******************************pas bien
}

bool list_check[2000];
Objet *list[2000];
} unit_list(1);

J'ai cit� toute la classe, mais seule la proc�dure new_object est probl�matique. La ligne �list[id] = &Objet(id, sort, x,y);� cr�e un objet, mais le d�molit directement apr�s la fonction. Comment pourrais-je r�gler ce probl�me???????????????

[Invit�(e)]

Bonjour
Le mot clef 'static' pourrait il r�soudre ton probl�me.
Dans une classe, un membre d�clar� statique est le m�me pour tous les objets instanci�s de la classe.

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
fichier .h
class toto
{
  static int i;
public:
  tell()
  {
    cout << "je suis " << this << endl;
  }
  toto()
  {
    ++i
  }
  ~toto()
  {
    --i;
  } 
  static stat()
  {
    cout << "il y a " << i << " objets toto" << endl;
  }
};
 
fichier .cxx
#include <iostream> // pour cout
using namespace std;
#include ....h
int toto::i = 0; // une initialisation des membres statiques est utile en dehors de la déclaration de la classe
...
int main()
{
  toto::stat() ;// appel de la méthode statique
  toto obj1;
  obj1.tell(); // méthode non statique
  toto obj2;
  obj2.stat();// appel de la méthode statique
  return 0;
}

mabu

[JolyLoic]

Je voulais cr�er une liste d'obets se modifiant facilement et j'ai pens� � cete classe :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
class List_objets
{
public:
	List_objets(int id)
	{
		list[2000];
	}

Que crois tu que fait ce code ?

En r�alit�, il lit la valeur de la 2001�me case d'un tableau qui n'en a que 2000, ce qui est une bonne raison de planter le programme, puis il n'en fait rien.

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
	int get_free_id()
	{
		int done = 0;
		int i = 0;
		while(!done && i<2000){if(list_check[i]) {i++;}else {done=true;break; return i;}}
		return 0;
	}

Que vient faire "done" ? O� as-tu vu que le tableau �tait initialis� avec des 0 (ou NULL) ?

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
 
	void new_objet(Object_sort &sort, int x, int y)
	{
		int id = get_free_id();
		list_check[id] = true;
		list[id] = &Objet(id, sort, x,y); //******************************pas bien
	}

Peut-�tre ce que tu cherches est le placement-new :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

Object *newObject = new(list_check + get_free_id()) Object(id, sort, x, y);

J'ai cit� toute la classe, mais seule la proc�dure new_object est probl�matique.

H�l�s, non...

[Herode]

La ligne �list[id] = &Objet(id, sort, x,y);� cr�e un objet, mais le d�molit directement apr�s la fonction. Comment pourrais-je r�gler ce probl�me???????????????

L'appel au constructeur Objet(id, sort,x,y) cr�e un objet temporaire anonyme. Cet objet temporaire est d�truit d�s fin de l'instruction. Pour �viter cela, puisque ta liste prend des pointeurs, mieux vaut faire une allocation dynamique normale :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

list[id] = new Object( id, sort, x, y );

Cela dit, ton code a pas mal d'autres probl�mes et il y a des chances qu'une solution nettement plus simple et robuste soit facilement envisageable. Le recours � un std::map, peut-�tre ?