This sort of thing is tedious, ugly, and error-prone: <div class="highlight highli

I was wondering about something like: <div class="highlight highlight-source-ruby

??? <div class="highlight highlight-source-ruby notranslate position-relative over

Refactor the conditional counters to make it easier to write about counter HOT 2 CLOSED

ideasasylum commented on August 16, 2024

Refactor the conditional counters to make it easier to write

from counter.

Comments (2)

ideasasylum commented on August 16, 2024

I was wondering about something like:

on_create :increment, if: ->(student) { student.subscribed? && student.confirmed? }
on_delete :decrement, if: ->(student) { student.subscribed? && student.confirmed? }
on_update :increment, if: ->(student) { student.has_changed? :subscribed_at, from: nil, to: Counter::Any }
on_update :increment, if: ->(student) { student.has_changed? :confirmed, from: false, to: true }
on_update :decrement, if: ->(student) { student.has_changed? :subscribed_at, from: Counter::Any, to: nil }
on_update :increment, if: ->(student) { student.has_changed? :confirmed, from: true, to: false }

i.e., be explicit about increment/decrement so we don't need to return numbers, and allow multiple checks for update.

from counter.

ideasasylum commented on August 16, 2024

???

increment_on_create_if ->(student) { student.subscribed? && student.confirmed? }
decrement_on_delete_if, ->(student) { student.subscribed? && student.confirmed? }
increment_on_update_if ->(student) { student.has_changed? :subscribed_at, from: nil, to: Counter::Any }
increment_on_update_if ->(student) { student.has_changed? :confirmed, from: false, to: true }
decrement_on_update_if: ->(student) { student.has_changed? :subscribed_at, from: Counter::Any, to: nil }
decrement_on_update_if ->(student) { student.has_changed? :confirmed, from: true, to: false }

or maybe

on :create do
  increment_if ->(student) { student.subscribed? && student.confirmed? }
end

on :delete do
  decrement_if ->(student) { student.subscribed? && student.confirmed? }
end

on :update do
  increment_if ->(student) { student.has_changed? :subscribed_at, from: nil, to: Counter::Any }
  increment_if ->(student) { student.has_changed? :confirmed, from: false, to: true }
  decrement_if ->(student) { student.has_changed? :subscribed_at, from: Counter::Any, to: nil }
  decrement_if ->(student) { student.has_changed? :confirmed, from: true, to: false }
end