The feedback from arlyxiao

Hi there 👋.

Current is the future.

Utilities

render - Cloud service.
backblaze - Store images.
diagrams - Friendly working flow tool.
namesilo - I always forgot the username/password.
racknerd - Server host.
codeclimate - Help improve coding quality.

feedback's People

Contributors

Stargazers

Watchers

feedback's Issues

重构了几个地方

第一个问题,已改完

措词一定要统一，方便大家沟通，和理解
比如 problem_reposts 中文统一用 ‘问题’ 这个词，不要一个页面用 ‘问题’ 这个词，另一个页面用 ‘报告’ 这个词

第二个问题,已改完

ProblemReportsController 的 index 方法，抽取成多个小方法

第三个问题,已改完

ProblemReportsController 的 create 方法中

@problem_report 先 create 又 save,这样不好，完全可以只 save 一次

第四个问题,已改完

用户提交反馈的表单中（/problem_reports/new.html.haml）

  %input{:type=>'hidden',:value => field.id, :name=>"problem_report[problem_field_data_attributes][#{field.id}][problem_field_id]"}

  %input{:type=>'text', :name=>"problem_field_data[#{field.id}]"}

改为

%input{:type=>'hidden',:value => field.id, :name=>"problem_report[problem_field_data_attributes][#{field.id}][problem_field_id]"}

%input{:type=>'text', :name=>"problem_report[problem_field_data_attributes][#{field.id}][content]"}

这样修改后，rails 可以把 problem_field_datas 的数据完全拼装好，下边的内容就可以省去了

@problem_report.problem_field_data.each do |field_data|
    field_data.content = params[:problem_field_data][field_data.problem_field_id.to_s]
    field_data.save
end

第五个问题,已改完

创建 problem_reports 时的 attachement 关联移动到 problem_reports 模型的 after_create 上
遵守逻辑尽量封装到模型上的原则

  attr_accessor :attachement_ids

  after_create :set_attachements
  def set_attachements
    unless self.attachement_ids.blank?
      self.attachement_ids.each do |attachement_id|
        problem_report_attachement = ProblemReportAttachement.find(attachement_id)
        problem_report_attachement.problem_report = self
        problem_report_attachement.creator = self.creator
        problem_report_attachement.save
      end

      # 生成 zip 包
      # self.build_attachements_zip(self.creator)
    end
  end