Vous êtes nouveau sur Developpez.com ? Créez votre compte ou connectez-vous afin de pouvoir participer !

Vous devez avoir un compte Developpez.com et être connecté pour pouvoir participer aux discussions.

Vous n'avez pas encore de compte Developpez.com ? Créez-en un en quelques instants, c'est entièrement gratuit !

Si vous disposez déjà d'un compte et qu'il est bien activé, connectez-vous à l'aide du formulaire ci-dessous.

Identifiez-vous
Identifiant
Mot de passe
Mot de passe oublié ?
Créer un compte

L'inscription est gratuite et ne vous prendra que quelques instants !

Je m'inscris !

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

Le , par dourouc05

0PARTAGES

8  0 
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)

Une erreur dans cette actualité ? Signalez-le nous !

Avatar de 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.
0  0 
Avatar de grefz
Membre à l'essai https://www.developpez.com
Le 16/11/2011 à 22:47
super merci, elle est où la rubrique QT?
0  0 
Avatar de 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.
0  0 
Avatar de 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?
0  0 
Avatar de dourouc05
Responsable Qt & Livres 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 !
0  0 
Avatar de 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/
0  0 
Avatar de dourouc05
Responsable Qt & Livres https://www.developpez.com
Le 18/11/2011 à 20:15
Liste à jour !
0  0