Was ist das Problem mit dem Controller-Code unten? Wie würdest du es reparieren?

Question

class CommentsController < ApplicationController 
    def users_comments 
    posts = Post.all 
    comments = posts.map(&:comments).flatten 
    @user_comments = comments.select do |comment| 
     comment.author.username == params[:username] 
    end 
    end 
end 

Answer 1

Dies wird so viele Abfragen erzeugen. Dieses Problem wird n + 1 genannt (siehe hier: https://www.sitepoint.com/silver-bullet-n1-problem/).

Ich schlage vor, mit so etwas wie diesem Code:

@user_comments = Comment.where(author: { username: params[:username] }) 

Ich glaube, Sie nicht brauchen die Beiträge habe ich recht?

Answer 2

Dadurch wird ActiveRecord angewiesen, die entsprechenden Kommentar- und Autoreneinträge sofort nach der ersten Anforderung für alle Posts aus der Datenbank abzurufen, wodurch die Anzahl der Datenbankanforderungen auf nur drei reduziert wird.

Answer 3

Es ist nur eine normale allgemeine Frage, die jeder googlen kann, da es die kleinste Änderung am bestehenden Code erfordert und keine Annahmen bezüglich der umgekehrten Zuordnung von Kommentar zu Beitrag macht. Lesen Sie mehr Interessante Fakten über Ruby on Rails Development von hier