- 1
- 2
- 3
- 4
$transaction_number = (int)$this->security->xss_clean(trim(htmlspecialchars(substr($transaction_number, 0, 50))));
if(!$transaction_number OR $transaction_number < 0)
$transaction_number = 0;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+165
$transaction_number = (int)$this->security->xss_clean(trim(htmlspecialchars(substr($transaction_number, 0, 50))));
if(!$transaction_number OR $transaction_number < 0)
$transaction_number = 0;
один из наших программистов пишет.
CodeIgniter.
Lure Of Chaos 07.09.2011 18:04 # +3
Я узнал CI!
index01d 07.09.2011 18:19 # +6
jokz 07.09.2011 18:19 # +3
уверен 100%, что это не долбоебизм, а копипаст
gegMOPO4 07.09.2011 18:51 # 0
index01d 07.09.2011 18:59 # 0
gegMOPO4 07.09.2011 19:07 # 0
index01d 07.09.2011 19:54 # +7
i-- 07.09.2011 19:30 # +6
И соответственно вариант !$transaction_number (NULL, FALSE) невозможен.
jokz 07.09.2011 21:47 # +1
faiwer 07.09.2011 20:45 # 0
$transaction_number = $transaction_number > 0 ? $transaction_number : 0;
как то так? я правильно понял задумку автора?
themars 07.09.2011 20:49 # 0
faiwer 07.09.2011 20:51 # 0
$tn = (int)$tn > 0 ? (int)$tn : 0;
Но в этом случае дважды используется приведение типов :)
cybervantyz 08.09.2011 10:59 # 0
Или не?
А вообще сабж пытается выудить из введённых данных то, что ему нужно, даже в том случае, если туда внапихали SQL-инъекций, html-кода и ещё неизвестно чего. Автор заботится о хакире пользователе, чо =)
i-- 08.09.2011 13:25 # 0
echo abs(-5); //5
Аффтар берет первую часть строки в 50 знаков (ОГО!) - это те данные которые ему нужны, потом антихакер, потом приведение.
Так что вы дважды неправы.
cybervantyz 08.09.2011 21:21 # 0
guest8 09.04.2019 11:28 # −999