Discussion :

[jamin]

Bonjour,

Je souhaite connaitre la taille d'un r�pertoire, et ce, en utilisant la librairie boost qui est pr�conis�e dans notre application.

J'ai vu un exemple int�ressant... mais impossible � compiler.
(les boost::transform_iterator, c'est pas forc�ment trivial je trouve !).

Avez vous une id�e ?

exemple :
http://cboard.cprogramming.com/linux...cplusplus.html

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <boost/cstdint.hpp>
#include <algorithm>
#include <boost/filesystem.hpp>
#include <boost/bind.hpp>
#include <boost/iterators/transform_iterator.hpp>
namespace fs = boost::filesystem;
 
boost::uintmax_t dir_size(const fs::path &p)
{
  return std::accumulate(
    boost::make_transform_iterator(&fs::file_size, fs::recursive_directory_iterator(p)),
    boost::make_transform_iterator(&fs::file_size, fs::recursive_directory_iterator()),
    static_cast<boost::uintmax_t>(0));
}

[Nogane]

Bonsoir,

Tout d'abord, il semble que make_transform_iterator prenne l'iterateur en premier argument, et la fonction en deuxi�me. Ensuite, la fonction file_size est template, donc il faut sp�cifier quel version tu veux. Un code corrig� ressemblerait a ca:

Code :

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

1
2
3
4
  return std::accumulate(
    boost::make_transform_iterator(fs::recursive_directory_iterator(p), &fs::file_size<fs::path>),
    boost::make_transform_iterator(fs::recursive_directory_iterator(), &fs::file_size<fs::path>),
    static_cast<boost::uintmax_t>(0));

(il ne faut pas oublier d'inclure <numeric> aussi, pour le accumulate)

Une version un peu plus facile a comprendre serai:

Code :

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

1
2
3
4
5
6
7
8
9
	size_t size = 0;
	fs::recursive_directory_iterator iter = fs::recursive_directory_iterator(p);
	fs::recursive_directory_iterator endIter = fs::recursive_directory_iterator();
	while(iter != endIter)
	{
		size += fs::file_size(*iter);
		++iter;
	}
	return size;

[jamin]

Merci pour ta r�ponse.
Le premier exemple compile bien... mais �a plante. On ne peut pas appeler get_size sur un directory. Il faut regarder le status auparavant.

D'o� la 2eme solution, qui est effectivement plus claire, et dans laquelle on peut checker le status.

Maintenant, quel est le but d'�crire du code aussi concis que la 1ere solution. Y-a t-il des gains de performance ? Le std::accumulate ne fait ni plus ni moins qu'un +=

[JolyLoic]

Le but est de rendre la boucle implicite, ce qui permet :
- D'�viter de faire des erreurs li�es au parcours lui m�me
- �ventuellement de remplacer un jour ce parcours s�quentiel par un parcours plus rus�, par exemple parall�lis�.

Le probl�me �tant qu'en C++ actuel (�a ira mieux en C++0x), �crire le corps de la boucle est peu ais� (par exemple faire le test de status dont tu parles. C'est possible, mais lourd).

[jamin]

Ok merci pour l'info. Ca me semble plus clair maintenant.