I would say that by using the module Date::Calc::Iterator you are more efficient as you iterate through the set of dates you want and not make a list of all dates and then discard most of them.

Also you don't have to pollute your database with a table of all dates in the range you want.