Как осортировать посты юзера по дате создания?

Question

[Иван]

Добрый день. Выбираю посты одного юзера по его id. Оно их сортирует в таком порядке как в базе данных. А мне надо сортировать по created_at, чтоб последные были первыми. Пробовал orderBy('created_at', 'desc'), оно пишет что такого метода нет. На методы sortBy('created_at') и sortByDesc('created_at') ничто не ругается, то и ничего они не меняют. Как это сделать? Метод где это делается примерно такой:

public function getPostsById($id)
	{
		$model = Post::findOrFail($id);
		$response = $model->posts;
		return response()->json($response);
	}

Comments

[Anton October]

судя по коду вы получаете 1 Post по id, а затем выполняете от него связь ->posts, не могли бы скинуть что происходит в этой связи?

Answer 1

[Anton October]

Вам нужно получить посты одного юзера

$user = User::find($id);
$posts = $user->posts; // Если есть связь от одного пользователя.

return response()->json($posts->sortByDesc('created_at')); // и вызвать сортировку коллекции.

Плюс нужно понимать различия:
1) orderBy это сортировка в QueryBuilder, в SQL грубо говоря Прочесть тут. Поэтому в вашем случае писало что нет такого метода ибо у коллекции вызывали его.

как вариант с orderBy

$posts = Post::where('user_id', '=', $user_id)
->orderBy('created_at', 'desc')
->get();

return response()->json($posts);

2) А sortBy / sortByDesc сортируют готовую коллекцию Почитать тут (пример выше).

Comments

[Иван]

Ошибок нет, но и результата тоже нет. Связь hasMany примерно такая как Вы написали...

[Иван Родичев]

Можете уточнить, почему нужно получать посты именно отношением у юзера?
Не проще ли ->where('user_id', $userId) у модели Post?

[Иван]

Иван Родичев, Так уже сделано. Не я это писал. Мне надо просто отсортировать.

[Anton October]

Иван Родичев, удобнее, если нужно получить посты одно пользователя, для этого есть отношение. Всегда его используем и имеем логичный результат.

С ->where('user_id', $userId) тоже самое получим. Но нужно писать больше кода. И если мы используем Eloquent почему бы не создавать связи чтобы было проще читать и понимать код.

Но да, вы правы, этот вариант тоже имеет место быть дополню ответ.

[Иван]

Anton October, Да, сортирует как надо. Спасибо. Только надо orderBy('created_at', 'desc).

[jazzus]

Нет смысла прописывать отношения и потом писать везде where('user_id', '=', $user_id)

[Anton October]

jazzus, это альтернативы, а не комплекс.

[jazzus]

Anton October, это не альтернативы, а бессмысленный код. Есть написанное отношение с ключами которое ты не используешь.

Answer 2

[jazzus]

$user->posts - коллекция
$user->posts() - запрос
запрос с сортировкой:

$user->posts()
     ->latest()
     ->get();

Comments

[Иван]

Да... Работает. Спасибо. Но человек трудился, много написал, пусть в этот раз его ответ будет решением.