Как вытащить отзывы из базы данных и отобразить в виде сетки

Question

Я следовал руководству по созданию сайта обзоров ресторанов на Ruby on Rails и адаптировал его для больниц.

Я хотел бы отображать результаты из базы данных в отсортированном формате, например, самые высокие отзывы или наибольшее количествообзоры или по местоположению (все доступные данные в базе данных) в виде таблицы (ограничено Top 6).

Я не уверен относительно того, с чего начать, я знаю, что это может звучать довольно просто, но я исследовали продолжал видеть так много вариантов и запутался относительно того, с чего начать.Некоторые решения подходят для небольших наборов данных, но я полностью ожидаю, что это будет расти в геометрической прогрессии в будущем, поэтому наиболее эффективный способ будет для меня лучшим.У меня есть SQLlite в моей локальной среде и Postgres в моей рабочей среде.

Это то, что у меня есть на моей странице поиска, которая отображает все мои больницы и то, что у меня есть на моей странице индекса, которая отображает все записи в сетке.Любая помощь будет принята с благодарностью.

search.html.erb

<div class="table-responsive table-box card"> 
  <%= form_tag search_hospitals_path, method: :get, class: "form-group" do %>
  <div class="input-group">
    <div class="input-group-addon">Search and Review</div>
          <p>
            <%= text_field_tag :search, params[:search],  class: "form-control formInput", placeholder: "Hospital Name" %>
          </p>
  </div>
    <% end %>

    <div class="container hospital_display">
      <% @hospitals.each do |hospital| %>
        <div class="row">
            <div class="col-sm-6">                
              <%= link_to image_tag(hospital.image), hospital, class: "responsive" %>
            </div>
              <div class="col-sm-6">
                  <%= link_to hospital.name, hospital %><br>
                  <%= hospital.address %><br>
                  <%= hospital.phone %><br>
                  <%= link_to hospital.website, hospital.website %>
                  <%# Checks for admin %>
                  <% if user_signed_in? && current_user.admin? %>
                    <%= link_to 'Edit', edit_hospital_path(hospital), class: "btn btn-link" %>
                    <%= link_to 'Destroy', hospital, method: :delete, data: { confirm: 'Are you sure?' }, class: "btn btn-link" %>
                  <% end%>
                  <%# Rating %>
                  <% if hospital.reviews.count == 0 %>
                    No reviews yet, be the first to write one
                  <% elsif hospital.reviews.count == 1 %>
                  <div class="star-rating" data-score= <%= hospital.avg_rating %> ></div>
                    1 Review
                  <% else %>
                  <div class="star-rating" data-score= <%= hospital.avg_rating %> ></div>
                    <%= hospital.length %> <%= "Reviews" %>
                <% end %>
              </div>
            </div>
          <% end %>
      </div>
    </div>
  </div>
</div>
<br>

<script>
    $('.star-rating').raty({
        path: 'https://s3-us-west-2.amazonaws.com/morafamedapp/stars',
        readOnly: true,
        score: function() {
            return $(this).attr('data-score');
        }
    });
</script>

У меня есть это на моей странице индекса index.html.erb

<div class="jumboFluid">
<div class="jumbotron">
<section class="content">


<%= form_tag search_hospitals_path, method: :get, class: "form-group" do %>
<div class="input-group">
<div class="input-group-addon">Search</div>
        <p>
          <%= text_field_tag :search, params[:search],  class: "form-control formInput", placeholder: "Eye, Maternity" %>
          <%# <%= submit_tag "Search", name: nil, class: "btn btn-default" %>
        </p>
</div>
  <% end %><br>



  <h3 class="intro2">Find the best HealthCare Options around you with Medapp.
  <br><%# Explore the best of healthcare available in your community.<br>
    Read and leave reviews to assist others seeking the best health care and keep our hospitals on their toes. %></h3>
</section>
</div>
</div>
<div class="hospitalList">
<h1 id="hospitalBanner">Hospitals</h1>
<blockquote> 
  <p class="text-center"><cite>&#8220;Explore the best of healthcare available in your community&#8221;</cite> </p>
</blockquote>

<% content_for(:body_attributes) do %>
    data-no-turbolink="false"
  <% end %>
<main>
<div class="container-fluid fluid">
<div class="row">
<% @hospitals.each do |hospital| %>
  <div class="col-md-4 col-xs-6">
    <div class="thumbnail">
        <%= link_to image_tag(hospital.image), hospital %>
        <div class="caption">
          <h4> <%= link_to hospital.name, hospital %></h4><br>
          <% if hospital.reviews.count == 0 %>
            0 Reviews
          <% elsif hospital.reviews.count == 1 %>
            <div class="star-rating" data-score= <%= hospital.avg_rating %> ></div>
            1 Review
            <% else %>
            <div class="star-rating" data-score= <%= hospital.avg_rating %> ></div>
            <%= hospital.length %> <%= "Reviews" %>
          <% end %>
    </div>
  </div>
  </div>
<% end %>
 </div>

</main>


<br>

<% if user_signed_in? && current_user.admin? %>
  <%= link_to 'New Hospital', new_hospital_path, class: "btn btn-primary btn-lg btn-special" %>
<% end %>

<!--Start of Tawk.to Script-->
<script type="text/javascript">
window.$_Tawk = undefined;
var Tawk_API=Tawk_API||{}, Tawk_LoadStart=new Date();
(function(){
var s1=document.createElement("script"),s0=document.getElementsByTagName("script")[0];
s1.async=true;
s1.src='https://embed.tawk.to/592da976b3d02e11ecc677a1/default';
s1.charset='UTF-8';
s1.setAttribute('crossorigin','*');
s0.parentNode.insertBefore(s1,s0);
s1.style.background = 'yellow';
})();
</script>
<!--End of Tawk.to Script-->

<script>
  $('.star-rating').raty({
    path: 'https://s3-us-west-2.amazonaws.com/morafamedapp/stars',
    readOnly: true,
    score: function() {
      return $(this).attr('data-score');
    }
  });
</script>

</div>

Hospitals_controller.rb

class HospitalsController < ApplicationController
  before_action :set_hospital, only: [:show, :edit, :update, :destroy]
  before_action :authenticate_user!, except: [:search, :index, :show]
  before_action :check_user, except: [:search, :index, :show]
  protect_from_forgery with: :null_session
  protect_from_forgery except: ["create"]

  def search
    if params[:search].present?
      @hospitals = Hospital.search(params[:search])
    else
      @hospitals = Hospital.all
    end
  end

  def import
   Hospital.import(params[:file])
  end


  # GET /hospitals
  # GET /hospitals.json
  def index
    @hospitals = Hospital.all
  end

  # GET /hospitals/1
  # GET /hospitals/1.json
  def show
    @reviews = Review.where(hospital_id: @hospital.id).order("created_at DESC")
    if @reviews.blank?
      @avg_rating = 0
    else
    @avg_rating = @reviews.average(:rating).round(2)
    end
  end

  # GET /hospitals/new
  def new
    @hospital = Hospital.new
  end

  # GET /hospitals/1/edit
  def edit
  end

  # POST /hospitals
  # POST /hospitals.json
  def create
    import if params[:file] # <= this here is the call to your import method

    @hospital = Hospital.new(hospital_params)

    respond_to do |format|
      if @hospital.save
        format.html { redirect_to @hospital, notice: 'Hospital was successfully created.' }
        format.json { render :show, status: :created, location: @hospital }
      else
        format.html { render :new }
        format.json { render json: @hospital.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /hospitals/1
  # PATCH/PUT /hospitals/1.json
  def update
    respond_to do |format|
      if @hospital.update(hospital_params)
        format.html { redirect_to @hospital, notice: 'Hospital was successfully updated.' }
        format.json { render :show, status: :ok, location: @hospital }
      else
        format.html { render :edit }
        format.json { render json: @hospital.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /hospitals/1
  # DELETE /hospitals/1.json
  def destroy
    @hospital.destroy
    respond_to do |format|
      format.html { redirect_to hospitals_url, notice: 'Hospital was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_hospital
      @hospital = Hospital.find(params[:id])
    end

    def check_user
    unless current_user.admin?
        redirect_to root_url, alert: "Sorry, only admins can do that!"
    end
  end

    # Never trust parameters from the scary internet, only allow the white list through.
    def hospital_params
      params.require(:hospital).permit(:name, :address, :city_town, :state,  :phone, :website, :safe_care, :jci, :cohsasa, :best_known_4, :image )
    end
end

Answer 1

Если вам не нужен сам доступ к значению, а нужен только заказ, я предлагаю перейти с andrew21s answer .После этого давайте перейдем к моему ответу.

Вопрос достаточно прост, хотя ответ сложнее, чем вы ожидаете.Я предполагаю следующие ассоциации в моделях:

class Hospital < ApplicationRecord
  has_many :reviews
end

class Review < ApplicationRecord
  belongs_to :hospital
end

С учетом вышеизложенного эта проблема может быть решена с помощью следующих областей:

class Hospital < ApplicationRecord

  # ...

  def self.include_review_count
    review_count = Review.select(Review.arel_table[:id].count)
                         .where(Review.arel_table[:hospital_id].eq(arel_table[:id]))
                         .as('review_count')

    select(*attribute_names.map(&:to_sym)).select(review_count)
  end

  def self.include_review_average
    review_average = Review.select(Review.arel_table[:rating].average)
                           .where(Review.arel_table[:hospital_id].eq(arel_table[:id]))
                           .as('review_average')

    select(*attribute_names.map(&:to_sym)).select(review_average)
  end

  # ...

end

Имея эти области видимости, вы теперь можете сделать следующее в вашем контроллере:

class HospitalsController < ApplicationController

  # ...

  def index
    @hospitals = Hospital.include_review_count
                         .include_review_average
                         .order('review_count DESC', 'review_average DESC', :id)
                         .limit(6)
  end

  # ...

end

Примечание: В приведенном выше коде вы не можете использовать .order(review_count: :desc, review_average: :desc, id: :asc), потому что "review_count"это не настоящий атрибут, а просто псевдоним.По этой причине вам придется использовать буквенный (String) вариант.То же самое верно при использовании этого добавленного атрибута в операторе , где .Например: .where(review_count: 2) не работает, тогда как .where('review_count = ?', 2) работает.

Это решение выполняет только один запрос, а также позволяет получить доступ к значениям без дополнительных запросов.Примером может быть (index.html.erb):

<table>
  <thead>
    <tr>
      <th>ID</th>
      <th>Name</th>
      <th>Review Count</th>
      <th>Review Average</th>
    </tr>
  </thead>
  <tbody>
    <% @hospitals.each do |hospital| %>
      <tr>
        <td><%= hospital.id %></td>
        <td><%= hospital.name %></td>
        <td><%= hospital.review_count %></td>
        <td><%= hospital.review_average %></td>
      </tr>
    <% end %>
  </tbody>
</table>

Answer 2

В вашей модели больницы вы можете просто определить области, которые вы можете добавить к любому запросу:

# app/models/hospital.rb
class Hospital < ActiveModel
  scope :order_by_highest_rated, -> { includes(:reviews).group("hospital_id").order("avg(reviews.rating) desc") }
  scope :order_by_most_reviews, -> { includes(:reviews).group("hospital_id").order("count(reviews.id) desc") }
end

, затем в контроллере больницы вы можете добавить функциональность для выбора порядка сортировки

class HospitalsController < ApplicationController
  def index
    if params[:search_by] == 'highest_ranked'
      @hospitals = Hospital.order_by_highest_rated
    else 
      @hospitals = Hospital.order_by_most_reviews
    end
  end
end

См. здесь и здесь для получения дополнительной информации