Оптимально ли написан код?

Question

[Bestolkovy]

Добрый день!

Мне очень понравилась идея, которую придумал habrahabr.ru не расставлять теги параграфа в контенте.

Потратив 3 дня и задолбав весь toster.ru, родился такой код:

// Расставляем в конце каждой строки тег <br>
$content = preg_replace( '/\r?\n/miu', "<br>\r\n", $content );

// Убираем все теги <br>, которые находятся внутри контейнера <pre>...</pre>
$content = preg_replace_callback(

  '/<pre[^>]*?>([\s\S]*?)<\/pre>/miu',

  function ( $matches ) {
    return preg_replace( '/<br>/miu', '', $matches[0] );
  },

  $content

);

// Убираем лишние пустые строки с тегом <br>
$content = preg_replace( '/(<br>)(\s*\1)+/miu', "<br>\r\n<br>", $content );

Скажите новичку, можно ли его ещё как-нибудь оптимизировать? Или это оптимальное решение?

Comments

[romy4]

узнал свою регулярку. отметьте пожалуйста решением тот вопрос с br

Answer 1

[Сергей Протько]

1) для таких вещей не очень удобно использовать регулярки. Разбирать xml/html регулярками вообще вредно
2) логичнее в тег pre ничего не добавлять что бы ненароком не удалить лишнего. Вдруг там по замыслу автора уже стоял тег
.
3) проще сразу убрать лишние строки или еще круче - завернуть их в параграфы.

Comments

[Bestolkovy]

[цитата]логичнее в тег pre ничего не добавлять что бы ненароком не удалить лишнего. Вдруг там по замыслу автора уже стоял тег [/цитата].
Подскажите пожалуйста, как в данном случае, можно это реализовать?

[Сергей Протько]

Bestolkovy: не через регулярки.

Answer 2

[Андрей Буров]

вместо

preg_replace( '/\r?\n/miu', "<br>\r\n", $content );

есть nl2br

Comments

[Bestolkovy]

Если использовать функцию nl2br вместо preg_replace, то, почему-то перестаёт работать функция, которая удаляет лишние пустые строки: $content = preg_replace( '/(<br>)(\s*\1)+/miu', "<br>\r\n<br>", $content );.
А как их удалить сразу, везде, кроме контейнера pre, я не знаю пока. Если знаете, подскажите.

[Андрей Буров]

Bestolkovy: посмотрите примеры на php.net и станет все ясно, nl2br по дефолту меняет на

Answer 3

[romy4]

Оптимальность — это многофакторная метрика. Если выполняет задачу, выполняет быстро, то почему бы и нет.