求职板项目中关联Job与JobApplication对象的Ruby逻辑咨询

• 核心逻辑错误：「撤回申请」按钮的判断条件包含了JobApplication不存在的场景，直接导致未申请职位的用户也会看到撤回按钮，完全不符合需求。
• 重复查询：同一条JobApplication记录在视图里重复查询了6次，会产生不必要的数据库请求，性能低下。
• 请求方法错误：跳转申请表单应该使用GET请求访问new路径，原有代码对new路径使用POST请求，路由匹配会直接报错。
• 逻辑冗余：两个按钮的判断条件完全重复，没有做变量复用，维护性差。

第一步：Controller层提前预查询 #

在JobsController的show动作中提前查询当前用户的对应职位申请记录，避免视图层重复查库：

# app/controllers/jobs_controller.rb
def show
  # 原有其他逻辑保持不变
  @job_application = current_user&.role == 'applicant' ? JobApplication.find_by(job: @job, user: current_user) : nil
end

第二步：替换视图层逻辑 #

将原有app/views/job/show.html.erb中的对应代码替换为以下内容：

<% if current_user.try(:role) == 'applicant' %>
  <div class="price-btn">
    <%# 申请按钮仅在无申请记录、或申请状态为已邀请时显示 %>
    <% if @job_application.nil? || @job_application.status == 'invited' %>
      <% if @job_application.nil? %>
        <%# 无申请记录（未收到邀请）跳转至申请表单 %>
        <%= link_to "Apply to Job", new_job_job_application_path(@job), class: "btn btn-lg btn-dark" %>
      <% else %>
        <%# 已收到邀请，点击直接更新申请状态为已申请 %>
        <%= link_to "Apply to Job", job_application_path(@job_application, status: 'applied'), method: :patch, class: "btn btn-lg btn-dark" %>
      <% end %>
    <% end %>

    <%# 撤回按钮仅在存在申请记录时显示，未申请过的用户不会看到该按钮 %>
    <% if @job_application.present? %>
      <%= link_to "Withdraw from Job", job_application_path(@job_application, status: 'rejected'), method: :patch, class: "btn btn-lg btn-dark" %>
    <% end %>
  </div>
<% end %>

• 移除了撤回按钮判断中允许无申请记录的逻辑，仅当用户有对应职位的申请记录时才显示撤回按钮，完全匹配需求。
• 提前在Controller层完成申请记录查询，视图层全程复用变量，消除了多次无效数据库查询。
• 修正了请求方法：跳转表单使用默认GET请求，修改已有申请状态使用符合RESTful规范的PATCH请求，避免路由报错。
• 删减了大量冗余重复的判断逻辑，代码可读性和维护性大幅提升。

内容的提问来源于stack exchange，提问作者user10953115