Add Deinflector.rulesMatch helper function (#530)

yomidevs · Jan 20, 2024 · be267e1 · be267e1
1 parent ff5db2c
commit be267e1
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/ext/js/language/deinflector.js b/ext/js/language/deinflector.js
@@ -59,7 +59,7 @@ export class Deinflector {
             for (const [reason, variants] of this.reasons) {
                 for (const [kanaIn, kanaOut, rulesIn, rulesOut] of variants) {
                     if (
-                        (rules !== 0 && (rules & rulesIn) === 0) ||
+                        !Deinflector.rulesMatch(rules, rulesIn) ||
                         !term.endsWith(kanaIn) ||
                         (term.length - kanaIn.length + kanaOut.length) <= 0
                     ) {
@@ -124,4 +124,15 @@ export class Deinflector {
         }
         return value;
     }
+
+    /**
+     * If `currentRules` is `0`, then `nextRules` is ignored and `true` is returned.
+     * Otherwise, there must be at least one shared rule between `currentRules` and `nextRules`.
+     * @param {number} currentRules
+     * @param {number} nextRules
+     * @returns {boolean}
+     */
+    static rulesMatch(currentRules, nextRules) {
+        return currentRules === 0 || (currentRules & nextRules) !== 0;
+    }
 }
diff --git a/ext/js/language/translator.js b/ext/js/language/translator.js
@@ -410,7 +410,7 @@ export class Translator {
             const definitionRules = Deinflector.rulesToRuleFlags(databaseEntry.rules);
             for (const deinflection of uniqueDeinflectionArrays[databaseEntry.index]) {
                 const deinflectionRules = deinflection.rules;
-                if (!partsOfSpeechFilter || deinflectionRules === 0 || (definitionRules & deinflectionRules) !== 0) {
+                if (!partsOfSpeechFilter || Deinflector.rulesMatch(deinflectionRules, definitionRules)) {
                     deinflection.databaseEntries.push(databaseEntry);
                 }
             }