кавычки

Можно и так:
" Ёлочки не нужны "

а я и просто могу написать: "ваши кавычки — говно то еще!"

Блин, ведь недавно совсем работало! «тест».

Упс, пашет.

Oh mein Gott!

Кто автор этого адового быдлокода?

https://github.com/maxcom/lorsource/blob/master/src/main/java/ru/org/linux/ut...

Впрочем я и так вижу что derlafff.

Т-рищ derlafff, я конечно понимаю что джава - прекрасный язык, на котором даже какашки будут работать, но пожалуйста, отрубите себе все 21 палец (особенно тот которым был написан этот код) и не мучайте больше мир своими испражнениями.

Этот ад надо переписать. Пацаны с винфака засмеют. maxcom, какие там у вас требование к разрабам? я вызываюсь добровольцем.

    if (isQuoteChar(before))
      return false;

    if (Character.isLetterOrDigit(after))
      return false;

    return true;

Г-споди, сделайте меня развидеть это!

// русский авось всегда спасет. авось прокатит :)

Адов ппц.

[bug] кавычки

Этого было бы достаточно для описания проблемы.

<<Кто хочет использовать ёлочки, может и так это делать>> (Бенджамин Франклер)

я вызываюсь добровольцем.

Гитхаб открыт. Шли реквесты.

// русский авось всегда спасет. авось прокатит :)

Троекратноёбаныехуепоняши!

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

Кстати, derlafff, а в чём хитрый смысл такого количества кода? Замену кавычек можно в 2 регулярных выражения сделать.

Сначала надо помощь извне: мне бы выдержку из базы с парой тысяч примеров с кавычками, т.е. ровно таких строк какие попадают на вход функции format(). А то не имея на чем отработать алгоритм я набыдлокодю ничуть не лучше.

А что там сложного?

\b("(\w+)")\b -> «\1»

, бери да адаптируй к джаве.

очевидно derlafff регекспы не осилил. я бы на месте maxcom реджектил пулл реквесты только изза javadoc не на английском.

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

Задача автозамены кавычек в общем случае не решается. паттерны «пробел кавычка буква» — «буква кавычка пробел» срабатывают далеко не всегда. Есть знаки препинания, скобки, переводы строк, лоркод, вложенные кавычки (мы ж хотим красоту «а „б“ в») итд.

пусть откатывают коммит ящитаю.

Перепись «специалистов»

Есть знаки препинания, скобки

Это вписывается в паттерн.

переводы строк

Если брать весь текст не глядя на переводы, то всё нормально отработает.

лоркод

Регулярка чуть усложнится добавлением «\[\w\]», сейчас лень придумывать полную форму. Но это реализуемо.

вложенные кавычки (мы ж хотим красоту «а „б“ в»)

Я бы не занимался такими извращениями, они глючат в большем количестве случаев, чем работают (в любой реализации). Но вообще и так можно сделать, добавив немного к тому же выражению.