Comment écrire des rails

#Mauvaise écriture
@incorrect_questions = session[:incorrect_question_ids].map { |n| Question.find(n.to_i) }

#Pourquoi pas?
#Si tel est le cas, une requête qui appelle la Question sera émise pour chaque nombre de cas, et s'il y a 100, 100 requêtes seront émises, c'est donc mieux.

#Bonne écriture
@incorrect_questions = Question.where(id: session[:incorrect_question_ids].map(&:to_i))

Concept de rails

・ L'opinion de DHH est qu'il est préférable d'utiliser un contrôleur différent pour des actions qui n'existent pas par défaut, et je le pense aussi. ・ Qu'est-ce que REST? ・ L'une des idées orientées objet est que l'objet a son propre état. ・ Il est dangereux d'utiliser la session, alors pourquoi ne pas préparer l'état de la personne à table? ・ Évitez le nom Test ◯◯ car il peut être confondu avec un fichier rspec. -Dans le système de rappel (before_save etc.), la méthode s'exécute involontairement, il n'est donc pas très préférable d'y écrire la logique.

#Mauvaise écriture
def index
  @questions = Question.all
end

#Pourquoi non?
#Je pense qu'il n'y a pas de problème avec le numéro actuel, mais si vous utilisez tout dans le service réel, vous devez faire attention car la mémoire peut être insuffisante et elle peut chuter.
Fondamentalement, la modification dynamique des données (par exemple, ce que l'utilisateur ajoute) est de sélectionner/Je pense qu'il vaut mieux prendre l'habitude de limiter les colonnes requises avec du pincement etc.

#Mauvaise écriture
<%= form_for(@question_similar, url: { controller: 'question_similar', action: 'create' }) do |f| %>

#Les raisons ne sont pas bonnes
url: {〜〜〜}Est xxx_Il semble que vous puissiez écrire avec chemin

#Mauvaise écriture
 def cancel_save
    question_similars.each do |question_similar|
      if question_similar.similar_word == ""
        question_similar.delete
      end
    end
  end

#.Utiliser du blanc
#Transaction ok?
#delete_Je pense que tout a de meilleures performances

params[:front_article_search_form][:heart_period]

Ce type de hachage est appelé un hachage bidimensionnel, Si le paramètre [: front_article_search_form] est nul en premier lieu Une erreur comme la méthode non définie `[] 'pour nil: NilClass (NoMethodError) se produit souvent. Vous devez donc faire attention à la façon de l'écrire.

Si c'était moi

article_search_form = params[:front_article_search_form]
if article_search_form.present?
 @heart_period = article_search_form[:heart_period]
end