An oversimplification of some rails best practices over time.

1_fat_controller.rb

class WidgetsController < ApplicationController
  def create
    @widget = Widget.new(params[:widget])
    
    if @widget.save
      SpamMailer.widget_spam(@widget).deliver
      redirect_to widget, notice: 'Widget created'
    else
      render 'new'
    end
  end
end

2_observer.rb

class WidgetsController < ApplicationController
  def create
    @widget = Widget.new(params[:widget])
  
    if @widget.save
      redirect_to widget, notice: 'Widget created'
    else
      render 'new'
    end
  end
end

class Widget < ActiveRecord::Base
  def spam_mailer
    SpamMailer.widget_spam self
  end
end

class WidgetObserver < ActiveRecord::Observer
  def after_create(widget)
    widget.spam_mailer.deliver
  end
end

3_service.rb

class WidgetsController < ApplicationController
  def create
    @widget_creator = WidgetCreator.new(params[:widget])
    @widget = @widget_creator.widget
  
    if @widget_creator.save
      redirect_to widget, notice: 'Widget created'
    else
      render 'new'
    end
  end
end

class Widget < ActiveRecord::Base
  def spam_mailer
    SpamMailer.widget_spam self
  end
end

class WidgetCreator
  attr_reader :widget
  def initialize(params)
    @widget = Widget.new(params)
  end
  
  def deliver_spam!
    widget.spam_mailer.deliver
  end
  
  def save
    result = widget.save
    deliver_spam! if result
    
    result
  end
end
    

:lol: