概要

便利な機能であっても、少なからずデメリットも発生することがあると思います。

なので使う側はメリット・デメリットのバランスを判断して、適切に実装を行うことができるようになる必要があるのでは？
と思い、記事にまとめてみました。

実際に保守しづらさのほうが勝っているケースがあり、改めて必要性があるのか？を検討するための参考になればと思います

まずattr_*を使うとどうなるか？

attr_reader :name とすることで引数の名称でgetterメソッドが定義されます

def name
  @name
end

attr_writer :name とすることで引数の名称でsetterメソッドが定義されます

def name=(val)
  @name = val
end

attr_accessor :name はreaderとwriterの複合メソッドです

通常、インスタンス変数はインスタンスの内部にて @hoge でアクセスできるものがpublicでメソッドが定義されるため外部からインスタンス変数の値を触ることができるようになります。

この時privateを使用することによってprivateでメソッドを定義させることも可能です。

private
attr_reader :hoge

メリットとデメリットについて

メリット

外部からの呼び出し

本来の用途である、publicメソッドとして切られるため外部からの使用ができます
※privateにもできます

ここは皆さんご存知のとおりでしょう。

改修のしやすさ

class Hoge
  private
  attr_accessor :hoge

  def initialize(hoge:)
    @hoge = hoge
  end

  def connect
    Service.call(hoge: hoge) # <- ココ
  end
end

このような実装になっているとき、

def hoge
  @hoge + 1
end

のように改修する必要が出たときにオーバーライドすることで修正はしやすくなります。

デメリット

スコープの意識

publicかprivateかなどを正しく考える必要がでる

メソッドの定義

メソッドを不用意に定義することで、メソッド名が競合してしまう可能性

可読性

オーバーライドできるということは、メソッドがオーバーライドされていないか？を確認する必要がでる => 可読性が下がる

例

class Hoge
  include Hoge::XXXable
  include Hoge::ZZZable

  private
  attr_accessor :hoge

  def initialize(hoge:)
    @hoge = hoge
  end

  def connect
    Service.call(param: param)
  end
end

module Hoge::XXXable
  include ::YYYable
  def param
    {
      hoge: hoge # <- ココ
    }
  end
end

このように対象のクラスにincludeされる前提のモジュール（切り出しモジュール）に対して
インスタンス変数を参照しているだけだがメソッド呼び出しにできます

これが

module Hoge::XXXable
  include ::YYYable
  def param
    {
      hoge: @hoge # <- ココ
    }
  end
end

となっていれば「includeして読み込まれるクラスのインスタンス変数をセットしている」とモジュール内のソースを見るだけでパッとわかります。
※インスタンス変数をほかのクラスやモジュールで直接触るようなものは書きたくない　という意味であれば気持ちは分かりますが。

しかし、これがattrのgetterメソッドとなっていることによって、XXXableのモジュールを見るだけだと、hogeは一体どこで定義されているメソッドなのか？が分かりません。
そのため下記のどこか？を探す作業が発生してしまいます。

XXXable内で定義されているか
XXXableにてincludeやextendされているモジュールの中
XXXableをincludeしている側のクラスやモジュール
XXXableをincludeしている側のクラスやモジュールが読み込んでいるモジュール

結論

メリットで挙げた、保守がしやすくなる というのは上書きが簡単という意味では当てはまりますが、
その必要性が現段階で想定できないようなケースであれば、逆に 保守がしにくく なっていないでしょうか？

使う際にはこうしたデメリットが大きくなるケースではないか？を今一度考えながら使用するかどうかを検討していき
よりよいプログラムを書いていきたいなと思います