编写《作业查询存储过程》

Signed-off-by: alan <547533434@qq.com>

app/controllers/courses_controller.rb

@@ -285,7 +285,7 @@ class CoursesController < ApplicationController
     member.course_group_id = group.id
     member.save
     @group = group
-
+    @members = student_homework_score
     @course_groups = @course.course_groups
     @membercount = @members.count
     search_group_members group
@@ -298,7 +298,7 @@ class CoursesController < ApplicationController
     member.save
     @group = group
     @course_groups = @course.course_groups
-
+    @members = student_homework_score
     @membercount = @members.count
     search_group_members group
   end
@@ -309,7 +309,7 @@ class CoursesController < ApplicationController
     @is_remote = true
     if params[:group_id] && params[:group_id] != "0"
       @group = CourseGroup.find(params[:group_id])
-      @results = student_homework_score.find_all {|mem| mem.course_group_id == group.id}
+      @results = student_homework_score.find_all {|mem| mem.course_group_id == @group.id}
     else
       @results = student_homework_score
     end

app/controllers/members_controller.rb

@@ -129,7 +129,7 @@ class MembersController < ApplicationController
               member =  Member.new(:role_ids => params[:membership][:role_ids], :user_id => user_id)
               role = Role.find_by_id(params[:membership][:role_ids])
               # 这里的判断只能通过角色名，可以弄成常量
-              if role.name == "学生"
+              if role.name == "学生" || role.name == "Student"
                 StudentsForCourse.create(:student_id => user_id, :course_id =>@course.id)
               end
               members << member
@@ -210,7 +210,7 @@ class MembersController < ApplicationController
         if (params[:membership][:role_ids])
           role = Role.find(params[:membership][:role_ids][0])
           # 这里的判断只能通过角色名，可以弄成常量
-          if role.name == "学生"
+          if role.name == "学生"  || role.name == "Student"
             StudentsForCourse.create(:student_id => @member.user_id, :course_id =>@course.id)
           else
             joined = StudentsForCourse.where('student_id = ? and course_id = ?', @member.user_id,@course.id)

db/migrate/20141226074532_select_member_score.rb

@@ -1,14 +1,12 @@
-
+#encoding=UTF-8
-DELIMITER $$
+class SelectMemberScore < ActiveRecord::Migration
+  def up
+    sql = ("
 
 CREATE
-    /*[DEFINER = { user | CURRENT_USER }]*/
+
     PROCEDURE `member_score`(IN courseid INT)
-    /*LANGUAGE SQL
+
-    | [NOT] DETERMINISTIC
-    | { CONTAINS SQL | NO SQL | READS SQL DATA | MODIFIES SQL DATA }
-    | SQL SECURITY { DEFINER | INVOKER }
-    | COMMENT 'string'*/
     BEGIN
 
 	CREATE TEMPORARY TABLE IF NOT EXISTS course_teachers (
@@ -61,13 +59,12 @@ CREATE
 
          INSERT INTO course_teachers (SELECT members.user_id FROM members WHERE members.user_id NOT IN (SELECT student_id FROM students_for_courses WHERE course_id = courseid));
 
-				#查出所有的学生
+
 
  INSERT INTO mems (id, user_id, course_id, created_on, course_group_id)
 						(SELECT members.id, members.user_id, members.course_id, members.created_on, members.course_group_id
 							FROM members WHERE course_id = courseid AND members.user_id NOT IN (SELECT id FROM course_teachers));
 
-				# 查出所有的学生列表及其作业
          INSERT INTO mem_home (id, user_id, course_id, created_on, course_group_id,home_id)
 						(SELECT members.id, members.user_id, members.course_id, members.created_on, members.course_group_id, homework_attaches.id
 							FROM members, homework_attaches WHERE course_id = courseid
@@ -92,6 +89,12 @@ CREATE
 		UPDATE mems, scores SET mems.score = scores.score WHERE mems.user_id = scores.user_id;
 
 		SELECT * FROM mems ORDER BY score DESC;
-    END$$
+    END;
 
-DELIMITER ;
+    ")
+      execute(sql)
+  end
+
+  def down
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20141210070327) do
+ActiveRecord::Schema.define(:version => 20141226074532) do
 
   create_table "activities", :force => true do |t|
     t.integer "act_id",   :null => false