Newsletter Developpez.com

Inscrivez-vous gratuitement au Club pour recevoir
la newsletter hebdomadaire des développeurs et IT pro

Un espace de revue de code a ouvert ses portes
Dans la rubrique Qt, venez participer

Le , par dourouc05, Responsable Qt
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)


Vous avez aimé cette actualité ? Alors partagez-la avec vos amis en cliquant sur les boutons ci-dessous :


 Poster une réponse

Avatar de ZiGoM@r ZiGoM@r - Membre averti https://www.developpez.com
le 16/11/2011 à 21:52
Super idée ! A exporter dans d'autres rubriques que Qt.
Avatar de grefz grefz - Membre à l'essai https://www.developpez.com
le 16/11/2011 à 22:47
super merci, elle est où la rubrique QT?
Avatar de Klaim Klaim - Membre expert https://www.developpez.com
le 17/11/2011 à 9:49
Ç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.
Avatar de Klaim Klaim - Membre expert https://www.developpez.com
le 17/11/2011 à 11:32
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?
Avatar de dourouc05 dourouc05 - Responsable Qt https://www.developpez.com
le 17/11/2011 à 17:08
Citation Envoyé par grefz  Voir le message
super merci, elle est où la rubrique Qt ?

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

Citation Envoyé par Klaim  Voir le message
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 !
Avatar de Klaim Klaim - Membre expert https://www.developpez.com
le 18/11/2011 à 19:16
Ok, merci, je viens de poster du coup : http://www.developpez.net/forums/d11...t-of-sequence/
Avatar de dourouc05 dourouc05 - Responsable Qt https://www.developpez.com
le 18/11/2011 à 20:15
Liste à jour !
Offres d'emploi IT
Architecte et intégrateur scade/simulink H/F
Safran - Ile de France - Vélizy-Villacoublay (78140)
Architecte électronique de puissance expérimenté H/F
Safran - Ile de France - Villaroche - Réau
Ingénieur analyste programmeur (H/F)
Safran - Auvergne - Montluçon (03100)

Voir plus d'offres Voir la carte des offres IT
Responsable bénévole de la rubrique Qt : Thibaut Cuvelier -