Re: [Chorem-devel] [Chorem-commits] r68 - trunk/chorem-web/src/main/java/org/chorem/gepeto/action
On Fri, 8 Apr 2011 11:11:46 +0200 (CEST) vbriand@users.chorem.org wrote:
+ + /** + * Tests if a quotation already has a project order + * + * @param quotation + * @return true if the quotation already has a project order, false otherwise + */ + protected boolean quotationHasProjectOrder(Quotation quotation) {
Deux petites remarques valentin : 1) la première ligne d'une javadoc doit être une phrase (donc se terminer par un point) 2) le nom de méthode est pas top, j'opterais plutôt pour un *isQuotationHasProjectOrder* Du détail..., mais qui permets de faire avancer un peu le chmilblick :) -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com
Le 08/04/2011 11:38, Tony Chemit a écrit :
2) le nom de méthode est pas top, j'opterais plutôt pour un *isQuotationHasProjectOrder*
Du détail..., mais qui permets de faire avancer un peu le chmilblick :)
par contre moi j'aurais plutôt mis *hasProjectOrderQuotation* pour faire plus anglais parce que le is avec has ça le fait pas du tout ;) et comme ça tu as l'indicatif du boolean dans le debut du nom ;) Entre la proposition de Valentin et la tienne, je préfère celle de Valentin que je préfère d'ailleurs à ce que je viens de proposer, mais je comprends que tu veuilles que la partie du nom qui indique le type de retour soit au début. Jean
On Fri, 08 Apr 2011 11:54:04 +0200 Jean Couteau <couteau@codelutin.com> wrote:
Le 08/04/2011 11:38, Tony Chemit a écrit :
2) le nom de méthode est pas top, j'opterais plutôt pour un *isQuotationHasProjectOrder*
Du détail..., mais qui permets de faire avancer un peu le chmilblick :)
par contre moi j'aurais plutôt mis *hasProjectOrderQuotation* pour faire plus anglais parce que le is avec has ça le fait pas du tout ;) et comme ça tu as l'indicatif du boolean dans le debut du nom ;)
Entre la proposition de Valentin et la tienne, je préfère celle de Valentin que je préfère d'ailleurs à ce que je viens de proposer, mais je comprends que tu veuilles que la partie du nom qui indique le type de retour soit au début.
bah surtout je veut que ma méthode définisse une action pas une phrase ? projectHasBlabla ca me dit rien qui vaille. Apres on utilise souvent pas abus de Javabean un is pour un retour boolean, je pense que Java c'est pas écrire un texte en anglais non plus... :( et donc le hasXXX je suis pas fan non plus. La méthode de dit bien : Est-ce que le projet as des blabla ? d'ùou le isXXX
Jean _______________________________________________ Chorem-devel mailing list Chorem-devel@list.chorem.org http://list.chorem.org/cgi-bin/mailman/listinfo/chorem-devel
-- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com
Le 08/04/2011 12:07, Tony Chemit a écrit :
Apres on utilise souvent pas abus de Javabean un is pour un retour boolean, je pense que Java c'est pas écrire un texte en anglais non plus... :( et donc le hasXXX je suis pas fan non plus.
Je suis d'accord, mais faut que le nom soit cohérent, avec is et has dans le même nom ça ne l'est pas.
La méthode de dit bien : Est-ce que le projet as des blabla ? d'ùou le isXXX
Non, d'où le hasXXX justement, c'est bien 'as des blabla' et pas 'est un blabla'. Il faut retourner les choses (comme Ben l'a fait d'ailleurs dans sa proposition) pour avoir quelque chose de cohérent. Mais on chipotte, on chipotte.
On Fri, 8 Apr 2011 11:38:35 +0200 Tony Chemit <chemit@codelutin.com> wrote:
+ protected boolean quotationHasProjectOrder(Quotation quotation) {
...
1) la première ligne d'une javadoc doit être une phrase (donc se terminer par un point)
C'est parce que javadoc met en resume la 1ere phrase, donc sans point, y'a a pas de 1ere phrase. (mais je pense que dans ton cas ca marcherait tout de meme)
2) le nom de méthode est pas top, j'opterais plutôt pour un *isQuotationHasProjectOrder*
Et pourquoi pas simplement: isQuotationWithProjectOrder(Quotation quotation) On a le is (comme le reclame la norme bean, et le nom est tres compréhensible meme pour un francophone qui comprend a peine l'anglais :) -- Benjamin POUSSIN -------------------- tél: +33 (0) 2 40 50 29 28 email: poussin@codelutin.com http://www.codelutin.com
On Fri, 8 Apr 2011 12:50:33 +0200 Benjamin POUSSIN <poussin@codelutin.com> wrote:
On Fri, 8 Apr 2011 11:38:35 +0200 Tony Chemit <chemit@codelutin.com> wrote:
+ protected boolean quotationHasProjectOrder(Quotation quotation) {
...
1) la première ligne d'une javadoc doit être une phrase (donc se terminer par un point)
C'est parce que javadoc met en resume la 1ere phrase, donc sans point, y'a a pas de 1ere phrase. (mais je pense que dans ton cas ca marcherait tout de meme) Dans mes vagues souvenirs, cela fait parti de la norme de la javadoc : la ligne est une phrase :)
2) le nom de méthode est pas top, j'opterais plutôt pour un *isQuotationHasProjectOrder*
Et pourquoi pas simplement: isQuotationWithProjectOrder(Quotation quotation)
On a le is (comme le reclame la norme bean, et le nom est tres compréhensible meme pour un francophone qui comprend a peine l'anglais :)
+1 c'est très bon, en générale ça devient compliqué dès qu'on a du *has* dans le nom d'une méthode... -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com
Le 08/04/2011 13:18, Tony Chemit a écrit :
On Fri, 8 Apr 2011 12:50:33 +0200 Benjamin POUSSIN <poussin@codelutin.com> wrote:
On Fri, 8 Apr 2011 11:38:35 +0200 Tony Chemit <chemit@codelutin.com> wrote:
+ protected boolean quotationHasProjectOrder(Quotation quotation) {
...
1) la première ligne d'une javadoc doit être une phrase (donc se terminer par un point)
C'est parce que javadoc met en resume la 1ere phrase, donc sans point, y'a a pas de 1ere phrase. (mais je pense que dans ton cas ca marcherait tout de meme) Dans mes vagues souvenirs, cela fait parti de la norme de la javadoc : la ligne est une phrase :)
2) le nom de méthode est pas top, j'opterais plutôt pour un *isQuotationHasProjectOrder*
Et pourquoi pas simplement: isQuotationWithProjectOrder(Quotation quotation)
On a le is (comme le reclame la norme bean, et le nom est tres compréhensible meme pour un francophone qui comprend a peine l'anglais :)
+1 c'est très bon, en générale ça devient compliqué dès qu'on a du *has* dans le nom d'une méthode...
J'aime bien aussi
participants (3)
-
Benjamin POUSSIN -
Jean Couteau -
Tony Chemit