• [^] # Re: Correction

    Posté par . En réponse au journal Les bitcoins n'intéressent pas que les geeks. Évalué à 5.

    D'ailleurs, en parlant du logiciel Bitcoin, j'ai été voir les sources et bein ça fait peur :

    • Très peu de structure, un bon gros mélange de C et de C++, à moitié orienté objet ;
    • Une tonne de variables globales avec leur section critique associée ;). D'ailleurs j'ai de la peine à voir pourquoi un tel logiciel à besoin d'être threadé (exception du calcul). Le threading c'est rarement nécessaire et plutôt dangereux dans les langages impératifs (évidemment je ne parle pas des langages comme Erlang et consorts) ;
    • Des énormes fichiers de source ~4000 lignes ;
    • Pas de tests automatisés ;
    • Pas mal de macro ;
    • Peu de commentaire, surtout pas de commentaire au niveau des classes: Cadre d'utilisation, responsabilité, etc.. ;
    • Énormément d'inlining inutile.

    Je vais essayer de trouver du temps pour m'y investir et essayer (d'un peu) améliorer les choses. J'espère fortement qu'a terme ça va s'arranger.