Как трансформировать массив?

Question

[Ilya Loopz]

Всем привет! Подскажите, как реализовать логику для изменения массива `messages` в `messages2`.
Нужно сгруппировать элементы по значению `customer_id`, сохранив их порядок. В данном примере сначала нужно сгруппировать элементы с `id = 1` и `2`, затем `3` и `4`, и так далее.

Данные

"messages": [
	{
		"id": 1,
		"customer": "Dr. Kane Hill",
		"customer_id": 1,
		"created_at": "2024-11-19 11:18:57",
		"text": "Hello, how are you?"
	},
	{
		"id": 2,
		"customer": "Dr. Kane Hill",
		"customer_id": 1,
		"created_at": "2024-11-19 11:20:57",
		"text": "I am good, thanks! How about you?"
	},
	{
		"id": 3,
		"customer": "Prof. Samir McClure III",
		"customer_id": 2,
		"created_at": "2024-11-19 11:21:57",
		"text": "Hey, what time is it?"
	},
	{
		"id": 4,
		"customer": "Prof. Samir McClure III",
		"customer_id": 2,
		"created_at": "2024-11-19 11:22:57",
		"text": "It is 3 PM."
	},
	{
		"id": 5,
		"customer": "Shad Leffler",
		"customer_id": 3,
		"created_at": "2024-11-19 11:23:57",
		"text": "Did you finish the project?"
	},
	{
		"id": 6,
		"customer": "Shad Leffler",
		"customer_id": 3,
		"created_at": "2024-11-19 11:24:57",
		"text": "New mesh"
	},
	{
		"id": 7,
		"customer": "Prof. Samir McClure III",
		"customer_id": 2,
		"created_at": "2024-11-19 11:26:57",
		"text": "Not bed?"
	},
	{
		"id": 8,
		"customer": "Prof. Samir McClure III",
		"customer_id": 2,
		"created_at": "2024-11-19 11:27:57",
		"text": "Cool?"
	}
]

"messages2": [
	{
		"customer": "Dr. Kane Hill",
		"messages_id": 1,
		"customer_id": 1,
		"messages": [
			{
				"id": 1,
				"created_at": "2024-11-19 11:18:57",
				"text": "Hello, how are you?",
			},
			{
				"id": 2,
				"created_at": "2024-11-19 11:20:57",
				"text": "I am good, thanks! How about you?",
			}
		]
	},
	{
		"customer": "Prof. Samir McClure III",
		"messages_id": 2,
		"customer_id": 2,
		"messages": [
			{
				"id": 3,
				"created_at": "2024-11-19 11:21:57",
				"text": "Hey, what time is it?",
			},
			{
				"id": 4,
				"created_at": "2024-11-19 11:22:57",
				"text": "It is 3 PM."
			}
		]
	},
	{
		"customer": "Shad Leffler",
		"messages_id": 3,
		"customer_id": 3,
		"messages": [
			{
				"id": 4,
				"created_at": "2024-11-19 11:23:57",
				"text": "Did you finish the project?"
			},
			{
				"id": 5,
				"created_at": "2024-11-19 11:24:57",
				"text": "New mesh"
			}
		]
	},
	{
		"customer": "Prof. Samir McClure III",
		"messages_id": 4,
		"customer_id": 2,
		"messages": [
			{
				"id": 6,
				"created_at": "2024-11-19 11:26:57",
				"text": "Not bed?"
			},
			{
				"id": 7,
				"created_at": "2024-11-19 11:27:57",
				"text": "Cool?"
			},

		]
	},
]

У меня получается какой-то монстр.

Код

public function toArray(Request $request): array
    {
        $messages = $this->whenLoaded('messages')->sortBy('id');

        $messagesGroupedByCustomer = [];
        $messagesIdCounter = 1;
        $currentCustomerId = null;
        $currentCustomerMessages = [];

        foreach ($messages as $message) {
            if ($currentCustomerId !== $message['customer_id']) {
                if ($currentCustomerId !== null) {
                    $messagesGroupedByCustomer[] = [
                        'customer' => $currentCustomerMessages[0]['customer']->full_name,
                        'messages' => array_map(function ($msg) {
                            return [
                                'id' => $msg['id'],
                                'created_at' => $msg['created_at']->toDateTimeString(),
                                'text' => $msg['text'],
                            ];
                        }, $currentCustomerMessages),
                    ];
                    $messagesIdCounter++;
                }

                $currentCustomerId = $message['customer_id'];
                $currentCustomerMessages = [];
            }

            $currentCustomerMessages[] = $message;
        }

        if ($currentCustomerId !== null) {
            $messagesGroupedByCustomer[] = [
                'customer' => $currentCustomerMessages[0]['customer']->full_name,
                'messages' => array_map(function ($msg) {
                    return [
                        'id' => $msg['id'],
                        'created_at' => $msg['created_at']->toDateTimeString(),
                        'text' => $msg['text'],
                    ];
                }, $currentCustomerMessages),
            ];
        }
        $data['date'] = "Сегодня";
        $data['item'] = $messagesGroupedByCustomer;

        return $data;
    }

Comments

[Adamos]

Не нужны никакие промежуточные сборки, все делается одним циклом, просто используй индексы

if(!array_key_exists($customerId, $messages2)) {
  $messages2[$customerId] = ... //вся инфа + пустой массив ['messages']
}
$mesages2[$customerId]['messages][] = ... // собственно message
...
return array_values($messages2);

Answer 1

[nokimaro]

Делается одним проходом в цикле
Ключем выставляем поле по которому делать группировку и внутрь уже ложим нужные данные
В конце array_values() чтобы переиндексировать результат и сбросить ключи

https://3v4l.org/mZQIN#v8.3.14

<?php
error_reporting(-1);
ini_set('display_errors', 1);

$json_data = <<<JSON
{"messages": [
	{
		"id": 1,
		"customer": "Dr. Kane Hill",
		"customer_id": 1,
		"created_at": "2024-11-19 11:18:57",
		"text": "Hello, how are you?"
	},
	{
		"id": 2,
		"customer": "Dr. Kane Hill",
		"customer_id": 1,
		"created_at": "2024-11-19 11:20:57",
		"text": "I am good, thanks! How about you?"
	},
	{
		"id": 3,
		"customer": "Prof. Samir McClure III",
		"customer_id": 2,
		"created_at": "2024-11-19 11:21:57",
		"text": "Hey, what time is it?"
	},
	{
		"id": 4,
		"customer": "Prof. Samir McClure III",
		"customer_id": 2,
		"created_at": "2024-11-19 11:22:57",
		"text": "It is 3 PM."
	},
	{
		"id": 5,
		"customer": "Shad Leffler",
		"customer_id": 3,
		"created_at": "2024-11-19 11:23:57",
		"text": "Did you finish the project?"
	},
	{
		"id": 6,
		"customer": "Shad Leffler",
		"customer_id": 3,
		"created_at": "2024-11-19 11:24:57",
		"text": "New mesh"
	},
	{
		"id": 7,
		"customer": "Prof. Samir McClure III",
		"customer_id": 2,
		"created_at": "2024-11-19 11:26:57",
		"text": "Not bed?"
	},
	{
		"id": 8,
		"customer": "Prof. Samir McClure III",
		"customer_id": 2,
		"created_at": "2024-11-19 11:27:57",
		"text": "Cool?"
	}
]}
JSON;

$messages = json_decode($json_data, true)['messages'];

$messages2 = [];
$messageCounter = 0;
foreach($messages as $row) {
    
    $key = $row['customer_id'];
    
    if(!isset($messages2[$key])) {
        $messages2[$key] = [
            'customer' => $row['customer'],
            'message_id' => (++$messageCounter),
            'customer_id' => $row['customer_id'],
            'messages' => [],
        ];
    }

    
    $messages2[$key]['messages'][] = [
        'id' => $row['id'],
        'created_at' => $row['created_at'],
        'text' => $row['text'],
    ];
}

$messages2 = array_values($messages2);

echo json_encode($messages2, JSON_PRETTY_PRINT);

upd: подправил ответ, так как по всей видимости message_id в сгруппированном результате должен быть счётчиком
upd2: по совету. Adamos обновил ответ убрав не нужные присваивания при каждом проходе цикла

Comments

[Adamos]

Notice: undefined index будет сыпать, если не наставить @ перед каждой строчкой.
Я же привел корректный код в комментарии, этот колхоз с ?? только запутывает читающего.

[nokimaro]

Adamos, ничего не будет сыпать, запустите код и проверьте
?? не колхоз, а такой же обычный оператор вместо того чтобы использовать одну из функций типа isset / empty / array_key_exists

код в ответе необорот написан максимально просто, foreach и присваивания, без всяких выебонов со стрелочными функциями, array_map и прочим

[Adamos]

nokimaro, если отключить вывод notice - не будет, конечно.
Претензия не к оператору, а к дурному переписыванию значений в каждой итерации, да тремя отдельными операторами, вместо одной проверки, есть они или нет, и создания данных один раз при необходимости.

[nokimaro]

Adamos, в вопросе автора есть нюансы, например что ему нужно не просто сгруппировать сообщения, а сделать более сложную структуру, не потеряв порядок, где внутри каждой группы лежат не только сообщения, а ещё и значения customer, message_id, customer_id равные тем что встречаются при первом нахождении группы

[Adamos]

nokimaro, я умею читать.

$key = $row['customer_id'];
if(!array_key_exists($key, $messages2)) {
    $messages2[$key] = [
        'customer' => $row['customer'],
        'message_id' => $row['id'],
        'customer_id' => $row['customer_id'],
        'messages' => []
    ];
}

[nokimaro]

Adamos, повторю, никаких notice'ов не будет. Можете запустить код или почитать как работает null coalescing оператор.

[Adamos]

nokimaro, что ?? не будет выдавать notice - это понятно. Я полагал, что само присвоение подмассиву несуществующего массива будет ругаться - но, видимо, спутал с JS. Да, notice-ов не будет.
Насчет того, что у ТС должно быть в message_id - он, похоже, и сам не знает. Какой может быть ID у целого массива... тут гадать смысла нет.

[nokimaro]

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