Почему nl2br добавляет br бесконечно раз?

Question

[rinaz22]

Всем привет! Есть textarea. Чтобы были переносы строк, я использую nl2br. Потом если я хочу изменить ранее добавленный текст и нажимаю сохранить, то nl2br опять добавляет br и так до бесконечности, сколько изменил, столько br.
Пробовал регулярку использовать, но при редактировании текста, переносы исчезают, т.е. все в один ряд.
Регулярка:

preg_replace(array("/\\r\\n/", "/\\r/", "/\\n/"), "<br/>", $_POST['text'])

Comments

[rinaz22]

Всем спасибо. Дмитрий Ким правильно в начале написал, nl2br надо использовать только при выводе.
А вот на счёт: записать данные пользователя сразу в бд - это не безопасно.

[Дмитрий Ким]

А вот на счёт: записать данные пользователя сразу в бд - это не безопасно.

Записать в базу - это безопасно. Небезопасно - это вывести как есть.

[Артем Спиридонов]

У вас где-то на вводе рекурсия. Там ошибка в программе. Найдите её.

nl2br - всего-лишь обработка данных, нет особого значения, где именно делать её.

Answer 1

[Дмитрий Ким]

Секрет в том, что вам не нужно "изменять пользовательский ввод". Как пришли к вам данные из textarea, так вы их и храните в базе. Надо вывести на странице? Выводите измененный (nl2br) текст (strip_tags) или html. А храните как есть.

Comments

[Артем Спиридонов]

Жесть! Учитесь программировать!

Жаль, что тут нельзя материться.

[Дмитрий Ким]

Артем Спиридонов, поясните без матов.
Зачем менять то, что ввёл пользователь? Просто нужно безопасно это выводить.

[rinaz22]

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

[Артем Спиридонов]

Дмитрий Ким, хорошо, виноват, скажу мягче. Хотя не уверен, что смогу переубедить самого YII-разработчика.

Хранить пользовательский ввод - не лучшая идея. В Н.У. запись происходит в тысячи раз реже чтения, поэтому баланс преобразований рационально смещать в сторону записи. К тому же пользовательский ввод не валиден в общем случае и не безопасен.

[Дмитрий Ким]

rinaz22, тем более. Не трогайте то, что вводите в textarea.
Ваш процесс сохранения сейчас:
1. Взять текст из textarea, заменить все переводы строк на теги br.
2. Сохранить в базу.
3. Вывести на экран как есть из базы.
Ваш процесс редактирования сейчас:
1. Заменить все теги br обратно на переводы строк и поместить в textarea.
+ первые 3 пункта.

Ваш возможный процесс сохранения:
1. Взять текст из textarea и сохранить в базу.
2. Вывести на экран текст из базы через nl2br.
Ваш возможный процесс редактирования:
1. Вставить в textarea значение из базы.
+ первые 2 пункта.

[Дмитрий Ким]

Артем Спиридонов, когда пользователь вводит Иавнов вместо Иванов, или 9180 вместо 1980 вы тоже исправляете это? Что произойдет, если топикстартер прикрутит к textarea tinymce или CKEditor? Вы и там будете менять переводы строк на br?

[Артем Спиридонов]

Взять текст из textarea... Сохранить в базу.

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

Дмитрий Ким, я подозревал, что это бессмысленно, пытаться вас переубеждать в чем-либо

хорошо, виноват, скажу мягче. Хотя не уверен, что смогу переубедить самого YII-разработчика

[Дмитрий Ким]

Артем Спиридонов, Артём, сходите в сортир и избавьтесь от желчи.

[Артем Спиридонов]

Дмитрий Ким, хорошо. Уже.

Благодарю за заботу!

Answer 2

[Rsa97]

nl2br() не удаляет символы конца строк \r и \n, а только добавляет
после них.
А при использовании preg_replace переносы строк исчезают потому, что вы сами их удаляете.
И в HTML тэг br не надо закрывать, он пишется просто <br>

Comments

[Дмитрий Ким]

И в HTML тэг br не надо закрывать, он пишется просто

- одиночные теги принято закрывать: <br />, <hr /> <img /> <input />, ...

nl2br() не удаляет символы конца строк \r и \n, а только добавляет после них.

- перед ними.

[Rsa97]

Дмитрий Ким, Кем принято? Закрытие тэгов было обязательным в XHTML - гибриде XML и HTML. По стандарту HTML закрытие тэгов допускается, но не обязательно.

[Дмитрий Ким]

Rsa97, по поводу тегов согласен, не обязательно. "Принято мной" для визуализации незакрывающихся тегов :)

Answer 3

[Артем Спиридонов]

Алгоритм в целом покажите. Рекурсия там где-то.