Question

I’m building an application that checks when a user needs to change the oil of one of his cars. Here’s how it works:

1. The user enters the name of his car (e.g. ‘Ford Mustang’)
2. The user enters the time the car will run on one oil change (e.g. 3.months)
3. The user saves the car to the database
4. Repeat above process for all his other cars. (Yes, it’s a wealthy user 😉

Now, when the user comes back to the site he wants to see a list of all his cars that are in need of an oil change. He then changes the oil and updates the ‘oil_changed_at’ property.

I’m using the following columns for the Car-model:

title:string
oil_changed_at:datetime
oil_expiration_timeframe:integer

My question: What is the preferred way to store this ‘oil_expiration_timeframe’ and what should my :conditions look like when I want to get a list of all cars that are in need of an oil change?

Keep in mind the oil_expiration_timeframe may vary from a few days to a few years. It will never be less than a day so I think I can just store the amount of days in that field. However, I’m not sure how to combine the integer and datetime in the query.

Answer 1

If you index your oil_changed_at column, following approach is efficient.

Car.all(:conditions => [
  "oil_changed_at > DATE_SUB(?, INTERVAL oil_expiration_timeframe day)", Time.now
])

Better approach would be to add a new column(next_oil_change_date) to Car model to store the next oil change date and update the col upon changes to the Car model.

class Car < ActiveRecord::Base
  before_save :set_next_oil_change_date

  def set_next_oil_change_date
    self.next_oil_change_date = next_oil_change_date + 
                                  oil_expiration_timeframe.days
  end
end

Now you can write your query as:

Car.all(:conditions => [
  "next_oil_change_date > ? ", Time.now
])

Don’t forget to index the next_oil_change_date column.