Как заменить по шаблону значения, которые находятся в другом массиве?

Question

[Aleksandr Yurchenko]

Здравствуйте.
Помогите решить задачку. Дано два массива:
1) массив с данными о работнике (значения которого нужно подставить)

$user = [
    'id' => 1,
    'name' => 'Ivan Ivanov',
    'role' => 'developer',
    'salary' => 100
];

2) Массив с шаблоном для замены (куда нужно подставить)

$apiTemplatesSet1 = [
    '/api/items/%id%/%name%',
    '/api/items/%id%/%role%',
    '/api/items/%id%/%salary%'
];

Регулярку сделал такую: ~%id%/%(.+)%~. Но что-то мне она не особо нравится, так как при попытке найти по ней значения

Получается

$result = preg_match('~%id%/%(.+)%~',$apiTmpSet,$matches);

array (size=2)
  0 => string '%id%/%name%' (length=11)
  1 => string 'name' (length=4)
array (size=2)
  0 => string '%id%/%role%' (length=11)
  1 => string 'role' (length=4)
array (size=2)
  0 => string '%id%/%salary%' (length=13)
  1 => string 'salary' (length=6)

Пробовал ещё такой вариант: '~%(.+)%/%(.+)%~', массив получается из трех элементов и первым идет значение с %...%/%...%. Можно ли получить только значения между знаков вопроса?

И насчет самой замены, подскажите как правильнее сделать? Сначала хотел перебрать foreach массив $apiTemplatesSet1, получить строки и заменить по регулярке, но встал вопрос, как подставить значения из другого массива ($user)? Да и решение, как мне кажеться через чур громоздкое.

Есть ли готовые функции в php, для данной задачи? И если нет, то как лучше написать собственную?

Comments

[Дмитрий]

Ну если ни кто не ответит, напишу велосипед

[twobomb]

$user = [
    'id' => 1,
    'name' => 'Ivan Ivanov',
    'role' => 'developer',
    'salary' => 100
];

$ats = [
    '/api/items/%id%/%name%',
    '/api/items/%id%/%role%',
    '/api/items/%id%/%salary%'
];

$res = [];
foreach($ats as $tem){
    foreach($user as $key=>$v)
        $tem = str_replace("%$key%",$v,$tem);
    array_push($res,$tem);
}

$res

array(3) {
  [0]=>
  string(24) "/api/items/1/Ivan Ivanov"
  [1]=>
  string(22) "/api/items/1/developer"
  [2]=>
  string(16) "/api/items/1/100"
}

[Aleksandr Yurchenko]

Дмитрий, ага спасибо :)
Я тут сам велосепедирую понемногу, если совсем накостылить:

$result = implode('*', $apiTemplatesSet1);
$result = preg_replace('~%id%~', $user['id'], $result);
$result = preg_replace('~%name%~', $user['name'], $result);
$result = preg_replace('~%role%~', $user['role'], $result);
$result = preg_replace('~%salary%~', $user['salary'], $result);
$result = explode('*', $result);

Вроде работает, а вроде бы хрен полная. Просто у меня в голове вертится array_map(), но ХЗ как её реализовать.

P.S. вариант twobomb уже ближе к тому что хочу :)

[Aleksandr Yurchenko]

twobomb, спасибо!

[Дмитрий]

$user = [
    'id' => 1,
    'name' => 'Ivan Ivanov',
    'role' => 'developer',
    'salary' => 100
];

$apiTemplatesSet1 = [
    '/api/items/%id%/%name%',
    '/api/items/%id%/%role%',
    '/api/items/%id%/%salary%'
];

class Invoker
{
    private $data;

    public function __construct(array $data)
    {
        $this->data = $data;
    }

    public function __invoke(array $key): string
    {
        return $this->data[$key[1]];
    }
}

$function = new Invoker($user);

$result = array_map(function ($key) use ($function)
{
    return preg_replace_callback('#%(.*)%#isU', $function, $key);
}
, array_values($apiTemplatesSet1));

Не проверял, пилил с телефона. Возможно надо до работать напильником =)
Вроде отладил

[Дмитрий]

twobomb, а чего не в ответ?

[Дмитрий]

Aleksandr Yurchenko, мой вариант то рабочий?, я просто не проверял

[Aleksandr Yurchenko]

Дмитрий, кстати могу адресовать вам тоже :) Сейчас в процедурном стиле закончил, и ваш вариант хочу сделать))

[twobomb]

Дмитрий, тот же вопрос

[Дмитрий]

Aleksandr Yurchenko, изменил, отладил с телефона
https://3v4l.org/DbIAi

[Aleksandr Yurchenko]

Дмитрий,
UPD: под ваш ответ засунул :)

Answer 1

[Dmitry Tallmange]

<?php
$user = [
    'id' => 1,
    'name' => 'Ivan Ivanov',
    'role' => 'developer',
    'salary' => 100
];

$apiTemplatesSet1 = [
    '/api/items/%id%/%name%',
    '/api/items/%id%/%role%',
    '/api/items/%id%/%salary%'
];

print_r(str_replace(array_keys($user), array_values($user), str_replace("%", "", $apiTemplatesSet1)));

Array
(
    [0] => /api/items/1/Ivan Ivanov
    [1] => /api/items/1/developer
    [2] => /api/items/1/100
)

Comments

[Aleksandr Yurchenko]

то что надо! Спасибо.

[Lander]

И не дай Бог вам заменить значение шаблона на что то типа
'/api/item/parole/%id%/%name%',

[Дмитрий]

Могут быть коллизии, что будет если попадётся строка где будет например такое?
Fname/Dname/%name%

[Aleksandr Yurchenko]

Дмитрий, Lander
Хоть это и тестовая задача, но все же хочу спросить – часто ли подобные случаи встречаются? Как мне кажется (утверждать не буду, опыта маловато), когда реализовывается API это закладывается (имею ввиду шаблоны) сначала и потом редко меняются и если такое происходит, то это проверяется и вносятся правки.

А если пришедшая строка не подходит не под один из указанных шаблонов, то выкидывается ошибка. Или я чего-то не понимаю?

[Дмитрий]

Aleksandr Yurchenko, просто решение было написано в лоб.. Если для собеседования писать, то могут сказать что вы запилили быдлокод. Надо учитывать разные варианты, тут товарищ просто выпилил плэйсхолдер символы и заменил ключи значениями, но я написал вариант когда это не проканает

[Lander]

Aleksandr Yurchenko, Написал свой ответ.

Answer 2

[Lander]

Поделюсь куском кода, из класса, который сам использую в одном из проектов (это не готовый код для использования, а просто пара методов, вырезанная из контекста, но это может дать представление о том как решаются такие задачи)

/**
     * Производит заливку шаблона значениями из массива
     *
     * Литералы шаблона вида {<liter_1>.<liter_2>...} будут заменены на соотвествующие значения из массива
     * $data[<liter_1>][<liter_2>]...
     *
     * @param $template Шаблон, содержащий литералы
     * @param $data Массив значений для подстановки
     * @return string Преобразованная строка
     */
    public function fill_template($template, $data)
    {
        preg_match_all('/\{([A-Za-z0-9_\.]+)\}/u', $template, $match);
        list($snippets, $fillers) = $match;
        
        if (!empty($snippets)) {
            
            $fillers = array_map(function($filler) use ($data) {
                $keys = explode('.', $filler);
                return $this->array_find($keys, $data);
            }, $fillers);
            
            return strtr($template, array_combine($snippets, $fillers));
            
        }
        
        return $template;
    }
    
    /**
     * Рекурсивно ищет значение в массиве в порядке обозначенным $keys
     *
     * @param array $keys Массив ключей в порядке поиска их в массиве
     * @param array $data Массив значений для поиска
     * @param string $default Значение по умолчанию, если нет соответствующего значения в массиве
     * @return mixed
     */
    public function array_find($keys, $data, $default = '')
    {
        if (!is_array($data) || empty($keys)) return $data;
        
        $key = current($keys);
        return (isset($data[$key]))
            ? $this->array_find(array_slice($keys, 1), $data[$key], $default)
            : $default;
    }

$user = [
    'id' => 1,
    'name' => 'Ivan Ivanov',
    'role' => 'developer',
    'salary' => 100
];

fill_template('/api/items/{id}/{name}', $user); // /api/items/1/Ivan Ivanov

upd: соответственно под свой шаблон нужно поменять строку

preg_match_all('/\{([A-Za-z0-9_\.]+)\}/u', $template, $match);

upd2: Занудам: Да, я знаю про array_shift. :)

Comments

[Aleksandr Yurchenko]

Спасибо!

[Дмитрий]

Я там свой велосипед отладил в коментариях

[Lander]

Дмитрий, Не видел. :( Вынесите в ответы, тоже пометим.

Answer 3

[Дмитрий]

$user = [
    'id' => 1,
    'name' => 'Ivan Ivanov',
    'role' => 'developer',
    'salary' => 100
];

$apiTemplatesSet1 = [
    '/api/items/%id%/%name%',
    '/api/items/%id%/%role%',
    '/api/items/%id%/%salary%'
];

class Invoker
{
    private $data;

    public function __construct(array $data)
    {
        $this->data = $data;
    }

    public function __invoke(array $key): string
    {
        return $this->data[$key[1]];
    }
}

$function = new Invoker($user);

$result = array_map(function ($key) use ($function)
{
    return preg_replace_callback('#%(.*)%#isU', $function, $key);
}
, array_values($apiTemplatesSet1));

echo '<pre>' . print_r($result, true);

https://3v4l.org/DbIAi

Comments

[Aleksandr Yurchenko]

Дмитрий, ага спасибо. А такой вопрос, можно ли как-то обойтись без магических метода? А сделать например:
• одно свойство - куда помещаем значения (private $data)
• приватную функцию, которая будет выступать в качестве калбека для preg_replace_callback()
• публичную функцию, которая будет выдавать сформированный после замены массив

Сходу не получилось, не подскажите в чем трабла? Вот примерная структура, может я на этом этапе уже косякнул

spoiler

<?php declare(strict_types=1);

class testAPI
{
    private array $data;
    public function __construct(array $data)
    {
        $this->data = $data;
    }

    public function getApiPath(array $template): array
    {
        return array_map(function (array $template) {
            $regex = '#%(.*)%#isU';
            return preg_replace_callback($regex, $this->prepareTemplate(), $template);
        },
            array_values($template));
    }
    private function prepareTemplate()
    {
        // callback который будет производить замену
    }

}

$apiTemplatesSet1 = [
    '/api/items/%id%/%name%',
    '/api/items/%id%/%role%',
    '/api/items/%id%/%salary%'
];

$test = new testAPI([
    'id' => 20,
    'name' => 'John Dow',
    'role' => 'QA',
    'salary' => 100
]);

[Дмитрий]

Aleksandr Yurchenko, Aleksandr Yurchenko, в конструктор надо будет передавать и шаблон тогда, его тоже сохранить в свойстве
Ну и то что в invoke перенести в тот метод

[Aleksandr Yurchenko]

Дмитрий, блин не подскажешь, че-т понять не могу.

spoiler

<?php declare(strict_types=1);

class testAPI
{
    private array $employee;
    private array $template;

    /**
     * testAPI constructor.
     * @param array $employee – массив с данными о работнике
     * @param array $template — массив с шаблонами
     */
    public function __construct(array $employee, array $template)
    {
        $this->employee = $employee;
        $this->template = $template;
    }

    public function getApiPath(): array
    {
        return array_map(function () {
            $regex = '#%(.*)%#isU';
            return preg_replace_callback($regex, $this->prepareTemplate(), $this->template);
        },
            array_values($this->template));
    }

    private function prepareTemplate(): array
    {
        $result = [];
        foreach ($this->employee as $key => $value) {
            $tmp = str_replace("%$key%", $value, $this->template);
            $result[] = $tmp;
        }
        var_dump($result);
        exit;
    }

}

$apiTemplatesSet1 = [
    '/api/items/%id%/%name%',
    '/api/items/%id%/%role%',
    '/api/items/%id%/%salary%'
];

$user = [
    'id' => 20,
    'name' => 'John Dow',
    'role' => 'QA',
    'salary' => 100
];

$test = new testAPI($user, $apiTemplatesSet1);
var_dump($test->getApiPath());

Результат:
array (size=4)
  0 => 
    array (size=3)
      0 => string '/api/items/20/%name%' (length=20)
      1 => string '/api/items/20/%role%' (length=20)
      2 => string '/api/items/20/%salary%' (length=22)
  1 => 
    array (size=3)
      0 => string '/api/items/%id%/John Dow' (length=24)
      1 => string '/api/items/%id%/%role%' (length=22)
      2 => string '/api/items/%id%/%salary%' (length=24)
  2 => 
    array (size=3)
      0 => string '/api/items/%id%/%name%' (length=22)
      1 => string '/api/items/%id%/QA' (length=18)
      2 => string '/api/items/%id%/%salary%' (length=24)
  3 => 
    array (size=3)
      0 => string '/api/items/%id%/%name%' (length=22)
      1 => string '/api/items/%id%/%role%' (length=22)
      2 => string '/api/items/%id%/100' (length=19)

Что не так? Как же я не люблю array_map,array_filter,array_redux...постоянно у меня в этих функциях какой-то головняк вылезает =)

P.S. ваш вариант пробовал занусуть, только в примере в методе __inoke предается один параметр, а тут не совсем понимаю что передавать о_О

[Дмитрий]

Aleksandr Yurchenko, функция preg_replace_callback принимает callback, то есть то что указали в аргументе callback должно вернуть callback, то есть
return function...
Если не получится, до компа доберусь гляну

[Aleksandr Yurchenko]

Дмитрий, вроде бы получилось, как с точки зрения реализации выглядит (убрал приватный метод и по рекомендации из манула запилил анонимную функцию):

public function getApiPath(): array
    {
        return array_map(function () {
            return preg_replace_callback(
                '#%(.*)%#isU',
                function ($matches) {
                    return $this->employee[$matches[1]];
                },
                $this->template);
        },
            array_values($this->template));
    }

[Дмитрий]

Aleksandr Yurchenko, норм

[Aleksandr Yurchenko]

Дмитрий, и последнее момент, при вызове метода результат не совсем то, что требуется:

spoiler

array (size=3)
  0 => 
    array (size=3)
      0 => string '/api/items/20/John Dow' (length=22)
      1 => string '/api/items/20/QA' (length=16)
      2 => string '/api/items/20/100' (length=17)
  1 => 
    array (size=3)
      0 => string '/api/items/20/John Dow' (length=22)
      1 => string '/api/items/20/QA' (length=16)
      2 => string '/api/items/20/100' (length=17)
  2 => 
    array (size=3)
      0 => string '/api/items/20/John Dow' (length=22)
      1 => string '/api/items/20/QA' (length=16)
      2 => string '/api/items/20/100' (length=17)

Почему дублируется 3 раза?

[Дмитрий]

Aleksandr Yurchenko, сейчас гляну

[Aleksandr Yurchenko]

Дмитрий, спасибо! Выложил на тот ресурс, где вы скидывали пример: https://3v4l.org/rhfVu

Удобно однако :)

[Дмитрий]

Aleksandr Yurchenko, https://3v4l.org/2BWag

[Aleksandr Yurchenko]

Дмитрий, хм :\ array_map не нужен...

Получалось три элемента, так как каждый элемент массива из-за функции array_map считался сам массивом? :)

[Дмитрий]

Aleksandr Yurchenko, отладка тоже скилл)

[Aleksandr Yurchenko]

Дмитрий, полностью согласен. Надо было подебажить блин, там бы увидел скорей все.

В любом случае спасибо большое! :)