• [^] # Re: .

    Posté par (site web personnel) . En réponse au journal Naissance du projet nanim. Évalué à 4.

    Le vrai programmeur ne commente pas, le code est évident!

    J'espère que c'est de l'humour. Mais en tout cas, un tel code, même évident, ne donne pas envie d'y contribuer malheureusement.

    Plus sérieusement

    ouf

    si quelqu'un a besoin d'aide sur mon code, je commenterais les parties non triviales

    hum…

    Plus sérieusement, avoir des commentaires bien fait ça permet aussi de générer une doc développeur (par exemple une javadoc) qui pourrait laisser un espoir à celui qui souhaite utiliser ton travail.

    Je n'indente rien moi même,je laisse cette tâche ingrate à mon IDE/éditeur

    Oui, moi aussi. Mais surtout je le configure pour que ça ressemble à quelque chose
    Ici c'est quand même pas génial…

    public class NanimDec 
    {
     private CommandLine commandLine;
     private Nanim nanim;
     private int nbImageDecoded = 0;
     public NanimDec(CommandLine line) {
     this.commandLine = line;
     }
     public static void main( String[] args ) throws ParseException, IOException
     {
     Options options = new Options();
     options.addOption("i", true, "input nanim file");
     options.addOption("o", true, "output directory");
     if(args.length == 0) {
     HelpFormatter formatter = new HelpFormatter();
     formatter.printHelp( "nanimenc [args]", options );
     return;
     }
     GnuParser parser = new GnuParser();
     CommandLine line = parser.parse(options, args);
     NanimDec nanimDec = new NanimDec(line);
     nanimDec.decode();
     nanimDec.save();
     }
    [...]
     public static boolean isFilenameValid(String pathname) {
     try {
     new File(pathname).getCanonicalPath();
     return true;
     } catch (IOException e) {
     return false;
     }
     }
    
    

    Raaaa ! oui le code est propre icitte mais c'est parce que les tabulations sont remplacées par 4 espaces… 'achement malin ! http://bci.im/devnewton/fossils/nanim/artifact/7bac83ad82b5bbee417fc3b7456f604e98e31cf7

    Histoire d'expliciter un peu plus :

    • devant { au début du static main, c'est des espaces et non des tabulation. Ton code dépendant donc de la valeur des tabulations (je suppose 4 pour toi)
    • au niveau de isFilenameValid tu as un sacré mélange d'erreur d'indentation, tabulations et espaces

    Tout ça, c'est pas pour casser du sucre sur le code (d'ailleurs j'ai pas vraiment regardé ce que ça fait) mais juste pour pointer des choses qui font qu'un code donne, de mon point de vue, envie d'être utilisé voir y contribuer, ou non.
    Et là, mauvaise indentation, aucun commentaire, ben ça donne pas trop envie.

    (perso, pour mon style en java j'ai fini par passer aux conventions de Google, je trouve que ça va plutôt bien : http://google-styleguide.googlecode.com/svn/trunk/)