Is there a more elegant way to do this in Rails (has_one association)

Question

Hey, I'm very new to Ruby and Rails. I was wondering if I was doing this the right way.

@user = User.new(params[:user])
@student.user = @user
@student.save
if @user.save
  ...rest of the code

In my app, a Student has one User, and each User belongs to a Student (think of a User as an Account). The association itself isn't my question though; is the block of code above the best way to save a new User that's associated to a Student?

Answer 1

If the correspondence between user and student is 1:1 may be this relationship is redundant. (maybe I'm missing something).

A cooler approach would be perform all the operations you need to the user object and then the last one would be:

@student.user = @user
@student.save

Answer 2

I think you have @-overload here. Only use @ if you want to make the variable an instance variable of the class, rather than a local variable that only exists inside your function. Anyway, here's one possible way to do it:

@student.user = User.create(params[:user])
@student.save

if @student.user.new_record?
  # didn't get saved...
end

ActiveRecord::Base#create creates a new object, tries to save it to the database, and then returns the object, and is a useful shortcut for the new-save pattern. It always returns the object, though, so you need to ask whether or not it was successfully saved, hence new_record?.

Answer 3

If the Student has_one User, then you should put the foreign key on the User. As in, the User will have the student_id column in the database. A user_id column on the Student isn't necessary.

In which case you can just do this:

@user = User.new(params[:user])
@user.student = @student
if @user.save
  ...rest of the code

And you won't even need to modify @student.

Answer 4

I'm surprised nobody mentioned the right way.

@user = @student.build_user(params[:user])

if @user.save
   # ... rest of the code ...
end

This way @user will be built already associated with @student.