Хабр Курсы для всех
РЕКЛАМА
Практикум, Хекслет, SkyPro, авторские курсы — собрали всех и попросили скидки. Осталось выбрать!
$ext = explode(".", $file);
$ext = $ext[count($ext)-1];
if ($ext == "php") {
include_once $filePath;
}Почему нельзя было написать так?$path = pathinfo($file);
if ($path['extension'] == 'php' {
include_once $filePath;
}Еще кусок ниже:$min = 9999999999;
...
if (count($v) <= $min){Такой подход неверен, несмотря на практическую ненужность использования >9999999999 элементов в $v. Лучше написать как-то так:$min = false;
...
if ($min === false || count($v) <= $min){$path = pathinfo($file);
if ($path['extension'] == 'php') {if (strrchr($file,'.') === '.php') {function isInArray($array, $el){
if (!count($array)) return false;
foreach ($array as $val){
if ($val == $el) return true;
}
}
function getValPos($array, $el){
if (!count($array)) return false;
foreach ($array as $key => $val){
if ($val == $el) return $key;
}
}
— вместо !count($array) было бы логичнее использовать is_array($array), потому что, например, строка (не массив, но по-сути массив) или число вполне пройдут сквозь;
— оформление: странно, что используются два стиля: для короткого if'a фигурные не используются (тоже так пишу — удобно), а вот для foreach'a — нет.
<?php
function test(array $array)
{
var_dump($array);
}
test('test');
?>if(!is_int($bar)) throw new BadTypeArgumentException;if (($res = foo($bar)) !== FALSE) {
// Делаем что-то полезное с $res
} else die("Всё плохо");
Разработка модульного движка на PHP