Как распарсить сложный заголовок?

Question

[di]

В заголовке запроса приходит довольно сложное содержание:
X-CALLBACK-ID timestamp=1563105451261; nonce=a07bfa17;value=E4YeOsnMtHZ6592U8B9S37238E+Hwtjfrmpf8AQXF+c=

Я хочу распарсить это в массив

$aaa = [
            'timestamp' =>1563105451261,
            'nonce'     =>'a07bfa17',
            'value'     =>'E4YeOsnMtHZ6592U8B9S37238E+Hwtjfrmpf8AQXF+c=',
        ];

Впринципе написал рабочее решение, но оно как то стремно выглядит

public function handle($request, Closure $next)
    {
        $xHeader = $request->header("X-CALLBACK-ID");
        if (!$xHeader) {
            abort(403);
        }

        $kvPairs = explode(";", $xHeader);

        if (count($kvPairs) != 3) {
            abort(403);
        }

        $parsedHeader = [];

        foreach ($kvPairs as $pair) {
            $pos = strpos($pair, "=");
            if (!$pos) {
                abort(403);
            }

            $parsedHeader[substr($pair, 0, $pos)] = substr($pair, $pos + 1);
        }

        if (!isset($parsedHeader['timestamp'], $parsedHeader['nonce'], $parsedHeader['value'])) {
            abort(403);
        }

Возможно есть какое то более производительное и читабельное решение? Подскажите плиз если знаете как реализовать это лучше?

Comments

[FanatPHP]

Традиционно для таких вещей используют parse_str с предварительным вызовом str_replace или регулярку

Хороший повод освоить последние кстати

[Евгений]

FanatPHP, там же urldecode отработает и будет не совсем то что надо

<?php
$text = "timestamp=1563105451261; nonce=a07bfa17;value=E4YeOsnMtHZ6592U8B9S37238E+Hwtjfrmpf8AQXF+c=";
parse_str(preg_replace('~\s*;\s*~', '&', $text), $data);
print_r($data);

Array
(
    [timestamp] => 1563105451261
    [nonce] => a07bfa17
    [value] => E4YeOsnMtHZ6592U8B9S37238E Hwtjfrmpf8AQXF c=
)

плюсик в последней переменной потерян

Answer 1

[nokimaro]

<?php

$xHeader = 'timestamp=1563105451261; nonce=a07bfa17;value=E4YeOsnMtHZ6592U8B9S37238E+Hwtjfrmpf8AQXF+c=';

preg_match_all('!([^=; ]+)=([^;]+)!si', $xHeader, $out);

$parsedHeader = array_combine($out[1], $out[2]);

print_r($parsedHeader);

результат

Array
(
    [timestamp] => 1563105451261
    [nonce] => a07bfa17
    [value] => E4YeOsnMtHZ6592U8B9S37238E+Hwtjfrmpf8AQXF+c=
)

Answer 2

[Антон Иванов]

Да нормально выглядит. Можно foreach заменить на array_map. Хорошим тоном считается, чтобы функция возвращала значение. Также делать дважды abort(403) тоже не красиво. Но код рабочий, это главное.

Ещё не вижу в коде использование входящего параметра next. Он лишний? Ну и тогда тег вопроса laravel видимо лишний.

Comments

[di]

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