refactor(4.2.6): fix to base global regexp

Author: azu

src/1.2.1.js

@@ -3,6 +3,7 @@
 import regx from 'regx';
 import {japaneseRegExp} from "./util/regexp";
 import {matchCaptureGroupAll} from "./util/match-index";
+import mergeMatches from "./util/merge-matches";
 const rx = regx("g");
 /*
 1.2.1. 句点(。)と読点(、)
@@ -27,23 +28,6 @@ const replaceSymbol = {
     ",": "、"
 };
 
-
-function mergeMatches(...aMatches) {
-    const results = [];
-    aMatches.forEach(matches => {
-        matches.forEach(targetMatch => {
-            const alreadyHave = results.some(match => {
-                const {text, index} = match;
-                return targetMatch.index === index && targetMatch.text === text;
-            });
-            if (!alreadyHave) {
-                results.push(targetMatch);
-            }
-        });
-    });
-    return results;
-}
-
 const reporter = (context) => {
     let {Syntax, report, fixer, getSource} = context;
     return {

src/4.2.4.js

@@ -10,6 +10,9 @@
  */
 import {isUserWrittenNode} from "./util/node-util";
 import {matchCaptureGroupAll} from "./util/match-index";
+import regx from 'regx';
+import {japaneseRegExp} from "./util/regexp";
+const rx = regx("g");
 function reporter(context) {
     let {Syntax, RuleError, report, fixer, getSource} = context;
     return {
@@ -19,7 +22,7 @@ function reporter(context) {
             }
             const text = getSource(node);
             // 和文で半角の･は利用しない
-            const matchHanNakaguro = /(･)/g;
+            const matchHanNakaguro = rx`(?:${japaneseRegExp}|[a-zA-Z])(･)(?:${japaneseRegExp}|[a-zA-Z])`;
             matchCaptureGroupAll(text, matchHanNakaguro).forEach(match => {
                 const {index} = match;
                 report(node, new RuleError("カタカナ複合語を区切る場合または同格の語句を並列する場合には全角の中黒（・）を使用します。", {

src/4.2.5.js

@@ -15,7 +15,7 @@ function reporter(context) {
                 return;
             }
             const text = getSource(node);
-            // 和文で半角の?は利用しない
+            // 数値の区切りに半角の~は利用しない
             const matchHanQuestion = /\d(~)\d/g;
             matchCaptureGroupAll(text, matchHanQuestion).forEach(match => {
                 const {index} = match;

src/4.2.6.js

@@ -7,6 +7,11 @@
 例外は、住所や電話番号の区切りに使う場合です。
  */
 import {isUserWrittenNode} from "./util/node-util";
+import {matchCaptureGroupAll} from "./util/match-index";
+import regx from 'regx';
+import {japaneseRegExp} from "./util/regexp";
+import mergeMatches from "./util/merge-matches";
+const rx = regx("g");
 export default function (context) {
     let {Syntax, RuleError, report, getSource} = context;
     return {
@@ -15,18 +20,19 @@ export default function (context) {
                 return;
             }
             let text = getSource(node);
-            // 基本的にハイフン(-)を使用しません
-            var index = text.indexOf("-");
-            if (index !== -1) {
-                // 例外として、住所や電話番号の区切りにはハイフン(-)を使用できる
-                // ?-? を取り出して \d-\d ならOK、そうでないならダメ
-                var bufOfIndex = text.slice(index - 1, index + 2);
-                // [^A-Za-z0-9_] = \w
-                if (!/^\w\-\w$/.test(bufOfIndex)) {
-                    report(node, new RuleError(`原則として和文ではハイフン(-)を使用しません。
-例外は、住所や電話番号の区切りに使う場合です。`, index));
-                }
-            }
+            // 和文ではハイフン(-)を使用しません
+            // right
+            const rightMatches = matchCaptureGroupAll(text, rx`${japaneseRegExp}(\-)`);
+            // left
+            const leftMatches = matchCaptureGroupAll(text, rx`(\-)${japaneseRegExp}`);
+            const matches = mergeMatches(leftMatches, rightMatches);
+            matches.forEach(match => {
+                const {index} = match;
+                report(node, new RuleError(`原則として和文ではハイフン(-)を使用しません。
+例外は、住所や電話番号の区切りに使う場合です。`, {
+                    column: index
+                }))
+            });
         }
     };
 }

src/util/match-index.js

@@ -2,6 +2,7 @@
 "use strict";
 const flagsGetter = require('regexp.prototype.flags');
 const assert = require("assert");
+
 /**
  * @typedef {Object} MatchCaptureGroup
  * @property {string} text - text is matched texts

src/util/merge-matches.js

@@ -0,0 +1,17 @@
+// LICENSE : MIT
+"use strict";
+export default function mergeMatches(...aMatches) {
+    const results = [];
+    aMatches.forEach(matches => {
+        matches.forEach(targetMatch => {
+            const alreadyHave = results.some(match => {
+                const {text, index} = match;
+                return targetMatch.index === index && targetMatch.text === text;
+            });
+            if (!alreadyHave) {
+                results.push(targetMatch);
+            }
+        });
+    });
+    return results;
+}

test/4.2.6-test.js

@@ -21,11 +21,11 @@ tester.run("4.2.6.ハイフン(-)", rule, {
             ]
         },
         {
-            text: "090 - 1234 - 5678",
+            text: "A市-B市",
             errors: [
                 {
                     message: "原則として和文ではハイフン(-)を使用しません。\n例外は、住所や電話番号の区切りに使う場合です。",
-                    column: 5
+                    column: 3
                 }
             ]
         }