- 1
- 2
- 3
- 4
- 5
- 6
- 7
//Функция проверки закачиваемого файла
function fileNameCheck($file) {
if(preg_match("/[^a-z\d\-\._]/i", $file)) {
return true;
}
return false;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+48
//Функция проверки закачиваемого файла
function fileNameCheck($file) {
if(preg_match("/[^a-z\d\-\._]/i", $file)) {
return true;
}
return false;
}
Приятно удивляет соответствие названия функции с её возвращаемым значением.
vistefan 22.08.2012 19:08 # 0
Edward 22.08.2012 20:01 # +2
vistefan 22.08.2012 20:14 # 0
Вообще, в данном случае это особенность архитектуры, или предположение автора, какое значение будет использоваться чаще, что бы реже использовать логическое отрицание.
bormand 22.08.2012 20:29 # +9
Steve_Brown 23.08.2012 12:23 # +1
bormand 23.08.2012 15:26 # +1
checkSomethingForCorrectness
checkSomethingForErrors
Поэтому я и не люблю методы, имя которых начинается с check...
Steve_Brown 23.08.2012 15:39 # 0
Edward 23.08.2012 11:21 # −3
С постом про isValidName/isInvalidName полностью согласен
psycho-coder 22.08.2012 19:14 # +2
Lure Of Chaos 22.08.2012 22:43 # +7
psycho-coder 23.08.2012 08:39 # +1
bormand 23.08.2012 16:11 # 0
Vasiliy 23.08.2012 16:24 # −1
nur 23.08.2012 13:17 # +4
psycho-coder 23.08.2012 19:16 # +2