Как проще написать удаление пробелов, тегов, и преобразование в нижний регистр?

Question

[Panzers]

Всем привет!
Есть код:

$pin = strtolower($_POST['crankpin-1'].$_POST['crankpin-2'].$_POST['crankpin-3'].$_POST['crankpin-4']);

$arr = str_replace('/\s/','',str_split($pin));

echo '<pre>';
print_r($arr);
echo '</pre>';

Вопросов несколько:
1) Как мне кажется, тут что то не так (громоздко что ли..), оно конечно и работает, но может есть проще варианты?
2) В форме 4 инпута текстовых, каждый ограничен в 4 символа. Соответственно в обработчик приходит 16 символов в целом. Далее нужно удалить пробелы, перевести все в нижний регистр, и разбить в массив. Все вроде бы неплохо, но если удаляем пробелы, то в массиве на выходе уже 17 символов. Wtf? Потому что дальше по коду нужно будет выдергивать из xml файла данные, а массив уже выходит косячный для выборки :(
3) Чем отфильтровать лучше вообще данные из формы по посту? На предмет тегов и прочего лишнего, там только буквы и числа и все нужно.

Comments

[John Didact]

Странно, что это работает)

[FanatPHP]

А зачем это всё? Вырезать, фильтровать?

[Panzers]

FanatPHP, В целом идея следующая: Есть xml файл с данными. И есть форма. В форму нужно ввести буквы и числа. В один инпут вводятся чисто числа (1,2,3), во второй инпут буквы (abc), в обработчике это все складывается, и формирует запрос, на основании которого из xml выводятся данные соответствующие этому запросу.

[galliard]

Panzers, угадывание пользовательского ввода - это путь в никуда. Не нужно ничего очищать и преобразовывать. Нужно отдельно провалидировать каждое поле на корректность ввода и если оно не корректно - вернуть пользователю ошибку.

[ThunderCat]

Panzers,

один инпут вводятся чисто числа (1,2,3), во второй инпут буквы (abc),

хм, морской бой штоле?

[Panzers]

galliard, Хм, об этом не подумал. Спасибо за идею.

[Panzers]

ThunderCat, да нет, подбор детали. Пользователь вводит данные со своей детали, ему выдается номер детали для заказа из базы. Вот и все :)

[FanatPHP]

я пониимаю зачем ввод.
я спросил, зачем что-то вырезать, фильтровать?
зачем удалять теги? зачем удалять таким кривым способом пробелы? чем trim() не устраивает?

Answer 1

[ThunderCat]

во первых - загадочная строка с $arr = str_replace('/\s/','',str_split($pin)); будет работать весьма неоднозначно, хотя бы по тому что str_split() не поддерживает работу с мультибайтовыми кодировками. Во вторых она там конечно же нафиг не нужна, так как str_replace() работает со строками в третьем аргументе, но зато не работает с регулярными выражениями в первом, по этому эта строка у вас вообще никак не работает. В остальном все вполне в лучших традициях бредогенераторов.

Коротко: Если у вас есть поля определенных типов - проверяйте их отдельно на совпадение нужным типам, для этого есть фильтры и хелперы. Экономия на буквах была актуальна в 17 веке, когда бумага дорогая была.

Answer 2

[marmuff]

По регистрам - есть в пыхе уже реализованные ф-ии для преобразования строк в lower case и UPPER CASE.
Для удаления пробелов и тэгов - регулярки к Вашим услугам.

Comments

[Panzers]

Это понятно, но вопрос в эффективности, через что именно лучше сделать? По пробелам например есть несколько функций, различаются между собой немногим. Сюда же регулярка. Чем проще и лучше это реализовать?
Ну и вопрос про нагромождение в целом, как бы это все сократить, и правильнее сделать..