feat(rule): 名詞同士に囲まれた、はカウントしない

箇条書きの、を判別するという目的 close #2
textlint-ja · Nov 27, 2015 · b65188d · b65188d
1 parent 110b303
commit b65188d
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 5 deletions.
diff --git a/src/max-ten.js b/src/max-ten.js
@@ -6,6 +6,17 @@ import {getTokenizer} from "kuromojin";
 import splitSentences from "sentence-splitter";
 import Source from "structured-source";
 const defaultOptions = {max: 3};
+
+function isSandwichedMeishi({
+    before,
+    token,
+    after
+    }) {
+    if (before === undefined || after === undefined || token === undefined) {
+        return false;
+    }
+    return before.pos === "名詞" && after.pos === "名詞";
+}
 /**
  * @param {RuleContext} context
  * @param {object} options
@@ -37,9 +48,18 @@ export default function (context, options = {}) {
                     let currentTenCount = 0;
                     let tokens = tokenizer.tokenizeForSentence(text);
                     let lastToken = null;
-                    tokens.forEach(token => {
+                    tokens.forEach((token, index) => {
                         let surface = token.surface_form;
                         if (surface === "、") {
+                            // 名詞に過去まわれている場合は例外とする
+                            let isSandwiched = isSandwichedMeishi({
+                                before: tokens[index - 1],
+                                token: token,
+                                after: tokens[index + 1]
+                            });
+                            if (isSandwiched) {
+                                return;
+                            }
                             currentTenCount++;
                             lastToken = token;
                         }
@@ -51,8 +71,8 @@ export default function (context, options = {}) {
                         if (currentTenCount >= maxLen) {
                             let position = source.indexToPosition(lastToken.word_position - 1);
                             let ruleError = new context.RuleError(`一つの文で"、"を${maxLen}つ以上使用しています`, {
-                                line: position.line-1,
-                                column:position.column
+                                line: position.line - 1,
+                                column: position.column
                             });
                             report(node, ruleError);
                             currentTenCount = 0;

diff --git a/test/max-ten-test.js b/test/max-ten-test.js
@@ -8,6 +8,7 @@ var tester = new TextLintTester();
 tester.run("max-ten", rule, {
     // default max:3
     valid: [
+        "名詞、名詞、名詞、名詞の場合は例外",
         {
             text: textIncludeTen(3 - 1)
         },
@@ -47,15 +48,15 @@ tester.run("max-ten", rule, {
             ]
         },
         {
-            text: `これは、長文、columnがちゃんと計算、されてるはずです。`,
+            text: `これは、長文の例ですが、columnがちゃんと計算、されてるはずです。`,
             options: {
                 "max": 3
             },
             errors: [
                 {
                     message: `一つの文で"、"を3つ以上使用しています`,
                     line: 1,
-                    column: 21
+                    column: 26
                 }
             ]
         },