Devblog - Greg

Aller au contenu | Aller au menu | Aller à la recherche

Tag - commentaire

Fil des billets - Fil des commentaires

mercredi, juin 16 2010

Le retour du Taiwanais...

Par Greg le mercredi, juin 16 2010, 20:46

Je viens de me rappeler d'un autre bout de code "mystique" aperçu dans les sources ^^ Cette fois-ci ce n'était pas gênant car c'était dans une partie du code qu'on n'utilise pas, mais bon...

En gros, vous avez une classe tableau (ou liste ou je sais pas ce que c'était, mais bon, en gros des données quoi ^^). Avec une belle fonction pour ajouter des choses dedans (à la suite). Jusque là, rien de spécial me direz-vous... Sauf qu'à la fin de la fonction d'ajout, on trouve ça :

void MyClass::add(...) {
    ... // Add to array
    size++;

    // Seems to slow down and bug when there's to much data
   if (size > 230)
        size = 230;
}

On n'a jamais compris pourquoi la limite est à 230, mais bon ^^

aucun commentaire aucun rétrolien

Code au rabais ?

Par Greg le mercredi, juin 16 2010, 20:22

Un taiwanais, ça coute pas cher. Mais est-ce que c'est bon ?

Je ne vais pas faire de généralités, mais j'ai trouvé un truc pas mal dans le code tout à l'heure (copié/collé d'un truc "pro" fait à Taiwan...). En fait c'était un bout de code pour convertir un double en 2 ints (un pour la partie entière, un pour la partie décimale). Donc en gros, on obtient le code suivant :

int p, d;
double val = XXX;
p = (int)val;
val = val - p;
while(val < 1) val *= 10;
d = val;

Bizarrement de temps en temps ça bloque le programme... (on notera que la récupération de la partie décimale est aussi super bien pensée... genre 1.01 renvoie 1, tout comme 1.1 et 1.0001 (entre autres...) ^^)

un commentaire aucun rétrolien

lundi, juin 14 2010

Commentaires douteux

Par Greg le lundi, juin 14 2010, 19:41

J'ai commencé mon stage aujourd'hui, et en farfouillant dans les sources d'une grande boite (dont je tairais le nom ^^), je suis tombé sur des commentaires pour le moins étrange. Ces commentaires concernaient 2 fonctions, déclarées de la sorte :

void XXXXX(void)
{
    // Doesn't matter
}

void YYYYY(void)
{
    // You should never get here
}

Et histoire de parfaire le tout, un peu plus loin j'ai trouvé un petit bout de code en commentaire avec un TODO associé :

// XXXX->ZZZZ();
// TODO : delete this

Ca fait plaisir de voir qu'on fait du code foireux même chez les grands :p Le moins que l'on puisse dire, c'est que des commentaires comme ça, c'est de la daube ^^ ;)

228 commentaires aucun rétrolien