Use a switch instead of a cascading if-else.
diff --git a/src/main/java/org/apache/commons/codec/language/ColognePhonetic.java b/src/main/java/org/apache/commons/codec/language/ColognePhonetic.java
index b1ad0a6..a7b0302 100644
--- a/src/main/java/org/apache/commons/codec/language/ColognePhonetic.java
+++ b/src/main/java/org/apache/commons/codec/language/ColognePhonetic.java
@@ -368,17 +368,24 @@
}
} else if (arrayContains(DTX, chr)) {
output.put('8');
- } else if (chr == 'R') {
- output.put('7');
- } else if (chr == 'L') {
- output.put('5');
- } else if (chr == 'M' || chr == 'N') {
- output.put('6');
- } else if (chr == 'H') {
- output.put(CHAR_IGNORE); // needed by put
- } else {
- // ignored; should not happen
- }
+ } else
+ switch (chr) {
+ case 'R':
+ output.put('7');
+ break;
+ case 'L':
+ output.put('5');
+ break;
+ case 'M':
+ case 'N':
+ output.put('6');
+ break;
+ case 'H':
+ output.put(CHAR_IGNORE); // needed by put
+ break;
+ default:
+ break;
+ }
lastChar = chr;
}
diff --git a/src/main/java/org/apache/commons/codec/language/Nysiis.java b/src/main/java/org/apache/commons/codec/language/Nysiis.java
index 28ae275..6b5d2ea 100644
--- a/src/main/java/org/apache/commons/codec/language/Nysiis.java
+++ b/src/main/java/org/apache/commons/codec/language/Nysiis.java
@@ -126,20 +126,23 @@
}
// 2. Q -> G, Z -> S, M -> N
- if (curr == 'Q') {
- return CHARS_G;
- } else if (curr == 'Z') {
- return CHARS_S;
- } else if (curr == 'M') {
- return CHARS_N;
- }
+
// 3. KN -> NN else K -> C
- if (curr == 'K') {
+ switch (curr) {
+ case 'Q':
+ return CHARS_G;
+ case 'Z':
+ return CHARS_S;
+ case 'M':
+ return CHARS_N;
+ case 'K':
if (next == 'N') {
return CHARS_NN;
}
return CHARS_C;
+ default:
+ break;
}
// 4. SCH -> SSS