質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
Ruby on Rails 5

Ruby on Rails 5は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

2回答

1458閲覧

moduleに定義したクラスメソッドが呼び出されません

shumbow

総合スコア35

Ruby on Rails 5

Ruby on Rails 5は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2019/11/23 02:50

編集2019/11/23 07:18

modelで共通のメソッドが欲しくて、インスタンス化したくないのでクラスメソッドとして呼び出したいのですが、undefined method になってしまいます。
model/concerns/common_mehod.rb

module CommonMethod extend ActiveSupport::Concern def self.set_factors_of(str, num) result = [] unless str.blank? add_str = "_" + str.to_s else add_str = "" end num.times do |n| result << "factor#{n + 1}" + add_str end return result end end

model/item.rb
この中のインスタンスメソッドの中で呼び出しています

include CommonMethod def set_all_avgs @article ||= self.article props = [:total_pt] props << CommonMethod.set_factors_of("avg", @article.factors_num) ⬅️これ props.each do |prop| result = self.votes.average(prop) self.send(prop.to_s.gsub(/pt/, "avg="), result) end end

コンソール

irb(main):002:0> item.set_all_avgs Traceback (most recent call last): 2: from (irb):2 1: from app/models/item.rb:25:in `set_all_avgs' NoMethodError (undefined method `set_factors_of' for CommonMethod:Module)

インスタンスメソッドとして定義すればいけます。

module CommonMethod extend ActiveSupport::Concern def set_factors_of(str, num) ##変更 result = [] unless str.blank? add_str = "_" + str.to_s else add_str = "" end num.times do |n| result << "factor#{n + 1}" + add_str end return result end end def set_all_avgs @article ||= self.article props = [:total_pt] props << self.set_factors_of("avg", @article.factors_num) props.each do |prop| result = self.votes.average(prop) self.send(prop.to_s.gsub(/pt/, "avg="), result) end end

追記
名前のつけ方が間違っていたのか名前を変えたら動くようになりました。
以前はファイル名common_method
モジュール名 CommonMethod
include CommonMethod
でした。それを以下のようにしましたら動きました。名前のつけ方何かまずかったでしょうか?

module Common extend ActiveSupport::Concern def hello puts "hello, world!" end module ClassMethods def set_factors_of(str, num) result = [] unless str.blank? add_str = "_" + str.to_s else add_str = "" end num.times do |n| result << "factor#{n + 1}" + add_str end return result end end end

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

2KOH

2019/11/23 03:30

common_mehod.rb を require し忘れていた、という可能性はないですか?
shumbow

2019/11/23 06:04

includeしているので問題ないかなと思っています。
guest

回答2

0

(出先からなので簡潔になってしまいますが、少し残していきます)

concernsでクラスメソッドを作りたい場合、専用の書き方があります。そのままモジュールのクラスメソッドとしても、組み込んだモデルからは使えません。

投稿2019/11/23 05:34

maisumakun

総合スコア145183

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

shumbow

2019/11/23 06:53

調べてやってみます。ありがとうございます。
guest

0

自己解決

現状はActiveSupportConcernに頼らずに普通にモジュールを書けば auto loadされてコンソールなどでも使えるので、この方向でいこうと思います。名前については謎ですが。

投稿2019/11/23 02:53

編集2019/11/23 07:29
shumbow

総合スコア35

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問