This is going to run a separate query, so if `current_course_data.students` are accessed anywhere else in this request you will be loading them all in twice. It would be better to filter out current_student in the view, e.g.

```ruby
current_course_data.students - [current_student]
```

Or define a method on CourseData to do the same, if that feels too wordy for a view.