Кавычки — зло

Призываю капитана очевидность на помощь.

профилировка показывает что заметная часть CPU уходит на замену кавычек

Ха-ха!

За русские комменты зачет автоматом. Стереотип о том, что комменты должны быть на английском устарел и не нужен.

Стереотип о том, что комменты должны быть на английском устарел и не нужен.

Это стереотип верен для проектов, которые хотя бы теоретически могут понадобиться хоть кому-то, кроме хозяина.
С трудом себе представляю, чтобы кто-то мог заинтересоваться движком ЛОРа, не имея к ЛОРу никакого отношения.

Ну и вынесли бы эту функцию в JS. Заодно те, кому замена не нравится, могли бы этот скрипт заблокировать.

В эпоху машинного перевода достаточно просто писать грамотно.

Это не в стиле поней, доброта должна быть для всех!

if (position == 0 && localBuff.length()==0)
       return false;
     else if (position == buff.length() - 1)
       return true;

+    input = QUOTE_IN_OPEN_PATTERN.matcher(input).replaceAll(QUOTE_IN_OPEN_HTML);
+    input = QUOTE_IN_CLOSE_PATTERN.matcher(input).replaceAll(QUOTE_IN_CLOSE_HTML);
+    input = QUOTE_OUT_OPEN_PATTERN.matcher(input).replaceAll(QUOTE_OUT_OPEN_HTML);
+    input = QUOTE_OUT_CLOSE_PATTERN.matcher(input).replaceAll(QUOTE_OUT_CLOSE_HTML);

Пздц. За такой код надо совать автору руки под пресс.

Ужас. Зачем они вообще юзают ReplaceAll, если им надо заменять простой символ, а не проходиться regexp'ом? Replace — это неканонично?

Да ладно с replaceAll, но какая религия им запрещает положить это в мапу вместо этой лапши?

Ну и вынесли бы эту функцию в JS

Если посмотреть на тестовую базу данных, видно что в БД код уже хранится как html, даже html-entities заэскейпены.

Так видимо, определять, если открывающаяся кавычка типа

(\s|^)"

Я упрощу:

private final static Pattern QUOTE_IN_OPEN_PATTERN = Pattern.compile(Character.toString('„'), Pattern.LITERAL);
input = QUOTE_IN_OPEN_PATTERN.matcher(input).replaceAll("„");

так почему не заменять уже html, как юзерскрипт заменяет их на нормальные кавычки

Лень разбираться в чужом коде.

Велосипед пейсать тоже лень.

Ибо есть https://commons.apache.org/proper/commons-lang/ и тому подобное.

Так, что лень...

То что там творился писец можно было определить без профилировки, такого индусокода как нарожала пони я давно не видел.

4 прохода одного текста?

Это конченный автомат вместо конечного.

Да пофиг пока на проходы, мне не нравится сам вид этого кода. Такую лапшу может написать только человек, которого на стажировку уже взяли, но первый коде ещё проверить никто не успел.

не нравится сам вид этого кода

Ага, т.е. ты просто запихнул бы это в списочек и успокоился.

Как минимум. Конечно там надо всё это удалить и написать заново, но для этого нужен опыт. А запихнуть в списочек и обезьяна могла бы.

Да

Ибо есть https://commons.apache.org/proper/commons-lang/

Вот это верно, это плюсую. Их replace и вызывать придётся один раз вместо четырёх, и он работает раза в 3-4 быстрее дефолтового. Итого скорость вырастет где-то на порядок.

Таки, да.

Самое лучшее это просматривать кошерный код от группы Apache или Google Guava.

У этих ребят, как правило, «вылизанный» код, ибо у них большая база программистов, использующих код.

Они следят друг за другом, обмениваются идеями, «рефакторят».

кошерный код от группы Apache

https://commons.apache.org/proper/commons-lang/javadocs/api-release/org/apach... - абстрактный класс вместо функционального интерфейса, идиоты

и такого овна там полно

Я посмотрел их реализацию. Задумка те же, что в моей самоделке, но они пошли чуть дальше. Время (миллисекунд) на миллион замен на моей машине при длине текста в 1000 символов:

input.replace(from,to);							2339
QUOTE_IN_OPEN_PATTERN.matcher(input).replaceAll(to);			2104
input.replaceAll(from,to); // неожиданно для меня			1532
самодельная реализация через StringBuilder и toCharArray		1125
StringUtils.replace(input, from, to); //Apache				478

Напишите им, если Вы уверены в своей правоте.

но они пошли чуть дальше

Попробуй еще дальше: https://ru.wikipedia.org/wiki/Алгоритм_Ахо_—_Корасик

Зачем? я просто не пользуюсь этим легаси с убогим дизайном.

Это бессмысленно, т.к. для данной задачи искомая подстрока всегда состоит только из одного символа.

Гм, вощето я про generic алгоритм из StringUtils который содержит стопицот циклов.

А, ок, в общем случае может быть полезно.

Ужас. Зачем они вообще юзают ReplaceAll, если им надо заменять простой символ, а не проходиться regexp'ом? Replace — это неканонично?

Не принципиально — replace и replaceAll внутри работают через regexp'ы. Самое эффективное тут решение было бы просто пройтись циклом и заменить в нем нужные символы в один проход.

чем изменял, jmh?

На бухфак, одинэсник!

Не принципиально — replace и replaceAll внутри работают через regexp'ы. Самое эффективное тут решение было бы просто пройтись циклом и заменить в нем нужные символы в один проход.

Да, я так в своей реализации и делал. Там основной расход времени у меня был на StringBuilder.append, т.к. я тупо каждый символ append'ил отдельно, что не очень быстро.

чем изменял, jmh?

Да просто профилировщиком NetBeans'а. Загнал 5 функций замены последовательно в цикл и смотрел время выполнения каждой и долю времени.

Только что придумал ещё одну очень эффективную реализацию:

String.join(to, input.split(from))

У меня работает почти так же быстро, как апачевская реализация. Возможно, я чего-то ещё не учитываю, конечно.

А на лоре java 8?

Не знаю. Полагаете, что-то серьёзно изменилось в плане производительности? В любом случае, недолго проверить скорость исполнения на целевой машине и выбрать наилучший вариант.

Хочешь сказать, что ты свой псто написал неграмотно? Гуглотранслейт перевёл как

In the era of machine translation is simple enough to write correctly.

Я хоть и не англичанин, но явно недостаёт подлежащего (должно быть it is), вместо simple должно быть simply.

Но смысл понятен? А комментарии сами по себе вторичны, если без них ни черта не понятно — то либо код полно говно и не стоит внимания, либо читатель дурак и ему следует идти шаурмой торговать.

Полагаете, что-то серьёзно изменилось в плане производительности?

String.join со всякими Stream API

достаточно просто писать грамотно

Моё судно на воздушной подушке полно угрей.

String.join со всякими Stream API

О, точно, я и забыл, что это в новой яве появилось только.

Но смысл понятен?

Фраза из «В эпоху машинного перевода достаточно просто писать грамотно» превратилась в «В эру машинного перевода писать грамотно — достаточно просто». Изначальный смысл утерян.

От контекста много зависит. Да я и не говорю, что машперевод идеален.