ruby on rails - loadError expected to define

Question

I have a problem for hours and hours but I can't find the solution. I find some similar questions to mine, but still no luck. I have a method in my controller :

def get_game_site_log_id(game)
  Game_site_log.connection.select_values("SELECT id FROM game_site_logs 
  WHERE game_id = " + game["game_id"] + "AND site_id = " + @site.id + ";")
end

I got LoadError expected game_site_log.rb to define Game_site_log. I tried to add this :

require File.dirname(__FILE__) + "game_site_log.rb"

But instead of loading the model from game_site_log.rb, it tried to find that file in controller. What is the best solution to this? Thanks :)

samuil · Accepted Answer · 2013-06-20 11:03:04Z

2

Ruby convention is to name classes in CamelCase, so your model should be named GameSiteLog.
As RoR is heavily using convention over configuration, to get everything running easily you should put game_site_log.rb file in app/models.

answered Jun 20, 2013 at 11:03

samuil

5,0911 gold badge39 silver badges44 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

thomasfedb · Accepted Answer · 2013-06-20 11:15:25Z

1

If your model is in app/models/game_site_log.rb then, by Rails conventions, you should define it as:

class GameSiteLog < ActiveRecord::Base
  # methods here
end

Then you can access it with GameSiteLog.connection.select_values().

Class names should be CamelCased and file names should be under_scored.

edited Jun 20, 2013 at 11:15

thomasfedb

5,9832 gold badges40 silver badges65 bronze badges

answered Jun 20, 2013 at 11:09

Giorgio Previtera

3931 silver badge6 bronze badges

Collectives™ on Stack Overflow

ruby on rails - loadError expected to define

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related