Je n'arrive pas à comprendre comment les modèles et les associations sont sauvegardés dans les rails

Question

Je suis novice dans le domaine du rubis et des rails et je souhaite suivre au mieux les normes et les conventions afin de ne pas prendre de mauvaises habitudes. J'ai deux modèles: cours et emplacement. Un cours appartient à un emplacement, car un cours ne peut avoir qu'un seul emplacement. Un emplacement a beaucoup de cours, car un emplacement peut être partagé par plus d'un cours.

Lors de la création d'un cours, il se peut qu'il existe déjà un emplacement identifié par son ID. Ou l'emplacement n'existe peut-être pas encore, auquel cas un nouvel enregistrement d'emplacement doit être créé. Mon contrôleur de cours a l'action de création suivante.

def create 
    @course = Course.new(params[:course]) 

    if params[:course][:location][:id].blank? 
    @course.location = Location.create(params[:course][:location]) 
    else 
    @course.location = Location.find_by_id(params[:course][:location][:id]) 
    end 

    @course.save 

    respond_with @course 
end 

Notez qu'il s'agit d'une API REST qui répond uniquement avec JSON. Le javascript qui rend les messages de demande un tableau JSON dans le même format qui serait renvoyé par une requête GET

{ 
    "course": 
    { 
    "title":"US History", 
    "credits":"3", 
    "max_students":"100", 
    "location": 
    { 
     "id":"", 
     "building":"Freedom Hall", 
     "room":"301" 
    } 
    } 
} 

or 

{ 
    "course": 
    { 
    "title":"US History", 
    "credits":"3", 
    "max_students":"100", 
    "location": 
    { 
     "id":"12", # this is the only difference 
     "building":"Freedom Hall", 
     "room":"301" 
    } 
    } 
} 

1. Par rapport à tous les exemples que je l'ai lu, ce code ne semble pas si élégant . Y a-t-il une meilleure façon de le prendre en compte?
2. Si Location.create déclenche une exception, @ course.save sera-t-il toujours appelé? Dois-je utiliser Location.create !?
3. De même, les erreurs de validation se retrouveront-elles dans @ course.errors même si les erreurs se trouvaient sur le modèle Location? Ai-je besoin de sauver de l'exception pour que je puisse retourner les erreurs au client?

Merci beaucoup pour toute l'aide!

Answer 1

Vous pouvez nettoyer ça à l'aide find_or_initialize_by_id. Cela devrait fonctionner:

def create 
    @course = Course.new(params[:course]) 
    @course.location = Location.find_or_initialize_by_id(params[:course][:location][:id], 
                 params[:course][:location]) 
    @course.save 
    respond_with @course 
end 

En ce qui concerne votre deuxième question, dans votre code que vous avez @course.save ne sera pas appelée si Location.create (ou Location.find) soulève une exception (parce qu'ils se produiraient en premier). Cependant, la façon dont je l'ai codé ci-dessus, l'exception se produirait au même point dans le code, quand save est appelée, à quel point l'association est également enregistrée.

Answer 2

Essayez cela, dans votre contrôleur

def new 
    @course = Course.new 
    @location = @course.location.build # if one..many relationship 
    @location = @course.build_location # if one..one relationship 
end 

def create 
@course = Course.new(params[:course]) 
if @course.save 
    respond_with @course 
else 
    render :action => "new" 
end 
end 

plus sur nested_attributes