[Dcmlib] NumPixel et introduction de gdcmParser

[Jean-Pierre ROUX]

At 17:01 +0100 27/03/2004, Eric Boix wrote:
>	Yo,
>
>Question essentiellement destine'e a Benoit et Jean-Pierre....
>
>En cherchant a simplifier le code de doxygenation de gdcmHeader::GetNumPixel
>je suis tombe' sur ce qui me semble etre IMVHO une incoherence.
>En effet gdcmHeader::GetNumPixel est un accesseur a NumPixel qui est
>en fait gdcmParser::NumPixel. Plusieurs questions s'imposent a moi:
>
> * Si NumPixel est bien a sa place dans la classe gdcmParser pourquoi
>   ne pas y mettre l'accesseur correspondant, plutot que de le releguer
>   dans UNE classe derive'e de gdcmParser en l'occurence gdcmHeader ?
> * Vue de loin, il semblerait que NumPixel ne concerne que les entetes
>   d'images (gdcmHeader) et non les gdcmDicomDir (correct?). Si tel est
>   le cas, NumPixel devrait alors etre un attribut de gdcmHeader, plutot
>   que de gdcmParser.

GetNumPixel -et ses soeurs- sont (devrait etre) en effet des methodes 
de gdcmHeader, et pas de gdcmParser.
C'est du à un découpage trop rapide.


>
>Comme il n'y a aucune doc sur le role de gdcmParser

Le nom lui même gdcmParser n'est pas des plus heureux, car il fait 
'OpenFile + Parse + LoadValues + CloseFile'.
Je n'avais pas trouve de nom 'meilleur'.
Peut etre gdcmLoader ?

>(et c'est bien
>regretable), j'avoue ne pas pouvoir me decider rapidement sans me pougner
>pas mal de code. Je pense aussi que NumPixel n'est pas le seul attribut
>dans ce cas (cf GrPixel ou encore countGrPixel).

countGrPixel est dans ce même cas, et il dégagera dès le prochaine 
tremblement de terre dans GDCM, car on pourra s'en passer.

Bon WE.

>Est-ce volontaire ou
>s'agit-il d'un reliquat du decoupage un peu rapide de gdcmHeader en
>gdcmHeader + gdcmParser ?
>
>	Frog.
>
>_______________________________________________
>Dcmlib mailing list
>Dcmlib at creatis.insa-lyon.fr
>http://www.creatis.insa-lyon.fr/mailman/listinfo/dcmlib

  Jean-Pierre ROUX
  UMR CNRS 5515-CREATIS
  Laboratoire de Radiologie Experimentale
  Hopital Cardiologique
  28 Avenue du Doyen LEPINE
  B.P. Lyon-Montchat
  69394 Lyon Cedex 03

  Tel      : (+33) 04 72 35 74 12
  Fax      : (+33) 04 72 68 49 16
  URL      : http://www.creatis.univ-lyon1.fr
  e-mail   : jpr at univ-lyon1.fr