🚀 go-pugleaf

"Richard Delorme" <abulmo@nospam.fr> wrote in message
news:43403d27$0$306$7a628cd7@news.club-internet.fr...
> Harpo a �crit :
> >
> > Je pense qu'il y a plus simple et plus performant :
> >
> > On prend une table dont la taille correspond aux valeurs possibles des
> > caract�res ( g�n�ralement 256 ), � l'emplacement dont l'offset
> > correspond � la valeur d'un caract�re, on place le caract�re par lequel
> > il doit �tre traduit.
>
> Je crois qu'il y a quelque subtilit� si l'on veut tenir compte du signe
> du type char, et avoir un truc portable. �a pourrait ressembler � :
>
> /*---------8<----------------*/
> #include <stdio.h>
> #include <limits.h>
> #include <ctype.h>
>
> void rot13(char s[])
> {
>     static int init = 0;
>     static char crypt[UCHAR_MAX];
>
>     if (!init) {

pas besoin d'une variable init s�par�e, on peut tester crypt[1] qui vaut 0 avant
initialisation et 1 ou une lettre de l'alphabet (donc pas 0) apr�s.

>        char alphabet[26] = "abcdefghijklmnopqrstuvwxyz";
>        char rot13_alphabet[26] = "nopqrstuvwxyzabcdefghijklm";

�viter les initialisations de tableaux automatiques: utiliser des const char *
ou des tableaux static.

>        unsigned int i;

pourquoi pas int ?

>        init = 1;
>        for (i = 0; i <= UCHAR_MAX; ++i) {
>           crypt[i] = i;
>        }
>        for (i = 0; i < 26; ++i) {
>           crypt[(int) alphabet[i]] = rot13_alphabet[i];

le cast en (int) de alphabet[i] est inutile, il ne fait rien du tout, de plus
alphabet[i] est forc�ment positif, donc inutile de caster en (unsigned char).

On peut aussi se passer de rot13_alphabet en �crivant:

crypt[alphabet[i]] = alphabet[(i + 13) % 26];

Mais c'est l�g�rement plus co�teux sur les rares machines o� la division est
lente.

>           crypt[toupper(alphabet[i])] = toupper(rot13_alphabet[i]);
>        }
>     }
>
>     for (;*s; ++s) {
>        *s = crypt[(int)((unsigned char) *s)];

*s = crypt[(unsigned char)(*s)];

c'est plus simple (on peut meme supprimer les ())

>     }
> }
>
> int main (void)
> {
>     char s[] = "La vie est un long fleuve tranquille!";
>
>     puts(s);
>     rot13(s);
>     puts(s);
>
>      return 0;
> }
> /*---------8<----------------*/

-- 
Chqrlie.