diff --git a/app/views/layouts/_logined_header.html.erb b/app/views/layouts/_logined_header.html.erb
index 41fdfc64..6e239e5d 100644
--- a/app/views/layouts/_logined_header.html.erb
+++ b/app/views/layouts/_logined_header.html.erb
@@ -5,7 +5,7 @@
       <li class="<%= subjects_controller.include?(params[:controller]) ? " active" : "" %>"><%= link_to "实训课程", subjects_path %></li>
       <li class="<%= course_controller.include?(params[:controller]) ? " active" : "" %>"><%= link_to "翻转课堂", courses_path %></li>
       <!-- 精选实训 -->
-      <li class="pr<%= shixuns_controller.include?(params[:controller]) ? " active" : "" %>"><%= link_to "开发社区", shixuns_path %><img src="/images/educoder/hot-h.png" class="nav-img" /></li>
+      <li class="pr <%= shixuns_controller.include?(params[:controller]) ? " active" : "" %>"><%= link_to "开发社区", shixuns_path %><img src="/images/educoder/hot-h.png" class="nav-img" /></li>
       
       <% careers = Career.published.order("created_at asc") %>
       <% if careers.present? %>
diff --git a/app/views/layouts/_unlogin_header.html.erb b/app/views/layouts/_unlogin_header.html.erb
index fb1e901e..d970fffc 100644
--- a/app/views/layouts/_unlogin_header.html.erb
+++ b/app/views/layouts/_unlogin_header.html.erb
@@ -3,16 +3,14 @@
   <div class="head-nav pr">
     <ul id="header-nav">
       <li class="active" style="display: none"><%= link_to "首页", home_path %></li>
-      <li><%= link_to "实训课程", subjects_path %></li>
-      <li><%= link_to "翻转课堂", courses_path %></li>
+      <li class="<%= subjects_controller.include?(params[:controller]) ? " active" : "" %>"><%= link_to "实训课程", subjects_path %></li>
+      <li class="<%= course_controller.include?(params[:controller]) ? " active" : "" %>"><%= link_to "翻转课堂", courses_path %></li>
+
+      <li class="pr <%= shixuns_controller.include?(params[:controller]) ? " active" : "" %>"><%= link_to "开发社区", shixuns_path %><img src="/images/educoder/hot-h.png" class="nav-img" /></li>
 
-      <li class="pr">
-        <%= link_to "开发社区", shixuns_path %>
-        <img src="/images/educoder/hot-h.png" class="nav-img" />
-      </li>
       <% careers = Career.published.order("created_at asc") %>
       <% if careers.present? %>
-        <li class="fl edu-menu-panel headIcon careershover" style="cursor: auto;">
+        <li class="fl edu-menu-panel headIcon careershover <%= params[:action] == "index" && params[:controller] == "careers" ? " active" : "" %>" style="cursor: auto;">
           <p>职业路径</p>
           <ul class="edu-menu-list edu-menu-listnew" style="top:60px">
             <% careers.each do |career| %>
@@ -21,8 +19,8 @@
           </ul>
         </li>
       <% end %>
-      <li><%= link_to "竞赛", competitions_path %></li>
-      <li><%= link_to "问答", forums_path %></li>
+      <li class="<%= params[:controller] == "competitions" ? " active" : "" %>"><%= link_to "竞赛", competitions_path %></li>
+      <li class="<%= params[:controller] == "forums" ? " active" : "" %>"><%= link_to "问答", forums_path %></li>
       <!--<li><%#= link_to "活动竞赛", competitions_path %></li>-->
     </ul>
     <div class="posi-search unlogin" id="posi-search" style="display: none" >
@@ -89,21 +87,5 @@
     window.onscroll=function(){
         var sl=-Math.max(document.body.scrollLeft,document.documentElement.scrollLeft);
         document.getElementById('nHeader').style.left=sl+'px';
-    }
-  $(function () {
-      var $header = $("#header-nav").children("li");
-      <% if params[:controller] == "welcome" %>
-      $("#header-nav").children("li").eq(0).addClass("active");
-      <% elsif subjects_controller.include?(params[:controller]) %>
-      $("#header-nav").children("li").eq(1).addClass("active");
-      <% elsif course_controller.include?(params[:controller]) %>
-      $("#header-nav").children("li").eq(2).addClass("active");
-      <% elsif shixuns_controller.include?(params[:controller]) %>
-      $header.length == 7 ? $header.eq(2).addClass("active") : $header.eq(3).addClass("active");
-      <% elsif params[:controller] == "competitions" %>
-      $header.length == 7 ? $header.eq(5).addClass("active") : $header.eq(4).addClass("active");
-      <% elsif params[:controller] ==  "forums" %>
-      $header.length == 7 ? $header.eq(6).addClass("active") : $header.eq(5).addClass("active");
-      <% end %>
-  })
+    };
 </script>
\ No newline at end of file