Функция str_replace() не работает?

Question

[JastaFly]

Есть вот такой вот незамысловатый код:

$this->title = str_replace('"', '', $seo['ELEMENT_META_TITLE']) ?? null;

Который должен убрать символьный код кавычек из строки, но этого почему-то не происходит. Модифицирую данный код, чтобы посмотреть что там происходит:

echo '<pre>';
        var_dump($seo['ELEMENT_META_TITLE']);
        $this->title = str_replace('&quot;', '', $seo['ELEMENT_META_TITLE']) ?? null;
        var_dump(str_replace('&quot;', '', $seo['ELEMENT_META_TITLE']));
        var_dump($this->title);

Получаю следующий вывод:

string(275) "Хомут обжимной просечной (оцинкованная сталь) ширина 8 мм 3/4" (13-19 мм) – купить по цене завода. Доставка по РФ. Смотреть характеристики, фото, отзывы."
string(275) "Хомут обжимной просечной (оцинкованная сталь) ширина 8 мм 3/4" (13-19 мм) – купить по цене завода. Доставка по РФ. Смотреть характеристики, фото, отзывы."
string(275) "Хомут обжимной просечной (оцинкованная сталь) ширина 8 мм 3/4" (13-19 мм) – купить по цене завода. Доставка по РФ. Смотреть характеристики, фото, отзывы."

То есть ничего не меняется, как будто бы функция не работает str_replace(). Но самое интересное что на тестовой площадке всё работает как надо и нежелательные символы уходят из строки, а вот в проде уже нет. Подскажите пожалуйста что за мистика?

Comments

[Дмитрий]

Ну вы меняете одно, а смотрите другое

[Дмитрий]

Сделайте $this->title = $seo['ELEMENT_META_TITLE'];

[Дмитрий]

Ну или там амперсанд тоже сущность хтмл
Посмотрите строку через ctrl + u в арбузере

[Vitsliputsli]

Проверяйте вашу строку на непечатные символы. У вас:

string(275) "Хомут обжимной просечной (оцинкованная сталь) ширина 8 мм 3/4" (13-19 мм) – купить по цене завода. Доставка по РФ. Смотреть характеристики, фото, отзывы."

У меня:

string(271) "Хомут обжимной просечной (оцинкованная сталь) ширина 8 мм 3/4" (13-19 мм) – купить по цене завода. Доставка по РФ. Смотреть характеристики, фото, отзывы."

гдето прячутся еще 4 байта, быть может quot это не 1-байтовая латиница, а чтото иное.

[Ипатьев]

а зачем в этом незамысловатом коде стоит ?? null?

[Adamos]

Ипатьев, это РНР-хайку:
Если значенье
Окажется null в переменной -
Пусть null и будет.

[Сергей delphinpro]

Adamos, загвоздка в том, что на выходе str_replace никогда не будет null

Вероятно задумывалось сделать так

str_replace('"', '', $seo['ELEMENT_META_TITLE'] ?? '');

[Ипатьев]

Vitsliputsli, Как заметил Дмитрий, "Ну или там амперсанд тоже сущность хтмл" - как раз 4 лишних байта.

Сначала напихиваем кучу htmlscpacialchars, причем на входе, а потом не понимаем, почему у нас не работает str_replace

[Adamos]

Если выводить через нормальный шаблонизатор - можно кавычки и не удалять.
Вся эта маета из-за того, что где-то дальше этот title лепится в верстку банальной конкатенацией.

[Ипатьев]

Adamos, это могут быть какие-то сео заморочки. но кстати в целом да, по уму вообще ничего удалять не надо

[JastaFly]

Дмитрий, Ипатьев, Vitsliputsli, Сергей delphinpro, Adamos,

Ну вы меняете одно, а смотрите другое

Дешифруйте пожалуйста?!

Сделайте $this->title = $seo['ELEMENT_META_TITLE'];

Не поверите именно так оно работало до того как пришло и вызывала проблему которую я и пытаюсь сейчас решить

Вероятно задумывалось сделать так

Да ничего так не задумывалось, правится древнее говнокодерское легаси без нормальной типизации. Как по мне null тут нах не нужен, но убирать стрёмно, хрен знает где ещё что может отвалиться при таких резких мувах

"Ну или там амперсанд тоже сущность хтмл" - как раз 4 лишних байта.

В яблочко

Сначала напихиваем кучу htmlscpacialchars, причем на входе,

А Вы думаете обработчик входных данные и их вывод всегда пишется одним и тем человеком?

по уму вообще ничего удалять не надо

Ды да все эти коды < > " ' & на фронте смотрятся не разу не убого, а заботливо оставленные бекендером двойные кавычки в строках, не разу не разматывают вёрстку к хренам. Очень по уму!!!

[Ипатьев]

Ды да все эти коды < > " ' & на фронте смотрятся не разу не убого

Вот поэтому их и не надо добавлять!

[JastaFly]

Ипатьев,

Вот поэтому их и не надо добавлять!

Суть моей проблемы в том что они уже туда добавлены

[Ипатьев]

Для непонятливых ещё раз: суть проблемы именно в добавлении. А потуги с использованием примитивной функции, которая "не работает" - это её следствия
Исправлять надо всегда причину, а не следствие.

[JastaFly]

Ипатьев,

Исправлять надо всегда причину, а не следствие.

Чтобы исправить причину нужно зачистить от этих символов базу с сотнями тысяч товаров и перелопатить часть ядра Битрикса отвечающую за выгрузку товаров. Я понимаю для бешеной собаки 100 вёрст не крюк, но мне есть чем ещё заняться

[Vitsliputsli]

JastaFly, он просто указал, как сделать правильно, чтобы не только решить текущую проблему, но и предотвратить проблемы в будущем. Если это сейчас слишком сложно, то "лечите" следствия, будет костыльно, но решит проблему.

Answer 1

[Сергей delphinpro]

str_replace работает. Просто в строке нет того, что вы ищете.
Вероятно там такое ширина 8 мм 3/4"

Comments

[nokimaro]

Это подтверждается и тем что var_dump в примере в вопросе показывает 275 байт, но если проверить строку с & то будет 271 то есть как-раз не хватает amp;
А автор вопроса этого не видит так как смотрит вывод var_dump в браузере где & отображается ему как &

[JastaFly]

Сергей delphinpro, nokimaro,
Всё так, написал простенький метод, чтобы вычищать всё это из строк:

public function clearFromHtmlSpecificChars(string $text): string
    {
        $search = ['<', '>', '"', ''', '&', 'quot;', 'amp;'];
        $replace = ['', '', '', '', '', '', ''];

        return str_replace($search, $replace, $text);
    }

[Сергей delphinpro]

JastaFly,

preg_replace('/&[a-z]+;/', '', $string);

[JastaFly]

Сергей delphinpro, не пойдёт, встречаются символьные коды и без амперсанда

Answer 2

[Виктор]

html_entity_decode()

https://onlinephp.io/c/9d203 (при работе с этой строкой, учтите, что в строке есть кавычка)

Comments

[JastaFly]

Виктор пробовал, не помогло

[Виктор]

JastaFly, видимо некорректно попробовали, в песочнице (onlinephp.io) работает корректно, пример я приложил.

[JastaFly]

Виктор,

видимо некорректно попробовали,

Всё было корректно, но данная функция не позволила мне избавиться от всех кодов HTML символов, некоторые всё равно проскальзывали

[Виктор]

JastaFly, возможно поможет указание flags, описание параметра есть в документации по первой ссылке в ответе

Answer 3

[Ипатьев]

В строке не должно быть никакого"
Надо взять железную линейку, и надавать по рукам тому, кто писал этот код.
После этого убрать из кода все вызовы функции htmlscpacialchars, все filter_var c идиотским FILTER_SANITIZE_STRING и вообще никак не портить входящие данные
И после этого спокойно удалять из строки обычную двойную кавычку.

А выводить любые данные через нормальный шаблонизатор.

Comments

[JastaFly]

@ipatiev

Надо взять железную линейку, и надавать по рукам тому, кто писал этот код.

Прекрасная мысль! Сейчас-же хватайте свою линеечку и направляйтесь в офис к разработчикам Битрикса, чтобы нещадно высечь всех причастных к созданию данной говноCMSки. Тысячи PHPшников будут чтить Вас в веках за сей подвиг!