Je viens de jeter un coup d'œil au code aussi. Je suis d'accord que les commentaires en Français c'est une très mauvaise idée. Il y a même un mélange d'anglais et de Français.
On voit dans le code que tu manques de l'expérience qu'on acquiert sur des projets comme KDE.
Quelques remarques en vrac sur ce que j'ai vu:
-tu ne respectes pas les conventions de codage de Qt et KDE (http://techbase.kde.org/Policies/Kdelibs_Coding_Style). Pourquoi se donner du mal?
-il y a beaucoup de code franchement dégueulasse comme:
QHttp::ConnectionMode mode = murl.scheme().toLower() == "https" ? QHttp::ConnectionModeHttps : QHttp::ConnectionModeHttp;
Quand tu contribues à un gros projet comme KDE, tu t'arranges pour que le code soit bien lisible pour la review. Tu pourrais apprendre pas mal de chose grâce à ces reviews
-beaucoup de commentaires sont inutiles. Des commentaires plus généraux sur des blocs de code sont plus intéressant que des commentaires qui dit ce que fait la ligne qui suit.
-le design des classes est parfois bancal.
Je pense que tu gagnerais vraiment a participer à un projet open-source. Si tu contribue à Qt ou KDE, du code comme celui que j'ai vu serais rejeté. MAIS tu apprendrais a améliorer ton code et le patch suivant serait mieux.
[^] # Re: Le Francais te tuera...
Posté par Benjamin Poulain . En réponse au journal Aider au développement d'un nouveau gestionnaire de paquets. Évalué à 6.
On voit dans le code que tu manques de l'expérience qu'on acquiert sur des projets comme KDE.
Quelques remarques en vrac sur ce que j'ai vu:
-tu ne respectes pas les conventions de codage de Qt et KDE (http://techbase.kde.org/Policies/Kdelibs_Coding_Style). Pourquoi se donner du mal?
-il y a beaucoup de code franchement dégueulasse comme:
QHttp::ConnectionMode mode = murl.scheme().toLower() == "https" ? QHttp::ConnectionModeHttps : QHttp::ConnectionModeHttp;
Quand tu contribues à un gros projet comme KDE, tu t'arranges pour que le code soit bien lisible pour la review. Tu pourrais apprendre pas mal de chose grâce à ces reviews
-beaucoup de commentaires sont inutiles. Des commentaires plus généraux sur des blocs de code sont plus intéressant que des commentaires qui dit ce que fait la ligne qui suit.
-le design des classes est parfois bancal.
Je pense que tu gagnerais vraiment a participer à un projet open-source. Si tu contribue à Qt ou KDE, du code comme celui que j'ai vu serais rejeté. MAIS tu apprendrais a améliorer ton code et le patch suivant serait mieux.