diff --git a/public/react/src/modules/courses/members/ChangeRolePop.js b/public/react/src/modules/courses/members/ChangeRolePop.js
index 9883c488f..d81c6db46 100644
--- a/public/react/src/modules/courses/members/ChangeRolePop.js
+++ b/public/react/src/modules/courses/members/ChangeRolePop.js
@@ -8,9 +8,11 @@ import axios from 'axios'
 */
 function ChangeRolePop({ member_roles = [], record, courseId, onChangeRoleSuccess, showNotification, getUserId, fetchUser }) {
     const [checkBoxRoles, setCheckBoxRoles] = useState(member_roles)
-    useEffect(() => {
-        setCheckBoxRoles(member_roles)
-    }, [member_roles])
+    // useEffect(() => {
+        // if (checkBoxRoles.length != member_roles.length) {  // 死循环
+            // setCheckBoxRoles(member_roles)
+        // }
+    // }, [member_roles])
     function onCheckBoxChange(val) {
         console.log(val)
 
diff --git a/public/react/src/modules/courses/members/studentsList.js b/public/react/src/modules/courses/members/studentsList.js
index ab29dde45..33d434329 100644
--- a/public/react/src/modules/courses/members/studentsList.js
+++ b/public/react/src/modules/courses/members/studentsList.js
@@ -149,7 +149,7 @@ const buildColumns = (that,isParent) => {
         return (
           <React.Fragment>
             <WordsBtn style2={{ marginRight: '12px' }} onClick={() => that.onDelete(record)} style={'grey'}>删除学生</WordsBtn>              
-            <ChangeRolePop
+            {record.member_roles && record.member_roles.length && <ChangeRolePop
               courseId={courseId}
               record={record}
               member_roles={record.member_roles}
@@ -157,7 +157,7 @@ const buildColumns = (that,isParent) => {
               showNotification={that.props.showNotification}
               getUserId={that.props.isUserid}
               fetchUser={that.props.fetchUser}
-            ></ChangeRolePop>
+            ></ChangeRolePop>}
           </React.Fragment>
         )
       },