J'ai pris un peu de temps pour regarder le résultat des derniers commits. Matthieu, je pense que la plupart de ces remarques sont pour toi. N'hésites pas à me demander si tu ne comprends pas certaines remarques et/ou à aller demander à un lutin. Essayes de voir avec Tony ou Éric pour configurer ton IDE afin d'éviter les imports du genre : +import javax.swing.*; +import java.awt.*; "when *touch* enter pressed" ... I'm pretty sure you ment *key*, right ? + if(selectedRow != table.getRowCount()){ + table.setRowSelectionInterval(table.getSelectedRow(), table.getSelectedRow()); + } else { + table.setRowSelectionInterval(table.getSelectedRow()-1, table.getSelectedRow()-1); + } Tu peux économiser au moins 3 table.getSelectedRow() au visuel, et 1 en runtime. Des logs de ce genre devraient plutôt : - être en "debug" ; - être en anglais. + if (log.isInfoEnabled()) { + log.info("Test sur JXDatePicker"); + } Tu as encore des soucis d'indentation. Cf FinancialTransactionTable#editCellAt(int,int,EventObject) Est-ce que la TVA ne devrait pas être dans l'AccountingRule en vigueur ? FinancialTransactionViewHandler#addEntry + //Actual (2012) tva percentage : 19,6% + BigDecimal tvaPercentAdd = new BigDecimal(0.196); Sur une méthode publique, essayes de plus respecter un format de documentation type javadoc : + //to reinitialize model attribute + public void resetAttribute(){ Sur cette même méthode (UnlettringSelectionModel#resetAttribute), je pense que tu as un gros bug, et que tu supprimes une ligne sur 2 (les lignes paires en l'occurrence) : for (int i = 0; i < selectedRows.size(); i ++){ selectedRows.remove(i); } Imagines que tu as une liste de 6 éléments : A B C D E F Tu supprimes l'élément à l'index 0 (A). Tous les éléments alors APRÈS A sont donc "déplacés" vers la gauche. B a alors l'index 0, C=1, D=2, ... B C D E F Tu incrémentes ton index : i=1. Tu supprimes l'élément à l'index 1 (C). B D E F Tu incrémentes ton index : i=2. Tu supprimes l'élément à l'index 2 (E). B D F Arrêt de la boucle car ta as attend le .size() = 2. Tu as donc supprimé tous les éléments pairs de ta liste :) Le plus simple aurait été d'utiliser une méthode pour vider la liste : selectedRows.clear(); -> Même problème dans LetteringSelectionModel#resetAttribute D'ailleurs, je vais peut-être dire une connerie, mais ces 2 classes ont l'air très similaire, est-ce qu'il n'aurait pas été possible de les mutualiser ? Bon courage, Arnaud