diff --git a/app/views/users/_user_group_attr.html.erb b/app/views/users/_user_group_attr.html.erb
index 5d1475991..864ce4f76 100644
--- a/app/views/users/_user_group_attr.html.erb
+++ b/app/views/users/_user_group_attr.html.erb
@@ -1,10 +1,10 @@
 <div id="groupHomework">
   <div> <span class="f14 mt5 fl fontGrey3 mr10">分组设置：</span>
     <div class="mr50 fl"> <span class="f14 fontGrey3 mr5">每组最小人数：</span>
-      <input id="min_num" type="text" name="min_num" class="markInput" value="<%=(edit_mode && homework.is_group_homework?) ? homework.homework_detail_group.min_num : 2 %>" <%=not_allow_select ? "onchange = regex_scope(#{homework.homework_detail_group.min_num}, 1)" : '' %>/>人
+      <input id="min_num" type="text" name="min_num" class="markInput" value="<%=(edit_mode && homework.is_group_homework?) ? homework.homework_detail_group.min_num : 2 %>" <%=(not_allow_select && homework.is_group_homework?) ? "onchange = regex_scope(#{homework.homework_detail_group.min_num}, 1)" : '' %>/>人
     </div>
     <div class="fl"> <span class="f14 fontGrey3 mr5">每组最大人数：</span>
-      <input id="max_num" type="text" name="max_num" class="markInput" value="<%=(edit_mode && homework.is_group_homework?) ? homework.homework_detail_group.max_num : 10 %>" <%=not_allow_select ? "onchange = regex_scope(#{homework.homework_detail_group.max_num}, 2)" : '' %>/>人
+      <input id="max_num" type="text" name="max_num" class="markInput" value="<%=(edit_mode && homework.is_group_homework?) ? homework.homework_detail_group.max_num : 10 %>" <%=(not_allow_select && homework.is_group_homework?) ? "onchange = regex_scope(#{homework.homework_detail_group.max_num}, 2)" : '' %>/>人
     </div>
     <span class="c_red undis ml20 mt7 fl" id="min_max_num_notice"></span>
     <div class="cl"></div>
@@ -25,7 +25,7 @@
         if($.trim($("#base_on_project").val()) == 1) {
             $("#base_on_project").attr('checked','checked');
         }
-        <% if not_allow_select %>
+        <% if not_allow_select && homework.is_group_homework? %>
         $("#min_num").on('change', function() {
             var min_num = <%=homework.homework_detail_group.min_num %>;
             if(parseInt($(this).val()) > min_num) {
diff --git a/app/views/users/_user_homework_form.html.erb b/app/views/users/_user_homework_form.html.erb
index b8785dc9a..842cf50f4 100644
--- a/app/views/users/_user_homework_form.html.erb
+++ b/app/views/users/_user_homework_form.html.erb
@@ -14,7 +14,7 @@
             }else if(type == "2"){
                 $("#homeworkSetting").removeClass("undis");
                 $("#homeworkSetting").html("<%=escape_javascript(render :partial => 'users/user_programing_attr', :locals => {:edit_mode => edit_mode, :homework=>homework, :not_allow_select => not_allow_select}) %>");
-            }else{
+            }else if(type == "3"){
                 $("#homeworkSetting").removeClass("undis");
                 $("#homeworkSetting").html("<%=escape_javascript(render :partial => 'users/user_group_attr', :locals => {:edit_mode => edit_mode, :homework=>homework, :not_allow_select => not_allow_select}) %>");
             }