In the index method the last line has an extra indentation level, since it is a chained call:

In that destroy method `$post is only used once after it is assigned:

One could argue that a more appropriate name for $post would be $posts since it could contain multiple records (though perhaps id is the primary key and unique so that wouldn't really be possible). If a method like firstWhere() was used instead of where() then $post would be appropriate.

¹

In the index method the last line has an extra indentation level, since it is a chained call:

In that destroy() method $post is only used once after it is assigned:

Post::where('id',$id)->delete();

In the index() method a collection of model records is fetched with the all() method:

$post = Post::all();

Perhaps a more descriptive variable name would be $posts.

When calling where on a model class a query builder I.e. Illuminate\Database\Query\Builder Is returned. Thus perhaps a better name than $post in the destroy() method would be $query.

Helper method to delete records

As the documentation suggests the destroy() method could be used to delete a record without needing to call the where() method:

Post::destroy($id);

Though be aware that this may be slower and consume more resources, depending on what handlers for events may be defined on the model.

The destroy method loads each model individually and calls the delete method so that the deleting and deleted events are properly dispatched for each model. ²

📝 Review

🏗 Suggestions

👓 Readability

💬 Docblocks

💡 Language features

💡 Framework features

💾 Single use variable

🏷️ Variable names

Review

Suggestions

Readability

Docblocks

Language features

Framework features

Single use variable

Variable names

Review

Suggestions

📝 Review

🏗 Suggestions