Question

I’m adding a categorization functionality to my app and struggling with it. Objects have many categories through categorizations. I’m trying to intercept the creation of a new categorization, check if theres a similar one, if so, increment it’s count, if not, create a new object. Here’s what I have so far.

  validate :check_unique

  protected

  def check_unique
    categorization = Categorization.where(:category_id => self.category_id, :categorizable_id => self.categorizable_id, :categorizable_type => self.categorizable_type)
    if categorization.first
      categorization.first.increment(:count)
    end
  end

Answer 1

This kind of logic should not exist in the controller. This is really business domain and should be in the model. Here’s how you should go about it:

categorization = Categorization.find_or_create_by_category_id_and_categorizable_id_and_categorizable_type(self.category_id, self.categorizable_id, self.categorizable_type)
categorization.increment!(:count)

find_or_create will try to find the category in the DB, and if it doesn’t exist, it’ll create it. Now just make sure that count defaults to zero, and this code will do what you want. (when initially created the count would be 1, then later it’ll increment)

PS: I’m not sure if find_or_create has changed in rails 3. But this is the main idea