[Desenvolupament] Catalanitzador (refactorings i estil del codi)

[Jordi Mas]

Hola David,

*Estil de codificació:*
>
>    - Declarar les constants i les macros al principi del .cpp i no
>    intercalat entre les definicions dels mètodes.
>    - Inicialitzar tots els arrays *char* o *wchar_t* a 0 (per exemple, *
>    char** szText[1024] = {0};*). En *debug *el compilador ja ho fa però
>    en *release* no i aquesta és una font comú d'errors.
>
>
Totalment d'acord, qualsevol cosa que no vegis en aquesta àrea soluciona-ho
si us plau.

Si ho pots també documentar aquí:
https://github.com/Softcatala/CatalanitzadorPerAWindows/blob/master/CODING_STANDARDS


>
>
>
> *Refactorització wchar_t:*
>
>    - En les classes derivades de *Action *no s'inicialitza *szName *i *szDescription
>    *fins que es truca als mètodes *GetName*() i *GetDescription*(). És
>    per alguna cosa en concret? Si el valor pot canviar dinàmicament d'acord té
>    sentit, però si el seu valor és invariant durant la vida del objecte té més
>    sentit inicialitzar-ho en el constructor de cada classe derivada. Més
>    encara el *GetName*() i *GetDescription*() es podrien implementar a la
>    classe base sense declarar-los com virtuals purs (Self Encapsulate Field
>    pattern) i que cada classe derivada inicialitzi al constructor el valor que
>    necessiti. D'aquesta manera també simplifiques la implementació de les
>    classes derivades de *Action*, ja que no cal implementar aquests dos
>    mètodes virtuals.
>
> Sí, endavant. Em semblen bé els canvis.


>
>
> *Errors:*
>
>    - Al mètode WindowsLiveAction::GetDescription() hi ha un petit bug.
>    Hauria de ser: *return
>    _getStringFromResourceIDName(IDS_WINDOWSLIVE_DESCRIPTION, szDescription
>    )*
>
> Sí, és un error. Si pots solucionar això com a primer commit.

T'he donat permís, pots pujar coses directament.


Jordi,
-------------- part següent --------------
Un document HTML ha estat eliminat...
URL: <http://llistes.softcatala.org/pipermail/catalanitzador/attachments/20121017/cd04d751/attachment.html>