文ごとにハイライトが出来ない

Question

文ごとにハイライトを行うプログラムを書いているのですが、
どうしても、最初の文だけがハイライトで表示され、しかも間違った部分をハイライトしてしまいます。
原因が分からないので、少しでも教えて頂けると幸いです。
※JSONファイルの「=>」の右側を取得し、ハイライトするプログラムを下記に記載しています。
dependencyRelationListとCountは今回は使っていませんので、空が入ります。

<実行結果>

※結果としては、「初期 と （初期 の二つをハイライトしたい。

'use strict';
import * as vscode from 'vscode';
import * as fs from 'fs';


export function activate(context: vscode.ExtensionContext) {
    console.log('Congratulations, your extension "semieditor" is now active!');

    const readsemiJson = JSON.parse(fs.readFileSync('C:\Users\specification\src\sample.json','utf8'));

    context.subscriptions.push(vscode.commands.registerCommand('extension.color', () => {
        vscode.window.showInformationMessage("Color Range Word");
        // 特定の単語に色付け
        const timeWordList = readsemiJson.時間;


        const timeWords: vscode.DecorationOptions[] = [];
        const dependencyRelationWords: vscode.DecorationOptions[] = [];

        var timeCount = 0;

        //child -> parent(値域) semiについて
        function fromObjecttoRange(WordList: any,foundWordList: vscode.DecorationOptions[],dependencyRelationList: vscode.DecorationOptions[],Count: number){
            const activeEditor = vscode.window.activeTextEditor;
            const text = activeEditor.document.getText(); //ドキュメント取得

            // // 取得したテキストを改行ごと(文章)に分割する
            // const sentence = text.match(/[^\r\n]*(\r\n|\r|\n|$)/g);
            var textArray = text.split(/\r\n|\r|\n/);

            if(WordList === timeWordList){
                for(var i = 0; i < readsemiJson['時間'].length; i++){
                    console.log(readsemiJson['時間'][i]);
                    const word = readsemiJson['時間'][i].split('=>');
                    // const parentword = word[0];
                    const rangeword = word[1];
                    for(var j = 0; j < textArray.length; j++){
                        console.log(textArray[j]);
                        if(textArray[j].indexOf(rangeword)){
                            const startPos = activeEditor.document.positionAt(textArray[j].indexOf(rangeword));
                            const endPos = activeEditor.document.positionAt(textArray[j].indexOf(rangeword) + rangeword.length);
                            // console.log(startPos, endPos);
                            const decoration = { range: new vscode.Range(startPos, endPos) };
                            foundWordList.push(decoration);
                        }
                    }
                }
                return {foundWordList,dependencyRelationList,Count};
            }
        }
        function decorateWord() {
            const activeEditor = vscode.window.activeTextEditor;
            // const text = activeEditor.document.getText(); //ドキュメント取得
    
            //時間
            fromObjecttoRange(timeWordList,timeWords,dependencyRelationWords,timeCount);
    
            activeEditor.setDecorations(timeMarkerDecoration,timeWords);
        }
        decorateWord();
    }));

        //時間に色付け
        const timeMarkerDecoration = vscode.window.createTextEditorDecorationType({
            'borderWidth': '1px',
            'borderRadius': '2px',
            'borderStyle': 'solid',
            // 'backgroundColor': 'rgba(255, 0, 0, 0.3)',
            'backgroundColor': 'rgba(128, 128, 0, 0.3)',
            // 'color':'red'
            'color':'orange'
        });
}    

{
    "時間": [
        "なし=>（初期",
        "なし=>「初期"
    ]
}

Answer 1

このへんを参考にすると

Positionインスタンス生成時に渡す数値は、行・桁関係ない先頭からの文字数のようです。

とあり、提示のロジックでは、改行で分割したあと、その行内ポジションでrangeを作っているので、例えばどこかの行の先頭に「初期や(初期があると、最初の行の先頭の#に色がつくのではないでしょうか？

対策としては、テキストの先頭からのオフセットを保持し、jのforループの最後で現在の行の長さをオフセットに加算、次の行に進んだときのstartPos endPosを求めるときにオフセットを組み込むとよいのではないかと想像します。
またこのさい、改行文字そのもの(\rや\n、\r\n)が何文字とカウントされるかによって、オフセットをtextArray[j]のlengthより1または2文字増やす必要があるかもしれないと思います。

Comments

[syen2501]

回答ありがとうございます。
ちなみになのですが、テキストの先頭からのオフセットを保持というのはどうやってやるんでしょうか？
文を読み込む際に、文の長さを変数で保持しfor文でループする際に加算するということで良いのでしょうか？

[papinianus]

それでいいと思います。
iのforのはじめあたりで、ゼロで初期化して、startPos=offSet+indexOfの値、としておいてjのループの最後でlengthを足す、感じかと。

[syen2501]

回答ありがとうございます。
textArrayで改行で分割した後だと改行が記載されなくなるので改行での判定が出来ないのですが、
分割する前で判定を行うしかないのでしょうか？

[papinianus]

var textArray = text.split(/(\r\n|\r|\n)/);
↑のところで、正規表現に()を含めると、splitに使った改行コードをtextArrayに含めることができます。例えばこの場合
> textArray = ["●#電源…判定する", "\r\n", ...];
のようになります(splitなので、偶数番目(インデクスが奇数の場所)に、正規表現でマッチした()で囲まれた部分が入ります)
なので、かなりイヤーなコードになってしまいますが、処理をかならず2個ずつ操作していけば、分割後の判定も可能になります
```
//for(var j = 0; j < textArray.length; j++){
for(var j = 0; j < textArray.length; j+=2){ //2個ずつやる。でtextArray[j]が本文テキストで、textArray[j+1]が改行コード。そのlengthをオフセットに加算する
```