在Ruby on Rails中推送到数组

Question

I am trying to create a string array inside my model, where I can push new strings (IDs) into the string array - but it's not pushing the correct values.

Here is how the model looks:

class CreateCompanies < ActiveRecord::Migration
  def change
  create_table :companies do |t|

  t.string :name
  t.string :email
  t.string :users, array: true, default: []
  t.timestamps null: false
  end
 end
end

And here is how I'm adding the new values to the array in the Controller:

c = Company.new
c.users << @user.id
c.save

A value does indeed get added to the array, but it's not the ID but just some random value like this:

users: "--- []\n\u{478A6}"

Been stuck at this for a couple of days now - does anyone know what I'm missing?

UPDATE: Added the code where I create a new Company

 def create
@user = User.new(user_params)
if @user.save

  @user.id = SecureRandom.random_number(999999)
  session[:user_id] = @user.id

  c = Company.new
  c.id = SecureRandom.random_number(999999)
  c.name = @user.company_name
  c.email = "email goes here"
  c.users << @user.id
  c.save

  @user.company_id = c.id
  @user.save

  redirect_to '/dashboard'
else
  redirect_to '/'
end
end

Answer 1

If you're trying to build a one-to-many relationship, what you should do is add a company_id field to the users table, add belongs_to :company to your app/models/user.rb and add has_many :users to your app/models/company.rb .

Then you'd be able to access the users just like you need.

UPDATE

If you want to create a company with the user you can just go like this.

@user.company = Company.create name: 'Some name'

Or even better to avoid duplication.

@user.company = Company.find_or_create_by name: 'Some name'

Answer 2

I wonder if here you want to build many to many relationship, the best way to do this is using a many to many table instead of store all ids as an array string in databases;

If you do want to store like this, the best way should be form a string instead of put array directly. For this things like array.join() would be very helpful for you.

If you still insist your way, maybe you should try array.push() instead of <<