既存メソッドのオーバライドはどのファイルに書けば良い？

Question

###前提・実現したいこと
to_jsonで出力されるデータのフォーマットを変更したいと考えてます。(後記のサイトを参考にしてます)

うまくいかないのは、オーバーライドするメソッドを記載するファイルが間違っている気がします。

config/application.rb
app/application_controller.rb

を試しました。
複数のモデルがあるので、共通してオーバライドを適用したいと考えてます。

###発生している問題・エラーメッセージ
変更されない（オーバライドしている気配がない）

###該当のソースコード

module AppName
  class Application < Rails::Application
    # Settings in config/environments/* take precedence over those specified here.
    # Application configuration should go into files in config/initializers
    # -- all .rb files in that directory are automatically loaded.
    config.time_zone = 'Tokyo'
  end

  class Time
    def to_json(*a)
      strftime('%Y年%m月%日').to_json(*a)
    end
  end
end

###試したこと
puts Model.last.to_json
puts JSON.generate(JSON.parse(Model.last.to_json))（こっちは変更されないのが当たり前だと思いますが、念のため）

いずれも、
"created_at":"2017-10-14T13:30:40.750+09:00"
と、フォーマットが変更されませんでした。

###補足情報(言語/FW/ツール等のバージョンなど)
出力されるJSONの形式をカスタマイズする

pry(main)> .ruby --version
ruby 2.4.1p111 (2017-03-22 revision 58053) [x86_64-linux-musl]
pry(main)> .rails --version
Rails 5.0.6

Answer 1

module AppNameの中に入れてしまっているため、class Timeは、実際にはAppName::Timeというクラスを新規作成する扱いとなってしまいます。グローバルにclass Timeとかく必要があります。

Railsの場合、そのような既存クラスなどの調整はconfig/initializer/*.rbに置くのが一般的です。

なお、既存のクラスを書き換えると副作用がある可能性がありますので、入念なチェックが必要です（特に、APIにJSONを投げる場合などは要注意です）。

Comments

[NCC1701]

ありがとうございました。方針を変更することにしました。
1. initializer/で書いたメソッドがオーバーライドしなかった
2. アドバイスの副作用（時間が経つとオーバライドしたことを忘れそう）を考慮して、フォーマット変更のメソッドを明示する
3. as_jsonで一旦処理できること
という理由です。