(Untitled)

[hayk]

$at_name = mb_strtoupper($anti_theft_brand.' '.$anti_theft_name, 'UTF-8');
$at_name = $at_name == ' ' ? '' : $at_name;

Comments

[voenniy]

trim?
Или вас что-то другое смутило?

[hayk]

Ну меня смутило не использование trim.

[sontar]

для двух аргументов и пробела можно использовать trim
для трёх и более аргументов или непробельного разделителя нужно использовать join($delimiter, array_filter(array($arg1, $arg2, ... $argN)))
а так да, подобных примеров полон говнокод

[ximaera]

На PHP так заведено, да.

[display_none]

В общем -- да. Если бы существовала книга "thinking in PHP', то именно с такой строчки она бы и начиналась.

Правда, я бы еще улучшил:
$anti_theft_brand.' '.$anti_theft_name

mb_strtoupper($anti_theft_brand.' '.$anti_theft_name.'', 'UTF-8');

[display_none]

Стесняюсь спросить: а если anti_theft_brand будет " "?

[anmiles]

Не говнокод.
Нельзя здесь trim использовать.
Переменные $anti_theft_brand или $anti_theft_name могут начинаться и заканчиваться пробелами, и тогда использование trim испортит значение $at_name.

Мания всё на свете упрощать иногда приводит к таким ошибкам, что вовек не разберёшься.

[hayk]

Переменные $anti_theft_brand или $anti_theft_name не могут содержать пробел в конце или начале, так же как Имя и Фамилия не могут их содержать.

[anmiles]

Из названия переменной $anti_theft_brand это никак не следует.
Вы бы сразу уточняли такое, раз знакомы с проектом.

[hayk]

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