reduce the number of loop in your code. Instead of students.each do |student| student.enrol(class) end. Do students.all.enrol(class).