PyLint, PyChecker ou PyFlakes?

Bien, je suis un peu curieux, alors je viens de tester les 3 moi-même droit après avoir posé la question 😉

Ok, ce n'est pas très grave, mais voici ce que je peux dire :

J'ai essayé les outils avec les paramètres par défaut (c'est important parce que vous pouvez très bien choisir votre chèque de règlement) sur le script suivant :

#!/usr/local/bin/python
# by Daniel Rosengren modified by e-satis

import sys, time
stdout = sys.stdout

BAILOUT = 16
MAX_ITERATIONS = 1000

class Iterator(object) :

    def __init__(self):

        print 'Rendering...'
        for y in xrange(-39, 39): 
            stdout.write('\n')
            for x in xrange(-39, 39):
                if self.mandelbrot(x/40.0, y/40.0) :
                    stdout.write(' ')
                else:
                    stdout.write('*')


    def mandelbrot(self, x, y):
        cr = y - 0.5
        ci = x
        zi = 0.0
        zr = 0.0

        for i in xrange(MAX_ITERATIONS) :
            temp = zr * zi
            zr2 = zr * zr
            zi2 = zi * zi
            zr = zr2 - zi2 + cr
            zi = temp + temp + ci

            if zi2 + zr2 > BAILOUT:
                return i

        return 0

t = time.time()
Iterator() 
print '\nPython Elapsed %.02f' % (time.time() - t)

Comme un résultat :

• PyChecker est problématique, car il compile le module pour l'analyser. Si vous ne voulez pas que votre code à exécuter (e.g, il effectue une requête SQL), c'est mauvais.
• PyFlakes est censé être lite. En effet, il a décidé que le code a été parfait. Je suis à la recherche de quelque chose de très grave donc je ne pense pas que je vais y aller.
• PyLint a été très bavard et a classé le code 3/10 (OMG, je suis un sale programmeur !).

Points forts de PyLint:

• Très précis et descriptif de rapport.
• Détecter certaines odeurs de code. Ici, il m'a dit de déposer ma classe pour écrire quelque chose avec les fonctions, car l'OO approche était inutile dans ce cas précis. Quelque chose que je savais, mais jamais attendu à un ordinateur pour me dire :-p
• Entièrement corrigé le code de courir plus vite (pas de classe, pas de référence de liaison...).
• Faite par une équipe française. Ok c'est pas un plus pour tout le monde, mais j'aime ça 😉

Les inconvénients de PyLint:

• Certaines règles sont très strict. Je sais que vous pouvez le changer et que la valeur par défaut est de faire correspondre le PEP8, mais est-ce un crime d'écrire "pour x dans seq'? Apparemment oui, car vous ne pouvez pas écrire le nom d'une variable de moins de 3 lettres. Je vais changer cela.
• Très très bavard. Être prêt à utiliser vos yeux.

Corrigé le script (avec lazy doc chaînes et les noms de variables) :

#!/usr/local/bin/python
# by Daniel Rosengren, modified by e-satis
"""
Module doctring
"""


import time
from sys import stdout

BAILOUT = 16
MAX_ITERATIONS = 1000

def mandelbrot(dim_1, dim_2):
    """
    function doc string
    """
    cr1 = dim_1 - 0.5
    ci1 = dim_2
    zi1 = 0.0
    zr1 = 0.0

    for i in xrange(MAX_ITERATIONS) :
        temp = zr1 * zi1
        zr2 = zr1 * zr1
        zi2 = zi1 * zi1
        zr1 = zr2 - zi2 + cr1
        zi1 = temp + temp + ci1

        if zi2 + zr2 > BAILOUT:
            return i

    return 0

def execute() :
    """
    func doc string
    """
    print 'Rendering...'
    for dim_1 in xrange(-39, 39): 
        stdout.write('\n')
        for dim_2 in xrange(-39, 39):
            if mandelbrot(dim_1/40.0, dim_2/40.0) :
                stdout.write(' ')
            else:
                stdout.write('*')


START_TIME = time.time()
execute()
print '\nPython Elapsed %.02f' % (time.time() - START_TIME)

EDIT :

Grâce à Rudiger Loup, j'ai découvert pep8 qui fait exactement ce que son nom suggère: correspondance PEP8. Il a trouvé plusieurs syntaxe non-n que PyLint n'a pas. Mais PyLint trouvé des choses qui n'était pas spécifiquement liée à la PEP8, mais intéressant. Les deux outils sont intéressants et complémentaires.

Finalement je vais utiliser à la fois car il y a vraiment faciles à installer (via les packages ou les setuptools) et le texte de sortie est donc facile de chaîne.

Pour vous donner une petite idée de leur sortie:

pep8:

./python_mandelbrot.py:4:11: E401 multiple imports on one line
./python_mandelbrot.py:10:1: E302 expected 2 blank lines, found 1
./python_mandelbrot.py:10:23: E203 whitespace before ':'
./python_mandelbrot.py:15:80: E501 line too long (108 characters)
./python_mandelbrot.py:23:1: W291 trailing whitespace
./python_mandelbrot.py:41:5: E301 expected 1 blank line, found 3

PyLint:

************* Module python_mandelbrot
C: 15: Line too long (108/80)
C: 61: Line too long (85/80)
C:  1: Missing docstring
C:  5: Invalid name "stdout" (should match (([A-Z_][A-Z0-9_]*)|(__.*__))$)
C: 10:Iterator: Missing docstring
C: 15:Iterator.__init__: Invalid name "y" (should match [a-z_][a-z0-9_]{2,30}$)
C: 17:Iterator.__init__: Invalid name "x" (should match [a-z_][a-z0-9_]{2,30}$)

[...] and a very long report with useful stats like :

Duplication
-----------

+-------------------------+------+---------+-----------+
|                         |now   |previous |difference |
+=========================+======+=========+===========+
|nb duplicated lines      |0     |0        |=          |
+-------------------------+------+---------+-----------+
|percent duplicated lines |0.000 |0.000    |=          |
+-------------------------+------+---------+-----------+