DEV Community

Cover image for SOLID Principles in Ruby on Rails
Shamila TP
Shamila TP

Posted on

SOLID Principles in Ruby on Rails

SOLID Principles in Ruby on Rails

SOLID is not a Rails or Ruby concept. It's a set of five object-oriented design principles that apply to any language/framework.

But Rails makes it surprisingly easy to violate all five of them.

Let's start with


Single Responsibility Principle

It states a class should have only one reason to change.

A classic example is a User model that handles authentication, sends emails, and formats reports. That's three responsibilities. Instead, you'd extract email sending into a Mailer class and reporting into a service object.

class User
  def authenticate
  end

  def send_welcome_email
  end

  def generate_report
  end
end
Enter fullscreen mode Exit fullscreen mode

The Fix

class User < ApplicationRecord
  has_many :orders

  validates :email, presence: true
end

class UserAuthenticationService
  def authenticate
  end
end

class UserMailer < ApplicationMailer
  def send_welcome_email(user)
  end
end

class UserReportService
  def generate_report
  end
end
Enter fullscreen mode Exit fullscreen mode

In Rails, the most common places SRP breaks down are: fat models stuffed with business logic, controllers that do too much, and callbacks that silently trigger side effects.


Open/Closed Principle

A class should be open for extension but closed for modification.

In practice this means: when you need new behaviour, add new code — don't change existing code. This protects stable, tested behaviour from being accidentally broken by new requirements.

class IssueTicketProcessor
  def report_ticket(ticket_type, ticket)
    case ticket_type
    when :hotsos
      p "Processing hotsos ticket : #{ticket}"
    when :know_cross
      p "Processing know_cross ticket : #{ticket}"
    end
  end
end

IssueTicketProcessor.new.report_ticket(:hotsos, "ac not working")
Enter fullscreen mode Exit fullscreen mode

Every time a new issue ticket type comes, this class has to be changed.

The Fix

class TicketIssuer
  def process(ticket)
    raise NotImplementedError
  end
end

class HOTSOSTicketing < TicketIssuer
  def process(ticket)
    puts "Handling HOTSOS ticket: #{ticket}"
  end
end

class KNOWCROSSTicketing < TicketIssuer
  def process(ticket)
    puts "Handling KNOWCROSS ticket: #{ticket}"
  end
end

class IssueTicketProcessor
  def report_ticket(ticket_issuer, ticket)
    ticket_issuer.process(ticket)
  end
end

IssueTicketProcessor.new.report_ticket(
  HOTSOSTicketing.new,
  "TV remote not working"
)
Enter fullscreen mode Exit fullscreen mode

Suppose tomorrow we need to add HoteSoft — no need to change IssueTicketProcessor:

class HoteSoftTicketing < TicketIssuer
  def process(ticket)
    puts "Handling HoteSoft ticket: #{ticket}"
  end
end
Enter fullscreen mode Exit fullscreen mode

Liskov Substitution Principle

Objects of a superclass should be replaceable with objects of its subclasses without breaking the application.

This one's about subclasses being replaceable for their base classes without breaking things.

The Problem

class PaymentMethod
  def process(amount)
    raise NotImplementedError
  end
end

class PaypalPayment < PaymentMethod
  def process(amount)
    p "processing paypal payment"
  end
end

class CreditCardPayment < PaymentMethod
  def process(amount, details)       # ← different signature! violates LSP
    p "processing creditcard payment"
  end
end

class PaymentProcessor
  def process(payment_method, amount)
    payment_method.process(amount)
  end
end

payment_method = CreditCardPayment.new
PaymentProcessor.new.process(payment_method, '100')
# Will raise error — CreditCardPayment#process expects 2 arguments
Enter fullscreen mode Exit fullscreen mode

LSP means a subclass should honor the contract of its parent. The fix is to match the parent's method signature:

class CreditCardPayment < PaymentMethod
  def process(amount)
    p "processing creditcard payment"
  end
end
Enter fullscreen mode Exit fullscreen mode

Interface Segregation Principle

No class should be forced to depend on methods it does not use.

Rails-ish Example

Bad

class PaymentGateway
  def charge; end
  def refund; end
  def create_subscription; end
end
Enter fullscreen mode Exit fullscreen mode

Now CashPaymentGateway < PaymentGateway only supports charge, but the parent contract forces it to also carry refund and create_subscription.

The Fix — split into focused modules

module Chargeable
  def charge(amount); end
end

module Refundable
  def refund(amount); end
end

module Subscribable
  def create_subscription(plan); end
end
Enter fullscreen mode Exit fullscreen mode

Since Cash only supports charging, include only Chargeable:

class CashPaymentGateway
  include Chargeable

  def charge(amount)
    puts "Cash payment #{amount}"
  end
end
Enter fullscreen mode Exit fullscreen mode

Dependency Inversion Principle

High-level modules should not depend on low-level modules. Both should depend on abstractions.

Before DIP

class UserRegistration
  def register
    GmailService.new.send_email
  end
end
Enter fullscreen mode Exit fullscreen mode

UserRegistration directly depends on GmailService. If Gmail is replaced with SendGrid, we must modify UserRegistration. This creates tight coupling.

After DIP

class EmailService
  def send_email
    raise NotImplementedError
  end
end

class GmailService < EmailService
  def send_email
    puts "Sending mail via Gmail"
  end
end

class SendGridService < EmailService
  def send_email
    puts "Sending mail via SendGrid"
  end
end

class UserRegistration
  def register(email_service)
    email_service.send_email
  end
end

# Usage:
UserRegistration.new.register(GmailService.new)
Enter fullscreen mode Exit fullscreen mode

Now there's loose coupling and easier testing.

Before DIP

UserRegistration --> GmailService
Enter fullscreen mode Exit fullscreen mode

After DIP

UserRegistration --> EmailService
                         ^
                         |
          ----------------------------
          |                          |
          v                          v
    GmailService            SendGridService
Enter fullscreen mode Exit fullscreen mode

Putting it all together

Principle Meaning
SRP One class, one responsibility.
OCP Extend behavior without modifying existing code.
LSP Child classes must honor the parent contract.
ISP Keep interfaces small and focused.
DIP High-level modules should depend on abstractions.

In a Rails production codebase, here's how they map to common pain points:

  • Models growing past 300 lines → SRP
  • Adding a new feature requires touching existing classes → OCP
  • Subclass behaviour is surprising or inconsistent → LSP
  • Including a concern drags in methods you don't need → ISP
  • Tests require heavy mocking of third-party services → DIP

That's the kind of codebase that scales!

Top comments (0)