[Dcmlib] sprintf

[Mathieu Malaterre]

> 
> From: Eric Boix <Eric.Boix at creatis.insa-lyon.fr>
> Date: 2004/10/10 Sun AM 11:46:57 EDT
> To: Mathieu Malaterre <mathieu.malaterre at kitware.com>
> CC: Dcmlib at tux.creatis.insa-lyon.fr
> Subject: Re: [Dcmlib] sprintf
> 
> 	Salut Mathieu,
> 
> Quoting Mathieu Malaterre <mathieu.malaterre at kitware.com>:
> > std::ostringstream buf;
> > buf << std::right << std::setw(4) << std::setfill('0') << std::hex <<
> > group  << "|" << std::right << std::setw(4) << std::setfill('0') << 
> > std::hex << element;
> > std::string key = buf.str();
> > 
> > J'ai fais un grep en fait dans gdcm on a /pour l'instant/ qu'un seul 
> > sprintf et c'est celui du mail. Donc une solution generique pour un seul 
> > type de formatage est-ce raisonable ?
> > 
> Ben, c'est vrai que c'est un peu lourdingue, mais comme il n'existe
> qu'une seule occurence (factorise'e justement pour eviter de multiplier
> les lourdeurs partout), je serais assez pour mettre le C++ pur
> (ci-dessus)...

Autant pour moi il y en a (avait) une dizaine. J'ai penser que les iostream c'etait tellement lourd (et personne ne maitrisait) qu'il fallait mieux une methode hybride, cf gdcmUtil::Format

Je trouve que ca condence bien le code, et comme ca retourne une std::string c'etait bien pratique...

Si ca va pas, je me (re)met une entrée dans la TODO list

Mathieu
Mathieu