Как усовершенствовать функцию?

Question

[Al Sm]

Здравствуйте. Есть функция, которая принимает запрос, проверяет наличие новых тегов, если таковые присутствуют добавляет в бд, получаем id и добавляем все в общий массив с индексами и в завершении синхронизируем.
Как бы вы сделали данную функцию?

public static function getTags($requestTags): array
{
    $tags = [];

    if ($requestTags) {
        foreach ($requestTags as $tag) {
            if (is_numeric($tag)) {
                $tags[] = $tag;
            } else {
                $newTag = Tag::upsert(['name' => $tag]);
                $tags[] = $newTag->id;
            }
        }
    }

    return $tags;
}

Можно ли, к примеру обойтись без форыча и воспользоваться стандартными функциями Php/Laravel?

Comments

[Александр null]

Как минимум изменить название, тут явно не getTags, а хотя бы updateTags

и вопрос что вы хотите получить (для чего ее совершенствовать?)

[Al Sm]

Александр null, может можно обойтись без форыча и воспользоваться стандартными функциями php или laravel?

[Александр null]

Александр Смирнов, например

public static function getTags($requestTags): array
{
$tags = [];

if ($requestTags && is_array($requestTags)) {
   $tags = array_map(function($elem){
                     if(!is_numeric($elem)){
                          $newTag = Tag::upsert(['name' => $elem]);
                          return $newTag->id;
                     } 
                     return $elem;
                 }, $requestTags);
}

return $tags;
}

[Al Sm]

Александр null, спасибо!

Answer 1

[Кирилл Несмеянов]

return \collect((array)$requestTags)->map(function($tag): int {
    return \is_numeric($tag) ? (int)$tag : Tag::upsert(['name' => $tag])->id;
});

Comments

[Al Sm]

Спасибо Кирилл! :)

Answer 2

[Андрей Николаев]

А что Вы в ней хотите усовершенствовать?
Ну например можно добавить typehint array у $requestTags и избавиться от проверки:

public static function getTags(array $requestTags): array
{
    $tags = [];

    foreach ($requestTags as $tag) {
        if (is_numeric($tag)) {
            $tags[] = $tag;
        } else {
            $newTag = Tag::upsert(['name' => $tag]);
            $tags[] = $newTag->id;
        }
    }

    return $tags;
}

Либо можно вынести в отдельную функицю сохранение тегов, например:

function mapTags( array &$value )
{
    if ( is_numeric($value) )
    {
        return $value;
    }

    $newTag = Tag::upsert(['name' => $tag]);
    return $newTag->id;
}

И тогда весь код метода свернется в:

public static function getTags( array $requestTags ): array
{
    return array_map('mapTags', $requestTags);
}

Можно просто уменьшить вложенность оставив скрипт (меньше уровней будет):

public static function getTags($requestTags): array
{
    $tags = [];

    if ( !$requestTags )
    {
        return $tags;
    }

    foreach ($requestTags as $tag) {
        if (is_numeric($tag)) {
            $tags[] = $tag;
        } else {
            $newTag = Tag::upsert(['name' => $tag]);
            $tags[] = $newTag->id;
        }
    }

    return $tags;
}

Если это очень частовыполняемая операция, то можно сэкономить на запросах:

public static function getTags(array $requestTags): array
{
    if ( empty($requestTags) )
    {
        return $requestTags;
    }

    $existedTags = array_filter($requestTags, 'is_numeric');
    $newTags = array_diff($requestTags, $existedTags);

    /**
     * Тут единый batch-метод который отправляет 
     * 1 запрос на все теги сразу.
     * В results - массив Tag, обработанных batch-методом
     */
    foreach ($results as $tag)
    {
        $existedTags[] = $tag->id;
    }

    return array_unique($existedTags);
}

Comments

[Al Sm]

Спасибо! То что нужно!