Matthieu MEZIL

L’art de compresser son code

Avant tout, je précise que ce post est complètement useless.

Comment juger de la qualité d’un code ?

En jugeant de sa rapidité d’exécution, de sa consommation en mémoire, parfois de la taille de celui-ci une fois compilé, très souvent de sa lisibilité.

Le problème avec ce dernier point est que la lisibilité est très relative. J’ai d’ailleurs un avis là-dessus très différent de celui de la plus part des développeurs que je cotoie. Certaines personnes (peut-être de mauvaise foie j’en conviens) vous expliqueront que plus un code est compact, plus il est rapide à relire donc plus il est lisible.

Ceci n’est pas uniquement le fruit de ma folie et / ou d’un moment de pure délire programmatique avec Roch. Des outils comme Resharper ont par exemple certaines règles qui vont en ce sens et que beaucoup trouveront hérétiques (comme l’utilisation systématique de var).

Prenons donc un exemple. Nous voulons afficher les 10 premiers multiples d’un nombre et la somme de ceux-ci :

using System;
 
class Program
{
     static void Main(string[] args)
     {
         int i = int.Parse(Console.ReadLine());
         int s = 0;
         for (int j = 1; j < 11; j++)
         {
             int s1 = i * j;
             Console.WriteLine(s1);
             s += s1;
         }
         Console.WriteLine(s);
     }
}

Ce code pourrait être condensé de la façon suivante :

using i = System.Int32;
using c = System.Console;
 
class Program
{
     static void Main(string[] args)
     {
         i i=i.Parse(c.ReadLine()),s=0,s1;
         for(i j=1;j<11;c.WriteLine((s1=j++*i)),s+=s1);
         c.WriteLine(s);
     }
}

De là à dire que c’est plus lisible, il y a un pas que je ne franchirai pas…

Un point intéressant maintenant : regardons l’IL généré.

Dans le second cas, nous avons :

Ce qui correspond à :

int i = int.Parse(Console.ReadLine());
int s = 0;
int j = 1;
while (j < 11)
{
     int s1;
     Console.WriteLine(s1 = j++ * i);
     s += s1;
}
Console.WriteLine(s);

Publié mardi 8 mars 2011 23:09 par Matthieu MEZIL