Optimisation de mon BreadCrumb

Question optimisation, c'est un peu tout pareil.

Pour le rendre plus clair, le mieux serait de programmer en montrant directement ton intention. Je te donne un exemple :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
static List<String> extractPathTokens(String path) {
  path = removeHtml(path);
  List<String> tokens = separateTokens(path);
  String token2 = tokens.get(1);
  token2 = removeNumbers(token2);
  token2 = replaceSeparators(token2);
  tokens.set(1, token2);
  return tokens;
}
 
private static String removeHtml(String path) {
    final String SUFFIX = ".html";
    if(path.endsWith(SUFFIX)) {
        return path.substring(0, path.length() - SUFFIX.length());
    } else {
        return path;
    }
}
 
private static final Pattern PATH_PATTERN = Pattern.compile("/(.+?)/(.*)");
 
private static List<String> separateTokens(String path) {
  Matcher matcher = PATH_PATTERN.matcher(path);
  if(!matcher.matches()) {
    throw new IllegalArgumentException("Unknown path pattern: " + path);
  }
  List<String> tokens = new ArrayList<>();
  tokens.add(matcher.group(1));
  tokens.add(matcher.group(2));
  return tokens;
}
 
private static String removeNumbers(String token) {
  // On veut pouvoir remplacer /12/blablah par /blablah alors on force un / devant, qu'on enlèvera ensuite
  token = "/" + token;
  token = token.replaceAll("/[0-9]+", "");
  // on enlève le / ajouté au début
  return token.substring(1);
}
 
private static String replaceSeparators(String token) {
  return token.replace('/', '.');
}

Le programme est bien décomposé en méthodes qui ont chacune leur travail, chaque ligne est claire, ça se relit assez bien, et c'est pas trop compliqué de se replonger dedans s'il y a des choses à changer.