diff --git a/modules/hero/module.go b/modules/hero/module.go
index a0ccb82ca..2cb36ce8d 100644
--- a/modules/hero/module.go
+++ b/modules/hero/module.go
@@ -949,7 +949,7 @@ func (this *Hero) QueryCrossMultipleHeroinfo(oid []string) (hero []*pb.DBHero, e
 					"_id": v,
 				})
 				_hero := &pb.DBHero{}
-				if err = sr.Decode(hero); err != nil {
+				if err = sr.Decode(_hero); err != nil {
 					this.modelHero.moduleHero.Errorf("find hero error: %v", err)
 				}
 				hero = append(hero, _hero)
diff --git a/modules/practice/api_enrolled.go b/modules/practice/api_enrolled.go
index 380fd247f..90deb4906 100644
--- a/modules/practice/api_enrolled.go
+++ b/modules/practice/api_enrolled.go
@@ -30,7 +30,10 @@ func (this *apiComp) Enrolled(session comm.IUserSession, req *pb.PracticeEnrolle
 		if code = this.module.ModuleHero.RegisterInstructor(session, req.Hero, req.Group); code != pb.ErrorCode_Success {
 			return
 		}
-		room.Full[req.Group]++
+		for i := 0; i < len(req.Hero); i++ {
+			room.Full[req.Group]++
+		}
+
 	} else {
 		if hero, err = this.module.ModuleHero.QueryCrossMultipleHeroinfo(req.Hero); err != nil {
 			code = pb.ErrorCode_ReqParameterError