N'hésitez plus : les revues de code

[dourouc05]

Bonjour à tous,

Suite à une certaine demande après le défi, il nous a semblé utile d'organiser quelque peu une revue de code Qt.

En deux mots, de quoi s'agit-il ? Des développeurs plus expérimentés lisent le code d'autres développeurs et le commentent : ceci aurait été mieux d'une autre manière, ce commentaire ne veut rien dire, ce pavé de code devrait être réécrit, etc. Des remarques sur le fond et la forme du code, sur son organisation, les patrons de conception utilisés, les concepts inhérents à Qt et/ou au C++ mal compris ou mal appliqués, etc. L'objectif est de confronter les opinions sur un code pour qu'au final tout le monde y gagne.

Pour l'organisation, autant faire simple : pour ceux qui souhaitent une revue de code, créez un sujet dans le forum Qt, postez-y un lien vers votre code (il doit donc être librement disponible et le rester, afin que l'exercice puisse être profitable à d'autres), avec quelques commentaires dessus que vous jugerez intéressants, en mentionnant dans le titre que vous souhaitez une revue de code à l'aide d'un tag [Revue de code] ; une fois ce sujet créé, postez à la suite de ce message, pour que l'on puisse le lister ici ; pour ceux qui souhaitent lire le code, n'hésitez pas à le faire, peu importe votre niveau, pour autant que vous apportiez quelque chose à la discussion (ce n'est pas l'endroit pour que les débutants posent leurs questions sur le code, il vaut mieux en ce cas ouvrir un sujet dédié dans le forum Débuter en précisant le bout de code sur lequel vous aimeriez avoir plus d'explications) ; vous pourrez trouver la liste des sujets de revue de code à la fin de ce message.

Note : comme toujours, ce sont les participants du forum qui pourront participer, bénévolement, il serait donc malvenu de pester parce que certaines remarques ne vous plaisent pas ou sont formulées un peu rapidement (tout le monde n'a pas des jours devant lui) .
Tout le monde peut proposer du code à relire, mais l'excès nuit en tout, préférez poster du code plus spécifiquement sur lequel vous aimeriez des avis extérieurs ou tout le code si c'est l'architecture qui vous semble à revoir (autant le préciser de suite ). De même, préférez indiquer approximativement votre niveau, cela permettra de savoir jusqu'à quel point il faut pousser les explications.

Un tout grand merci d'avance à tous ceux qui participeront !

Sujets de revue de code

MedCenter (application du deuxième défi de LittleWhite)
FreeWindows (Art of Sequence) (Klaim)

[ZiGoM@r]

Super idée ! A exporter dans d'autres rubriques que Qt.

[grefz]

super merci, elle est où la rubrique QT?

[Klaim]

Ça tombe bien pour moi!

Juste une note: ça serait bien d indiquer clairement que les commentaires sur le style (indentation etc.) n ont rien à faire dans une code review, pour ceux qui n en ont pas l habitude.

[Klaim]

J'ai une question aussi : j'ai du code où il manque une chose mais qui reste fonctionnel et que j'aimerai montrer (en fait il est déjà accessible). J'ai besoin de retours et de conseils pour compléter la feature. Est-ce qu'on peut considérer ça comme code review ou bien c'est plus une "question" classique?

[dourouc05]

Envoyé par grefz

super merci, elle est où la rubrique Qt ?

Ici même : http://www.developpez.net/forums/f37...liotheques/qt/.

Envoyé par Klaim

J'ai une question aussi : j'ai du code où il manque une chose mais qui reste fonctionnel et que j'aimerai montrer (en fait il est déjà accessible). J'ai besoin de retours et de conseils pour compléter la feature. Est-ce qu'on peut considérer ça comme code review ou bien c'est plus une "question" classique?

Considère ça comme tu veux, je dirais que ça peut rentrer dans les deux catégories !

[Klaim]

Ok, merci, je viens de poster du coup : http://www.developpez.net/forums/d11...t-of-sequence/

[dourouc05]

Liste à jour !