Этот код считается трудночитабельным?

Question

[Илья Бондаренко]

Мне нужно было сформировать массив определенным образом. Чтобы была структура:

[
[0] => [
[original] => "<img width="448" src="..." height="366" title="Вопрос?" alt="Вопрос?">",
[attributes] => [
[width] => "448"
[src] => "/upload/medialibrary/412/comics_151341_640.png"
[height] => "366"
[title] => "Вопрос?"
[alt] => "Вопрос?"
], 
[1] => ...
]

Получившаяся структура наглядно:


Всё сделал, но понимаю что код как-будто не читабельный:

preg_match_all('/(?<=\<img).*?(?=>)/', $this->html, $match, PREG_PATTERN_ORDER);

    $this->tags = array_map(
      fn($strTag) => [
        "original" => "<img " . $strTag . ">",
        "attributes" => array_map(
          fn($attribute) => array_pop(explode("=", $attribute)),
          array_combine(
              array_map(
                fn($attribute) => array_shift(explode("=", $attribute)),
                explode(" ", trim($strTag))
              ),
              explode(" ", trim($strTag))
          )
        )
      ],
      $match[0]
    );

Подскажите, как сделать его более читабельным? Возможно перемудрил. Я рад был бы что-то почитать на эту тему, буду рад если подскажите.

Comments

[Дмитрий]

Не думаю что на циклах получиться красивее и читабельнее. Вполне норм функциональщина

[Adamos]

Увеличить читаемость этого кода трудновато, да в общем, и незачем - тому, кому придется его читать, в любом случае придется читать его досконально, с листа тут врубиться шансов нет.
Но есть-таки альтернативный метод улучшения читаемости - это не маяться велосипедами и использовать готовые DOM-парсеры :)

Answer 1

[Slava Rozhnev]

Для повышения читабельности разбеыте код на отдельные функции. Напромер так:

<?php
function getTags($strTag) {
    return array_combine(
        array_map(
            fn($attribute) => array_shift(explode("=", $attribute)),
            explode(" ", trim($strTag))
        ),
        explode(" ", trim($strTag))
    );
}

function getAttributes($strTag) {
    return array_map(
        fn($attribute) => array_pop(explode("=", $attribute)),
        getTags($strTag)
    );
}

preg_match_all('/(?<=\<img).*?(?=>)/', $this->html, $match, PREG_PATTERN_ORDER);

$this->tags = array_map(
    fn($strTag) => [
        "original" => "<img " . $strTag . ">",
        "attributes" => getAttributes($strTag)
    ],
    $match[0]
);

https://phpize.online/sql/mysql57/undefined/php/ph...

Comments

[Илья Бондаренко]

Круто, спасибо!