Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SakeControllerでSakeHelperのIncludeをやめる #594

Open
momocus opened this issue Mar 19, 2023 · 3 comments
Open

SakeControllerでSakeHelperのIncludeをやめる #594

momocus opened this issue Mar 19, 2023 · 3 comments
Assignees

Comments

@momocus
Copy link
Owner

momocus commented Mar 19, 2023

理想:
無駄が多いので、SakeControllerでSakeHelperをインクルードしたくない。

現状:
bottom_bottleメソッドを使うためにControllerでHelperをIncludeしている。

どうする:
bottom_bottleはSakeクラス(Sakeモデル)の定数として実装して、Sake::BOTTOM_BOTTLEでコントローラからもヘルパーからも参照できるようにする。
これにより、SakeControllerでSakeHelperをインクルードする必要がなくなる。

@momocus momocus self-assigned this Mar 23, 2023
@momocus
Copy link
Owner Author

momocus commented Mar 23, 2023

to_byもSakeControllerで使ってたわ

@momocus
Copy link
Owner Author

momocus commented Mar 27, 2023

after_initialize do |sake|
  sake.brewery_year ||= default_value
end

この書き方だとbrewery_year がnil(不明)の値を受け付けなくなってダメだった。
after_initialize がnew以外の時も走るからか?
https://qiita.com/supercell1023koeda/items/46aad5929491acd20108

@momocus
Copy link
Owner Author

momocus commented Mar 28, 2023

after_initialize(if: :new_record?) do |sake|
  sake.brewery_year ||= default_value
end

これでよかった。
(if: :new_record?)が大事

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant