Discussion :

[Goduak]

Hello!

Voilà je débute avec Boost et les smart pointers et je voudrais savoir si le code suivant est correct...
D'après ce que j'ai compris des docs, mon scoped_ptr va faire en sorte de supprimer le pointeur quand ma mythrd sera fini d'être exécuté.
Le fait qu'il soit dans un while est-ce problématique ?

Code :

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

1
2
3
4
5
6
7
while (!loop)
{
	req = get();
 
        func = boost::bind( &MyClass::myMethod, req );
        scoped_ptr<boost::thread> mythrd ( new boost::thread ( func ));
}

Merci pour vos réponses!

Go

[hegros]

Normalement une variable déclarée dans un bloc n'existe plus en dehors de ce bloc il me semble qu'elle est détruite.

Par rapport à la tâche qui s'execute je ne vois pas où cela pourrait poser probléme par contre

[Matthieu Brucher]

A chaque itération, un nouveau thread est créé et le précédent est supprimé, ou plutôt l'objet contenant le thread est supprimé, pas le thread lui-même.

[Goduak]

Merci de vos réponses!
Je suis d'accord que l'objet qui contient l'adresse du thread est écrasé à chaque itération, mais le fait de mettre un "scoped_ptr" ne sert pas sauf pour le dernier thread créé en cas de sortie du while n'est ce pas ?

[hegros]

En sortie de boucle ta variable pointeur intelligent sera aussi détruite(je pense) car elle est ne devrait plus exister en dehors du bloc dans lequel elle a été déclarée.

[corrector]

Merci de vos réponses!
Je suis d'accord que l'objet qui contient l'adresse du thread est écrasé à chaque itération, mais le fait de mettre un "scoped_ptr" ne sert pas sauf pour le dernier thread créé en cas de sortie du while n'est ce pas ?

Lors de la sortie du bloc, c'est à dire à chaque itération, le scoped_ptr sera détruit (pas "écrasé").
Le rôle de scoped_ptr est précisément de faire :

Code :

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

delete ptr;

ptr étant le pointeur passé à son constructeur, dans ce cas cela équivaut à :

Code :

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

delete new boost::thread ( func );

ce qui ne sert à rien, autant écrire juste

Code :

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

boost::thread ( func );