Stratégie de mise en place de revues de code

@dwursteisen #DevoxxFR #CodeReview Stratégie de mise en place de revue
de code

@dwursteisen #DevoxxFR #CodeReview #DevoxxFR #CodeReview

@dwursteisen #DevoxxFR #CodeReview DEI Formation Consultant

@dwursteisen #DevoxxFR #CodeReview

@dwursteisen #DevoxxFR #CodeReview Cécile Jean-rené Xavier Ezequiel

@dwursteisen #DevoxxFR #CodeReview Moons Air Rail Ferry Car

@dwursteisen #DevoxxFR #CodeReview Moons Air Car Rail Ferry

@dwursteisen #DevoxxFR #CodeReview Mais qu’est-ce que c’est que CE CODE
?

@dwursteisen #DevoxxFR #CodeReview if ("Settings".equals(method)) { Settings request = exchange.getIn().getBody(Settings.class);
for (Element settingElement : request.elements()) { if (!users.contains(settingElement.getUsername())) { badUsers.add(settingElement.getUsername()); } } }

@dwursteisen #DevoxxFR #CodeReview if (!"Settings".equals(method)) { return; } Settings request
= exchange.getIn().getBody(Settings.class); for (Element settingElement : request.elements()) { if (!users.contains(settingElement.getUsername())) { badUsers.add(settingElement.getUsername()); } }

@dwursteisen #DevoxxFR #CodeReview LOG.warn(new StringBuilder("No user found").toString());

@dwursteisen #DevoxxFR #CodeReview LOG.warn(new StringBuilder("No user found").toString()); Fausse optimisation

@dwursteisen #DevoxxFR #CodeReview private static final TO_BE_REPLACED = new char[]
{ '_', ','; '-'}; public static String normalize(final String string) { if (string == null) { return null; } String normalizedString = string; for (char charToReplace : TO_BE_REPLACED) { normalizedString = normalizedString.replace(charToReplace, ' '); } return normalizedString.trim(); }

@dwursteisen #DevoxxFR #CodeReview private static final TO_BE_REPLACED = new char[]
{ '_', ','; '-'}; public static String normalize(final String string) { if (string == null) { return null; } String normalizedString = string; for (char charToReplace : TO_BE_REPLACED) { normalizedString = normalizedString.replace(charToReplace, ' '); } return normalizedString.trim(); } Remplace des caractères spéciaux par des espaces

@dwursteisen #DevoxxFR #CodeReview private static final String PATTERN = "[_,-]";
public static String trimSpecialCharacters(final String string) { if (string == null) { return null; } return StringUtils.trim(string.replaceAll(PATTERN, " ")); }

@dwursteisen #DevoxxFR #CodeReview private static final String PATTERN = "[_,-]";
public static String trimSpecialCharacters(final String string) { if (string == null) { return null; } return StringUtils.trim(string.replaceAll(PATTERN, " ")); } Remplace (également) des caractères spéciaux par des espaces

@dwursteisen #DevoxxFR #CodeReview + - ↗ ?

@dwursteisen #DevoxxFR #CodeReview + - ↗ ? Qualité du code
!

@dwursteisen #DevoxxFR #CodeReview + - ↗ ? Qualité du code
! revue de code ?

@dwursteisen #DevoxxFR #CodeReview mise en place de revue de code
systématique

@dwursteisen #DevoxxFR #CodeReview A Faire En cours Implémentée Terminée MNS-
456 MNS-135 MNS- 265

@dwursteisen #DevoxxFR #CodeReview AUCUNE tâche n’a été passée en REVUE
DE CODE

@dwursteisen #DevoxxFR #CodeReview + - ↗ ?

@dwursteisen #DevoxxFR #CodeReview + - ↗ ? revue de code
!

@dwursteisen #DevoxxFR #CodeReview mise en place de revue de code
quand c’est nécessaire

456 MNS-135 MNS- 265

@dwursteisen #DevoxxFR #CodeReview (toujours) AUCUNE tâche n’a été passée en
REVUE DE CODE

@dwursteisen #DevoxxFR #CodeReview Code review

456 MNS-135 MNS- 265 Code review

456 MNS-135 MNS- 265 Code review Code review

456 MNS-135 MNS- 265 Code review Code review Revue de code nécessaire

456 MNS-135 MNS- 265 Code review Code review

@dwursteisen #DevoxxFR #CodeReview Comment est faîte une revue de code
?

@dwursteisen #DevoxxFR #CodeReview To: [email protected] Subject : Revue de code
MNS-435 Salut ! J’ai regardé le code de la tâche MNS-435. Mes retours : Dans la class BookManagerAugmented, je pense que la variable toto, ligne 42 n’est pas très bien nommé. if (toto == null) { return false; } Que penses-tu de changer son nom par tata plutôt ? C’est mieux non ?

@dwursteisen #DevoxxFR #CodeReview To: [email protected] Subject : Revue de code
MNS-435 Salut ! J’ai regardé le code de la tâche MNS-435. Mes retours : Dans la class BookManagerAugmented, je pense que la variable toto, ligne 42 n’est pas très bien nommé. if (toto == null) { return false; } Que penses-tu de changer son nom par tata plutôt ? C’est mieux non ? Mauvaise Idée

@dwursteisen #DevoxxFR #CodeReview Code

@dwursteisen #DevoxxFR #CodeReview Commentaires

@dwursteisen #DevoxxFR #CodeReview if (s == null) { return false;
}

@dwursteisen #DevoxxFR #CodeReview if (s != null) { return true;
}

@dwursteisen #DevoxxFR #CodeReview Quels impacts sur le code ?

@dwursteisen #DevoxxFR #CodeReview if(Integer.ZERO.equals(amount)) { StringBuilder sb = new StringBuilder();
sb.append("Deal ID is").append(id) .append(" and deal amount is ").append(amount) .append(". "); LOGGER.error(sb.toString()); throw new ValidationException("Price error. Check the price"); }

@dwursteisen #DevoxxFR #CodeReview if(Integer.ZERO.equals(amount)) { LOGGER.error("Deal ID is" + id
+ " and deal amount is " + amount + ". "); throw new ValidationException("Price error. Check the price"); }

@dwursteisen #DevoxxFR #CodeReview if(Integer.ZERO.equals(amount)) { LOGGER.error("Deal ID is {} and
deal amount is {}. ", id, amount); throw new ValidationException("Price error. Check the price"); }

@dwursteisen #DevoxxFR #CodeReview if(Integer.ZERO.equals(amount)) { LOGGER.warn("Deal ID is {} and
deal amount is {}. ", id, amount); throw new ValidationException("Price error. Check the price"); }

@dwursteisen #DevoxxFR #CodeReview Quels impacts sur l’équipe ?

@dwursteisen #DevoxxFR #CodeReview Binômage

@dwursteisen #DevoxxFR #CodeReview aujourd’hui ?

@dwursteisen #DevoxxFR #CodeReview Il faut porter attention à ce que
cette pratique ne se dissipe pas

@dwursteisen #DevoxxFR #CodeReview Questions ? (je suis là pour y
répondre)

Stratégie de mise en place de revues de code

Stratégie de mise en place de revues de code

More Decks by David

Other Decks in Programming

Featured

Transcript