Можно ли исправить эту очень полезную функцию PHP?

Question

[XZdev]

День добрый!
Имеется функция, которая должна рекурсивно получать значения и изменять массив в удобном для меня формате - через точки вместо кучи квадратных скобок.
Существует массив $CONF в котором заданы параметры. Эти параметры могут изменяться на лету, в процессе работы приложения. Это выглядит удачно, но это не работает:(
Возможно в старых версиях PHP такой синтаксис и пройдёт, но на PHP 7.2 выдаёт ошибку:
[ParseError] syntax error, unexpected '&' (0)

function aGetByKey(&$array, $key, $value = null) {
    list($index, $key) = explode('.', $key, 2);
    if (!isset($array[$index])) throw new Exception("No such key: " . $index);

    if(strlen($key) > 0)
        return aGetByKey(&$array[$index], $key, $value);

    $old = $array[$index];
    if ($value !== null) $array[$index] = $value;
    return $old;    
}

function conf($key, $value = null) {
    global $CONF;
    return aGetByKey(&$CONF, $key, $value);
}

Хотелось-бы чтобы она работала так:
echo conf('category.name'); //Hello
echo conf('category.name', 'World'); //true
echo conf('category.name'); // World

echo conf('category.data', ['amount' => 158, 'kind' => 36]); //true
echo conf('category.data')['amount']; //158

У меня не хватает опыта, чтобы портировать её на новый синтаксис, ибо не имею представления о том как оно должно работать эффективно, кроме как перебор через foreach и explode.
Прошу вашей помощи,ваш сниппет будет полезен многим заглянувшим.

Comments

[Максим Грищенко]

получать значения и изменять массив в удобном для меня формате - через точки вместо

Прошу вашей помощи,ваш сниппет будет полезен многим заглянувшим.

Может взять готовое решение из фреймворка Laravel?

Arr::get()



data_get()

Answer 1

[Ninazu]

Если уже переписываете, то переписывайте нормально. Избавляйтесь от глобального массива $CONF, рекурсии и функций без неймспейсов. Также у вас функция как записывает так и читает значения, что немного выносит мозг. Также считаю что исключению там не место, лучше заменить на значение по умолчанию

class Config {

	protected static array $register = [];

	public static function get(string $name, $default = null) {
		$keys = explode('.', $name);

		$pointer = self::$register;

		foreach ($keys as $key) {
			if (!array_key_exists($key, $pointer)) {
				/*Если ключа нет, выставить его*/
				//self::set($name, $default);

				/*Бросить исключение*/
				//throw new Exception("No such key: {$name}");

				/*Вернуть значение по умолчанию*/
				return $default;
			}

			$pointer = &$pointer[$key];
		}

		return $pointer;
	}

	public static function set(string $name, $value = null) {
		$arr = &self::$register;
		$keys = explode('.', $name);

		foreach ($keys as $key) {
			$arr = &$arr[$key];
		}

		if (is_array($arr) && is_array($value)) {
			$arr = array_replace_recursive($arr, $value);
		} else {
			$arr = $value;
		}
	}

	public static function getTree() {
		return self::$register;
	}
}

Config::set("config", [
	"name" => "test",
]);
print_r(Config::get("config"));
echo "(config)\n\n";

Config::set("config.class.name", "A");
print_r(Config::get("config.class.name"));
echo "\n(config.class.name)\n\n";

Config::set("config", [
	"class" => [
		"test" => "B",
	],
]);
print_r(Config::get("config.class"));
echo "\n(config.class)\n\n";

Config::set("config.class", null);
print_r(Config::get("config.class"));
echo "\n(config.class)\n\n";

print_r(Config::get("config.fake.key.without.value", "UNDEFINED"));
echo "\n(config.fake.key.without.value)\n\n";

Config::set("config.nullable", null);
print_r(Config::get("config.nullable", "NULL"));
echo "\n(config.nullable)\n\n";

print_r(Config::getTree());
echo "\n(all)\n";

Работает в PHP7.4

Comments

[kafkiansky]

И в чем прикол? Заменили ф-ции на класс ф-ций (ООПшно!).

[Ninazu]

kafkiansky, Тут класс используется не как обьект, а как неймспейс для группировки функций, плюс позволяет перенести из глобала переменную в своё тело. Плюс убрана рекурсия.

P.S. Если для вас нет разницы между этими реализациями, то мне очень жаль....

[kafkiansky]

Ninazu, мне тоже жаль, что для вас класс со статическим ф-циями чем-то отличается от просто ф-ций.

[Ninazu]

kafkiansky,
1. Для подключения классов один раз пишется autoloader. Чтоб подключить чистые функции придется вставлять include в каждый файл
2. Классы служат для группировки функций. В чистых функциях разве что использовать префикс в имени для разграничивания по функционалу.
3. В данном контексте с чистыми функциями приходится использовать глобальную переменную, которую нужно пробрасывать по всему проекту. Когда-то это может послужить коллизией, и кто-то перетрёт вашу глобальную переменную своей.

Теперь пожалуйста ваши аргументы в пользу чистых функций

[kafkiansky]

Ninazu,

Чтоб подключить чистые функции придется вставлять include в каждый файл

Вжух и никакого инклюда:

"autoload": {
        "files": ["src/functions.php"]
    },

Классы служат для группировки функций. В чистых функциях разве что использовать префикс в имени для разграничивания по функционалу

Ну группируйте так же в файле, пср не запрещает.

В данном контексте с чистыми функциями приходится использовать глобальную переменную, которую нужно пробрасывать по всему проекту. Когда-то это может послужить коллизией, и кто-то перетрёт вашу глобальную переменную своей.

return \call(\array_filter(\array_map(fn($value) => $value['id'], $data)))

Что ряд ф-ций, то ряд статических метод, одно и то же.

[kafkiansky]

Ninazu, нет, я за объектно-ориентированный подход, но не за классо-ориентированный, вот и вся разница. Единственное преимущество подхода, который вы показали, это то, что результат можно складывать в свойство, но вызывать каждый раз `Config::set` вам все равно придется, как и, внезапно, с простой ф-цией.

Answer 2

[Андрей]

return aGetByKey($CONF, $key, $value);

Answer 3

[AUser0]

Исходный вариант кода, но исправленный:

$CONF = array('category' => array('name' => "Hello", 'data' => array()));

function aGetByKey(&$array, $index, $value = null) {
    if (strpos($index, "."))
        list($index, $key) = explode('.', $index, 2);
    if (!isset($index) OR !isset($array[$index]))
        throw new Exception("No such key: " . $index);

    if(!empty($key))
        return aGetByKey($array[$index], $key, $value);

    $old = $array[$index];
    if ($value !== null)
        $array[$index] = $value;

    return $old;
}

function conf($key, $value = null) {
    global $CONF;
    return aGetByKey($CONF, $key, $value);
}

//Хотелось-бы чтобы она работала так:
echo conf('category.name'); //Hello
echo conf('category.name', 'World'); //Hello, потому что возвращается предыдущее значение $old
echo conf('category.name'); // World

conf('category.data', ['amount' => 158, 'kind' => 36]); // echo() нельзя, потому что вернётся массив
echo conf('category.data')['amount']; //158

Comments

[AUser0]

Ещё момент, для удачного присвоения conf('category.data', array(...)); в массиве $CONF уже должно существовать значение 'data': $CONF['category']['data'] = array();. Иначе будет ошибка, и никакого присвоения...