Feature/tab alignment

[asosnoviy]

Описание

tab alignment diagnostic

Связанные задачи

Closes: #197

Чеклист

Общие

• Ветка PR обновлена из develop
• Отладочные, закомментированные и прочие, не имеющие смысла участки кода удалены
• Изменения покрыты тестами
• Обязательные действия перед коммитом выполнены (запускал команду gradlew precommit)

Для диагностик

• Описание диагностики заполнено для обоих языков (перевод на английский не обязателен)

Дополнительно

[nixel2007]

Перфоманс алерт жёсткий Оо

[asosnoviy]

Перфоманс алерт жёсткий Оо

да трындец, дороже запросов

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

100.0% Coverage
0.0% Duplication

[nixel2007]

А регулярка на весь контент, а не на строку, не будет дешевле?

[asosnoviy]

А регулярка на весь контент, а не на строку, не будет дешевле?

А как с нее позиции посчитать потом?

[nixel2007]

через groups (или как-то так) у MatcherResult. как в оскрипте РегулярноеВыражение.НайтиСовпадения()[0].Группы[1]....

[theshadowco]

а так не быстрее будет:

• получить все не вайтспейс токены
• получить все вайтспейс токены
• пройтись по вайспейстокенам и с условием
  • не перенос строки
  • предыдущий токен есть в списке не вайтспейсов
  • токен содержит таб

[theshadowco]

@asosnoviy

[theshadowco]

порядок условий можно поменять

[asosnoviy]

через groups (или как-то так) у MatcherResult. как в оскрипте РегулярноеВыражение.НайтиСовпадения()[0].Группы[1]....

а там номера строк будут? не только смещение?

[nixel2007]

@asosnoviy

Вообще смещение, да. Возможно по смещению как-то можно найти позицию в исходном content и сматчить ее на contentList... Но вообще смотри, что я нашел - https://docs.oracle.com/javase/7/docs/api/java/io/LineNumberReader.html

Может быть если чуть-чуть переписать получение contentList, то и по оффсету будет проще искать позицию?

Дока по матчеру:

https://docs.oracle.com/javase/9/docs/api/java/util/regex/Matcher.html

Методы start и end могут в себя принимать номер группы. количество групп известно из groupCount.
Возможно нужно еще явно вызвать toMatchResult(), чтобы получить отдельный потокобезопасный инстанс с результатами, а то кажется, что приколы с параллельностью могут выстрелить.

[asosnoviy]

Вспомнить бы зачем нужен contentList. Несколько раз пытался его заюзать, и кажется ни разу не вышло.

[asosnoviy]

И ще заметил, что "/n/t" липнут в один токен и тогда строка у него начинает врать

[nixel2007]

И ще заметил, что "/n/t" липнут в один токен и тогда строка у него начинает врать

а там строка - это ж строка начала токена.

[asosnoviy]

И ще заметил, что "/n/t" липнут в один токен и тогда строка у него начинает врать

а там строка - это ж строка начала токена.

Может отделим переносы от всех вайтспейсов?

[nixel2007]

а зачем? я помню какое-то одно место, где схлопнутые вайтспейсы мешали. в остальном меньше токенов - меньше памяти :)

[theshadowco]

надо написать парсер , где значаей инормацией будут вайтспейсы :)