- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
public function isValidModule($module)
{
if (!is_string($module)) {
return false;
}
$module = strtolower($module);
$controllerDir = $this->getControllerDirectory();
foreach (array_keys($controllerDir) as $moduleName) {
if ($module == strtolower($moduleName)) {
return true;
}
}
return false;
}
sonic 21.04.2011 13:56 # +1
hacpaka 21.04.2011 14:00 # 0
rO_ot 21.04.2011 14:18 # +1
инфа от разработчиков
ling 21.04.2011 14:36 # +1
hacpaka 21.04.2011 15:08 # 0
увы, но скорее всего в ГК((((
Lure Of Chaos 21.04.2011 15:20 # 0
hacpaka 21.04.2011 15:21 # 0
Lure Of Chaos 21.04.2011 15:26 # −1
hacpaka 21.04.2011 15:32 # 0
Lure Of Chaos 21.04.2011 15:37 # 0
hacpaka 21.04.2011 15:40 # 0
Вам нравится зеленый цвет? Или поиск от google?
Lure Of Chaos 21.04.2011 15:42 # +1
hacpaka 21.04.2011 15:45 # 0
А с чего сударь изволит интересоваться моей персоной, даколи не секрет?
Lure Of Chaos 21.04.2011 15:49 # +1
Все проще. Я не понял, что такое "Hacu Pakasy", думал, гугл подскажет. Гугл подсказал только вот это
sonic 21.04.2011 15:51 # +2
Lure Of Chaos 21.04.2011 16:01 # 0
sonic 21.04.2011 16:04 # 0
Lure Of Chaos 21.04.2011 16:05 # 0
rO_ot 21.04.2011 17:22 # 0
Я на него хоть молиться готов
Lure Of Chaos 21.04.2011 17:23 # 0
hacpaka 21.04.2011 17:23 # 0
Lure Of Chaos 21.04.2011 23:03 # 0
hacpaka 21.04.2011 23:06 # 0
bugmenot 22.04.2011 09:40 # 0
Lure Of Chaos 22.04.2011 09:57 # +1
hacpaka 22.04.2011 13:41 # 0
Дама пик...
bugmenot 21.04.2011 15:59 # +1
насрака, хрен ли тут гуглевать
hacpaka 21.04.2011 16:01 # 0
Lure Of Chaos 21.04.2011 16:02 # 0
guest 21.04.2011 16:05 # 0
hacpaka 21.04.2011 18:44 # +1
Lure Of Chaos 21.04.2011 16:06 # 0
вы тоже это увидели!
hacpaka 21.04.2011 16:15 # +3
guest 21.04.2011 22:20 # 0
GhOsTMZ 21.04.2011 22:37 # 0
по мне так достаточно примерно так (только к glob еще и путь к папке передать:
return is_string($moduleName) && glob(strtolower($moduleName)) !== false;
на крайняк есть функция поиска в массиве, что проще чем его перебирать
guest 21.04.2011 23:29 # 0
guest 21.04.2011 22:39 # 0
тождественно равно
а в контексте задачи и вообще уместным будет:
guest 21.04.2011 23:03 # 0
hacpaka 21.04.2011 23:19 # 0
Следует отметить что в контексте задачи работоспособны оба варианта, но если ключи могут быть инициализированы значением null и такое значение является в рамках задачи валидным то isset отпадает.
guest 21.04.2011 23:25 # 0
hacpaka 21.04.2011 23:37 # 0
То есть если нагадили "специально" это не ГК? Я правильно вас понял?
guest 21.04.2011 23:53 # 0
И в любом случае мы с Вами обсуждаем лишь данный метод с конкретным набором входных данных (уж какие есть), а не фреймворк вцелом.
hacpaka 22.04.2011 00:05 # 0
Nikitin 22.04.2011 16:02 # +1
Без контекста - код нормальный и оптимизировать его не получится.
hacpaka 22.04.2011 16:07 # 0
hacpaka 21.04.2011 23:06 # 0
склероз.
пост мой.
И кстати логичнее было бы вместо false в начале и false в конце функции запихнуть все в условия. Получилось бы несколько более гармонично: одно true внутри вложенных блоков условий и false в конце функции если все плохо.
guest 21.04.2011 23:20 # 0
Да, можно применять функции для работы с массивами, чтобы получить все ключи, привести их к нижнему регистру и выполнить in_array, но первоначальный вариант оптимальнее: возможно нам не придётся все модули приводить к нижнему регистру, а вернем true после первой же итерации.
hacpaka 21.04.2011 23:28 # 0
guest 21.04.2011 23:57 # 0
Что же касается данного метода, который Вы привели, я надеюсь мы с Вами пришли к согласию: в условиях таких входных данных это достаточно оптимальный вариант и в принципе не говнокод :)
hacpaka 22.04.2011 00:10 # 0
Ну почему же. Я не пишу на этом "прекрасном фреймворке" как вы сказали. Однако иногда приходится вляпываться.
ли фича где-то используемая
Эта фича называется не иначе как говнокод.
Что же касается данного метода, который Вы привели, я надеюсь мы с Вами пришли к согласию: в условиях таких входных данных это достаточно оптимальный вариант и в принципе не говнокод :)
Именно говнокод. Так получается - говнокод в одном месте плодит тоны говнокода в другом. То что этот говнокод является частью "великого" Зенда - сути не меняет.
Если есть желание проверить оптимальность - можете заняться на досуге.