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

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

ただいまの
回答率

89.54%

railsでモデルをコントローラーから呼び出す

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 260

soft_admin

score 57

railsを勉強中です。
モデルをコントローラーから呼び出す方法を教えてください。
コントローラーでモデルのファイルをrequireすればよいのかと思ったのですが、
NoMethodErrorが発生します。

・コントローラー
index_controller.rb

require "user.rb"

class HomesController < ApplicationController

  def index
    @db_user = User.user_info
    render "homes/index"

  end

end

・モデル
user.rb

class User < ApplicationRecord

  def user_info
    db_user = "ユーザー名"
    return db_user
  end

end

追記:下記のように修正しました。NoMethodErrorがでます。

・コントローラー
index_controller.rb

class HomesController < ApplicationController

  def index
    @db_user = User.user_info
    render "homes/index"
  end

end

・モデル
user.rb

class User < ApplicationRecord

  def user_info
    user = User.find(1)
    info = user.name
    return info
  end

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • asm

    2019/08/13 08:45

    user_infoメソッドについて、単純に"ユーザー名"という文字列を返したいのでしょうか?
    それとも、データベースに登録されている情報を用いた何かを返したいのでしょうか?

    キャンセル

  • soft_admin

    2019/08/13 08:49

    データベースに登録されている情報を返すように実装するつもりですが、とりあえずシンプルに値を返すだけの処理を書いてみました。

    キャンセル

回答 2

checkベストアンサー

+1

そもそものエラーの原因の話ですが

class User < ApplicationRecord
  def user_info
    user = User.find(1)
    info = user.name
    return info
  end
end


ここで定義されているuser_infoはインスタンスメソッドです。一方、

class HomesController < ApplicationController
  def index
    @db_user = User.user_info
    render "homes/index"
  end
end


ここで使われているuser_infoはクラスメソッドです。

user_infoクラスメソッドが定義されていないのでNoMethodErrorが発生しています。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/13 19:43

    おっしゃる通りでした。
    ありがとうございました。

    キャンセル

0

まず、

require "user.rb"

については、必要ありません。
すでにロードされています。

データベースをfindfind_by where等で検索し目当てのレコードを取り出す必要があります。

例えば

# idが1のレコード 無い場合は例外が飛び 404になる
user = User.find(1)

# nameカラムがhogeのレコードの先頭
# find_byだと無い場合はnilが返り
# find_by!だと無い場合は例外が飛び 404になる
user = User.find_by!(name: "hoge")

# params[:id]をもちいてURLやPOSTされたパラメータから取り出す事が多い
user = User.find(params[:id])

info = user.user_info
# 全てのレコード
users = User.all

# nameカラムがhogeのレコード 全て
users = User.where(name: "hoge")

# ageカラムが10..18のどれか
users = User.where(age: 10..18)

# ageカラムが18より上
users = User.where("age > ?", 18)

# allやwhereで取り出した場合はRelationという配列のような検索条件的な物が返り
# eachなどで各々のレコードを呼び出せる
infos = users.map{|user| user.user_info }

以下追記

普通、データベースには以下のように複数のレコードが保存されます。

例: ユーザー毎のテストの得点

id name score
1 hoge 100
2 foo 50
3 bar 85

これに対して、「コントローラーからモデルを呼び出したい」ときというのは大別して2パターン考えられます。
a. 全てのユーザーの平均点が欲しい
b. ある1人のユーザーの得点が欲しい

データベースに入ってる得点が実際の点数の10分の1だったとします。
(メソッドを作る理由を無理やり作っただけです。)

a. 平均点を計算するメソッドが欲しい場合
注意: 大抵のものはRailsが用意しています。 Rails tips: ActiveRecord count系機能の基本と応用(翻訳)

# user.rb
class User < ApplicationRecord
  # クラスメソッド
  def self.my_ave
    User.average(:score) * 10.0
  end
end

# controller.rb
  def index
    @average = User.my_ave
  end

b. ある1人のユーザーの得点が欲しい

# user.rb
class User < ApplicationRecord
  # インスタンスメソッド
  def get_score
    score * 10
  end
end

# users_controller.rb
class UsersController < ApplicationController
  def show
    user = User.find(params[:id])
    @score = user.get_score
  end
end

# params[:id]は routes.rbに
# get '/users/:id', to: 'users#show'
# や
# resources :users
# と
# 書いてある等の場合につかえます。
# そうでない場合は、なんらかの方法でデータベースを検索させる必要があります。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/13 10:03

    すみません...上の回答だとどこに何を書けば良いのかが分からないです。
    info = user.user_info
    は何が入ってくるのでしょうか?

    キャンセル

  • 2019/08/13 10:15

    コントローラーから ということなので
    コントローラーに記載するものを提示しています

    キャンセル

  • 2019/08/13 11:44

    DBからデータを取得するロジックは、モデルに書くべきだと思っていたのですが、そうではないのですか?
    コントローラーからモデル内のDB取得をする関数を呼び出す、というイメージだったのですが。

    キャンセル

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

  • ただいまの回答率 89.54%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる