you're using this semester here multiple times. You can pull this into a private method.

`````ruby
def new
  find_semester
  ...
end

private

def find_semester
  @semester = Semester.find(params[:semester_id])
end
```