refactor course helper

app/helpers/courses_helper.rb

@@ -9,56 +9,52 @@ module CoursesHelper
 	TeacherRoles = [3, 4, 7, 9]
 	StudentRoles = [5, 10]
 
-
+	## return people count
-	# 返回学生数量，即roles表中定义的Reporter
-	def studentCount project
-		searchStudent(project).count
-	end
 
 	# 返回教师数量，即roles表中定义的Manager
 	def teacherCount project
+		# searchCountByRoles project, TeacherRoles
+		# or
 		searchTeacherAndAssistant(project).count
 	end
+	# 返回学生数量，即roles表中定义的Reporter
+	def studentCount project
+		# searchCountByRoles project,StudentRoles
+		# or
+		searchStudent(project).count
+	end
 
 	# =====================================================================================
-
+	# return people list
 	def searchTeacherAndAssistant project
-		members = []
+		searchPeopleByRoles(project, TeacherRoles)
-		TeacherRoles.each do |role|
-			members += searchPeopleByRoles(project, role)
-		end
-		members
 	end
 
 	def searchStudent project
-		members = []
+		searchPeopleByRoles(project, StudentRoles)
-		StudentRoles.each do |role|
-			members += searchPeopleByRoles(project, role)
-		end
-		members
 	end
 	# =====================================================================================
-	#useless
+	
 	def searchCountByRoles project, roles_id
 		people = searchPeopleByRoles project, roles_id
 		people.count
 	end
 
-	#useless
+	def searchPeopleByRoles project, roles_id
-	def searchPeopleByRole project, role_id
 		people = []
 		begin
-			people = project.members.joins(:member_roles).where("member_roles.role_id = :role_id", {:role_id => role_id })
+			people = project.members.joins(:member_roles).where("member_roles.role_id IN (:role_id)", {:role_id => roles_id})
 		rescue Exception => e
 			logger.error "[CoursesHelper] ===> #{e}"
 		end
 		people
 	end
 
-	def searchPeopleByRoles project, roles_id
+	#useless
+	def searchPeopleByRole project, role_id
 		people = []
 		begin
-			people = project.members.joins(:member_roles).where("member_roles.role_id IN (:role_id)", {:role_id => roles_id})
+			people = project.members.joins(:member_roles).where("member_roles.role_id = :role_id", {:role_id => role_id })
 		rescue Exception => e
 			logger.error "[CoursesHelper] ===> #{e}"
 		end