diff --git a/app/helpers/homework_attach_helper.rb b/app/helpers/homework_attach_helper.rb
index d8d418da5..750a71f6a 100644
--- a/app/helpers/homework_attach_helper.rb
+++ b/app/helpers/homework_attach_helper.rb
@@ -96,13 +96,22 @@ module HomeworkAttachHelper
     if array.nil? || array.count == 0
       return "()"
     end
-    array.each do |member|
-      if member == array.last
-        ary += member.id.to_s + ")"
+    array.length.times do |i|
+      if i == array.length - 1
+        ary += array[i].id.to_s + ")"
       else
-        ary += member.id.to_s + ","
+        if  !(array[i].nil? ||  array[i].id.nil? || array[i].id.to_s == "")
+          ary +=  array[i].id.to_s + ","
+        end
       end
     end
+    #array.each do |member|
+    #  if member == array.last
+    #    ary += member.id.to_s + ")"
+    #  else
+    #    ary += member.id.to_s + ","
+    #  end
+    #end
     ary
   end
 end
\ No newline at end of file
diff --git a/app/views/homework_attach/_history.erb b/app/views/homework_attach/_history.erb
index 3376d696f..53de7d33f 100644
--- a/app/views/homework_attach/_history.erb
+++ b/app/views/homework_attach/_history.erb
@@ -49,12 +49,12 @@
                 <% ids = 'project_respond_form_'+ journal.id.to_s%>
                 <% if reply_allow %>
                     <div id='<%= ids %>' class="respond-form">
-                      <%= render :partial => 'words/new_respond', :locals => {:journal => journal, :m_reply_id => journal} %>
+                      <%= render :partial => 'words/new_respond', :locals => {:journal => journal, :m_reply_id => journal,:show_name => true} %>
                     </div>
                 <% end %>
                 <div style="clear: both;"></div>
                 <div>
-                  <%= render :partial => "words/journal_reply", :locals => {:journal => journal } %>
+                  <%= render :partial => "words/journal_reply", :locals => {:journal => journal, :show_name => true} %>
                 </div>
               </div>
               <div class="cl"></div>
diff --git a/app/views/homework_attach/_show_star.html.erb b/app/views/homework_attach/_show_star.html.erb
index e21b3bc24..6dc82e238 100644
--- a/app/views/homework_attach/_show_star.html.erb
+++ b/app/views/homework_attach/_show_star.html.erb
@@ -1,4 +1,4 @@
-<% if is_comprehensive_evaluation == 3 %>
+<% if is_comprehensive_evaluation == 3 || User.current == homework.user%>
     <%= l(:label_work_rating) %>：
     <%= render :partial => 'show_score', locals: {:stars => totle_score}  %>
 <% elsif  is_comprehensive_evaluation == 2 %>