The best way to do with or without IF:

withorwithoutIF

This way with if

S.find(:all).each do |each|
  if each.expired?
     each.renew!
  else
     each.update!
  end
end


OR this one wotout if:

s = S.find :all
expired = s.select {|each| each.expired?}
expired.each {|each| each.renew!}
active = subscriptions.reject {|each| each.expired?}
active.each {|each| each.update!}

active, expired = S.find(:all).partition(&:active?)
active.each &:update!
expired.each &:renew!

first one or my comment above. But please please do not call the block parameter "each". Call it "e" (element) or something more semantic ("subscription")