Меню
Что нового?

Проверка входящих значений функции

  • Автор темы Автор темы nigirma
  • Дата начала Дата начала

nigirma

Новичок
Есть функция, в самом ее начале производится проверка входящего(их) значения.
PHP: 
/**
 * Демонстрационная функция проверки входящих значений
 * 
 * @param  int    $size Число от 0 до 3
 */
function demoCheckParam($num = 0) {
    // ПРОВРКА $num
    
    // код функции не интересующий нас сейчас
}

Есть несколько вариантов проверок:
1)
PHP: 
if (($num < 0) || ($num > 3)) {
    $num = 0;
}

2)
PHP: 
if (($num !== 0) && ($num !== 1) && ($num !== 2) && ($num !== 3)) {
    $num = 0;
}

3)
PHP: 
if (!in_array($num, $access)) {
    $num = 0;
}

4)
PHP: 
if (!preg_match('/^([0-3])$/', $num)) {
    $num = 0;
}

Для каждого есть свои минусы:
1) Неправильно обрабатывается если введена строковая переменная. Например если $num = 'gkhuimn' то проверка будет пройдена, т.к. при сравнении $num с числом, $num будет равна нулю...
2) Ужасная нечитабельная запись. Также теряется возможность задавать в качестве параметра параметры так: '2'. А иногда бывают случаи такого объявления, например через форму на сайте.
3) В несколько раз медленней чем первый вариант и имеет такую же ошибку как первый, т.ч. отметается 100%
4) Самый работающий без ошибок вариант, но медленней чем третий в 2 раза (примерно в 8-10 раз медленней чем первый вариант).

Посоветуйте какой вариант проверки значения выбрать или предложите свой.
Не маловажно быстродействие кода, не смотря даже на то что различия совсем небольшие
 
nigirma написал(а):
1) Неправильно обрабатывается если введена строковая переменная. Например если $num = 'gkhuimn' то проверка будет пройдена, т.к. при сравнении $num с числом, $num будет равна нулю...
Нажмите для раскрытия...
Либо забей, либо жди type hinting'а для скаляров, либо смени PHP на другой язык.

Есть ещё вот это, кстати: http://ru2.php.net/manual/ru/book.spl-types.php
 
PHP: 
$access = array(0,1,2,3);
$num = $access[array_search($num,$access)];

А 4 вариант пропустит 3333333. Зачем там плюс-то?
 
Фанат
Фанат написал(а):
А 4 вариант пропустит 3333333. Зачем там плюс-то?
Нажмите для раскрытия...
да плюсик по ошибке поставил. Убрал сейчас

Фанат написал(а):
Афигеть проблема, сервер ляжет от двух посетителей.
Не стыдно такую фигню писать?;
Нажмите для раскрытия...
Не стыдно. Микрооптимизация это.
Десять тыщ. человек (посещаемость) * 10 (кол-во открытых страниц) * 20 (в среднем функций на страницу) * 3 (проверяемых аргумента в среднем в функции) = небольшая но экономия =)
 
nigirma написал(а):
Не стыдно. Микрооптимизация это.
Десять тыщ. человек (посещаемость) * 10 (кол-во открытых страниц) * 20 (в среднем функций на страницу) * 3 (проверяемых аргумента в среднем в функции) = небольшая но экономия =)
Нажмите для раскрытия...
Вот откуда они берутся, такие наивные-то?
Добавочная стоимость железки которая покроет эти тысячи этих «микрооптимизаций» меньше зарплаты программиста за день.
Элеметарный кеш полученных значений покроет 99% «микрооптимизаций» махом и вообще ничего не будет стоить.

ДЕЛОМ ЗАНИМАЙСЯ, НЕ ФИГНЕЙ.
 
флоппик
ты не понимаешь. никакое железо здесь не нужно.
и никакой кэш. здесь вообще никакой экономии нет.
разговаривая с ним про железки, ты соглашаешься с тем, что разница какая-то есть.
а ее нет.
 
Да я не о экономии, а о том, что надо программировать, а не фигней страдать.
 
Ну, а что до разницы — в трех примерах разный код, который, на самом деле, делает разные вещи )
 
Оптимизация — не плохо. Только оптимизации подлежит не код, а архитектура. А самое главное, что бы оптимизировать, надо видеть, в чем проблема, которую надо решить, и чем надо пожертвовать для этого. А у тебя нет реальной проблемы, поэтому чем бы ты не пожертвовал — пользы не будет, а будет только вред.
 
Войдите или зарегистрируйтесь для ответа.
Сверху