sinnrrr / testapp Goto Github PK

Привіт!)
Ти гарно попрацював і щоб стало ще краще я зібрав кілька правок, які потрібно внести. Їх потрібно зробити у кількох контролерах, я не дублював однакові коментарі.

1. Якщо буде дуже багато коментарів, наприклад більше 10к, то сайт буде дуже тормозити.
   Або зовсім перестане працювати, якщо закіниться оперативна пам'ять.
   Потрібно завантажувати не всі, а частинами, придумакй як)
   

   testapp/app/Http/Controllers/CommentController.php

   Line 19 in 0f9a41a

   return response()->json(Comment::all());

2. Тут краще завантажувати дані через модель і вхідні дані з запиту потрібно валідувати (спробуй FormRequest)
   

   testapp/app/Http/Controllers/CommentController.php

   Lines 31 to 36 in 0f9a41a

   	$validateComments = DB::table('comments')
   	->where([
   	'owner_id' => $request->owner_id,
   	'marker_id' => $request->marker_id,
   	'body' => $request->body
   	])->first();

3. Тут else не потрібне, тільки ускладнює читання коду
   і краще написати коротше: abort(500)
   

   testapp/app/Http/Controllers/CommentController.php

   Lines 46 to 51 in 0f9a41a

   	if ($comment->save()) {
   	$comment->message = 'Comment successfully created';
   	return response()->json($comment);
   	} else {
   	return response()->view('errors.500', [], 500);
   	}

4. При виведенні даних з моделі в API краще використовувати Resource, щоб ти міг в одному місці міг контролювати, що виведеться в API і чи не потраплять дані до людей, які не повинні їх бачити
   

   testapp/app/Http/Controllers/CommentController.php

   Line 65 in 0f9a41a

   return response()->json(Comment::findOrFail($id));