chore(2.2.2): エラー表示を改善

Author: azu

src/2.2.2.js

@@ -2,7 +2,7 @@
 "use strict";
 import {isUserWrittenNode} from "./util/node-util";
 function matchToReplace(text, pattern, matchFn) {
-    var match = text.match(pattern);
+    var match = pattern.exec(text);
     if (match) {
         return matchFn(text, pattern, match);
     }
@@ -115,26 +115,34 @@ export default function (context) {
             let text = getSource(node);
             // 漢数字 -> 算用数字
             let toNumber = (text, pattern, match) => {
-                var expected = text.replace(pattern, function (all, match) {
+                let matchedString = match[0];
+                let index = match.index;
+                var expected = matchedString.replace(pattern, function (all, match) {
                     let result = 0;
                     match.split("").forEach(kanNumber => {
                         result += numberMap[kanNumber];
                     });
                     return all.replace(match, result);
                 });
-                report(node, new RuleError(`${text} => ${expected}
-数量を表現し、数を数えられるものは算用数字を使用します。任意の数に置き換えても通用する語句がこれに該当します。`));
+                var ruleError = new RuleError(`${matchedString} => ${expected}
+数量を表現し、数を数えられるものは算用数字を使用します。任意の数に置き換えても通用する語句がこれに該当します。`, {
+                    column: index
+                });
+                report(node, ruleError);
             };
 
 
             // 算用数字 -> 漢数字
 
             let toKanNumber = (text, pattern, match) => {
-                var expected = text.replace(pattern, function (all, match) {
+                var matchedString = match[0];
+                var expected = matchedString.replace(pattern, function (all, match) {
                     return all.replace(match, _num2ja(match, {'with_arabic': false}));
                 });
-                report(node, new RuleError(`${text} => ${expected}
-慣用的表現、熟語、概数、固有名詞、副詞など、漢数字を使用することが一般的な語句では漢数字を使います。`));
+                report(node, new RuleError(`${matchedString} => ${expected}
+慣用的表現、熟語、概数、固有名詞、副詞など、漢数字を使用することが一般的な語句では漢数字を使います。`, {
+                    column: matchedString.index
+                }));
             };
 
             // 数えられる数字は算用数字を使う

test/2.2.2-test.js

@@ -14,22 +14,41 @@ tester.run("2.2.2.算用数字と漢数字の使い分け", rule, {
         {
             text: "一億百十万人",
             errors: [
-                {message: `一億百十万人 => 1億110万人
-数量を表現し、数を数えられるものは算用数字を使用します。任意の数に置き換えても通用する語句がこれに該当します。`}
+                {
+                    message: `一億 => 1億
+数量を表現し、数を数えられるものは算用数字を使用します。任意の数に置き換えても通用する語句がこれに該当します。`,
+                    line: 1,
+                    column: 1
+                }
             ]
         },
         {
             text: "百八つのボタン",
             errors: [
-                {message: `百八つのボタン => 108つのボタン
-数量を表現し、数を数えられるものは算用数字を使用します。任意の数に置き換えても通用する語句がこれに該当します。`}
+                {
+                    message: `百八つ => 108つ
+数量を表現し、数を数えられるものは算用数字を使用します。任意の数に置き換えても通用する語句がこれに該当します。`
+                }
             ]
         },
         {
             text: "第三回大会",
             errors: [
-                {message: `第三回大会 => 第3回大会
-数量を表現し、数を数えられるものは算用数字を使用します。任意の数に置き換えても通用する語句がこれに該当します。`}
+                {
+                    message: `三回 => 3回
+数量を表現し、数を数えられるものは算用数字を使用します。任意の数に置き換えても通用する語句がこれに該当します。`
+                }
+            ]
+        },
+        {
+            text: "サンフランシスコマラソン第三回大会",
+            errors: [
+                {
+                    message: `三回 => 3回
+数量を表現し、数を数えられるものは算用数字を使用します。任意の数に置き換えても通用する語句がこれに該当します。`,
+                    line: 1,
+                    column: 14
+                }
             ]
         }
     ]
@@ -55,31 +74,39 @@ tester.run("2.2.2.算用数字と漢数字の使い分け", rule, {
     ],
     invalid: [
         {
-            text: "世界1",
+            text: "これは世界1",
             errors: [
-                {message: `世界1 => 世界一
-慣用的表現、熟語、概数、固有名詞、副詞など、漢数字を使用することが一般的な語句では漢数字を使います。`}
+                {
+                    message: `世界1 => 世界一
+慣用的表現、熟語、概数、固有名詞、副詞など、漢数字を使用することが一般的な語句では漢数字を使います。`
+                }
             ]
         },
         {
-            text: "1部の",
+            text: "1部の文章",
             errors: [
-                {message: `1部の => 一部の
-慣用的表現、熟語、概数、固有名詞、副詞など、漢数字を使用することが一般的な語句では漢数字を使います。`}
+                {
+                    message: `1部の => 一部の
+慣用的表現、熟語、概数、固有名詞、副詞など、漢数字を使用することが一般的な語句では漢数字を使います。`
+                }
             ]
         },
         {
-            text: "1番に",
+            text: "朝1番に",
             errors: [
-                {message: `1番に => 一番に
-慣用的表現、熟語、概数、固有名詞、副詞など、漢数字を使用することが一般的な語句では漢数字を使います。`}
+                {
+                    message: `1番に => 一番に
+慣用的表現、熟語、概数、固有名詞、副詞など、漢数字を使用することが一般的な語句では漢数字を使います。`
+                }
             ]
         },
         {
             text: "数100倍",
             errors: [
-                {message: `数100倍 => 数百倍
-慣用的表現、熟語、概数、固有名詞、副詞など、漢数字を使用することが一般的な語句では漢数字を使います。`}
+                {
+                    message: `数100倍 => 数百倍
+慣用的表現、熟語、概数、固有名詞、副詞など、漢数字を使用することが一般的な語句では漢数字を使います。`
+                }
             ]
         }
     ]