CODEWITHSUNDEEP
ruby-on-railsactiverecord
user230527
user230527

Reputation:

Rails model: "has many" *simple* attribute

Let's assume this model:

Movie
- Title: String
- Has many:
    - Alternative Title: String

My questions is, how should I store the alt. title attribute? I am deciding between three approaches:

The latter two seems logically equivilent. I am leaning towards the CSV approach. Can anyone give some advise on this? What would be the implications on speed and searchability?

Upvotes: 0

Views: 357

Answers (3)

andykram
andykram

Reputation: 140

I would also vote for a separate model even though it seems like overkill it will allow you to basically follow the Rails way the easiest. However, if you choose not to reap the benefits of all the baked in magic associated with associations, then I would recommend YAML or JSON over CSV. CSV is quite simple, but Rails has baked in support for YAML serialization and would probably be the easiest solution. Check out RDoc on #serialize. For the given example this would basically amount to:

class Movie < ActiveRecord::Base
  serialize :alternate_titles
end

With that, Rails would handle a lot of the drudgery for you and you'll have a nice array of alternate titles always available.

Upvotes: 0

amitkaz
amitkaz

Reputation: 2712

If most of the time you only use the primary title, I'll go with your CSV option.

If most of the time you use all the titles, I'll put all the titles (primary and secondary) inside a single CSV column (named "titles") and just get the first when the primary is needed (with a helper function).

Why? Because it makes things simple- and if the time has come, like Jordan said, that you need another attribute you can always migrate to a separate model.

Until then, YAGNI.

Upvotes: 0

Jordan Running
Jordan Running

Reputation: 106027

If a movie can have many titles, it makes most sense to have a Title model and give the Movie model a has_many :titles relation, especially if you later on decide to add more metadata about titles. It may seem like overkill, but I think it will be the least hassle in the long run. Furthermore, I think that a movie's "main" title should be a Title object as well, perhaps with an is_main_title or similar attribute to distinguish it from the others.

Upvotes: 2

Related Questions