Include all the ruby files from a directory inside a block as inline code

Question

Structure of ruby file is as shown below,

market
  |__abcd
       |__classification
              |__for_sale_mobile_phone.rb
              |__catalogues
                  |__mobile_phone_brands
                      |__acer.rb
                      |__apple.rb
                      |__samsung.rb

In for_sale_mobile_phone.rb, I like to include all brands of mobile_phone_brands under a block.

I m trying to include brands in below fashion,

 .....
    c.tree_field :brand, { child_key: :model } do |b|
      Dir[
        File.dirname(__FILE__) + '/catalogues/mobile_phone_brands/*.rb'
      ].each { |brand| load brand }

      b.required = true
    end
.....

here is how brand file looks like. for example, apple.rb

b.value "apple" do |brand|
  brand.value "6plus"
  brand.value "6s"
  brand.value "6s-plus"
  brand.value "se"
  brand.value "7"
  brand.value "7-plus"
  brand.value "other-model"
end

I m getting below error,

undefined local variable or method `b' on line 1: apple.rb

How can I include file under a block scope?

Thanks in advance!

Not sure I'm understanding this correctly. You want to include variables from other files as local variables in another file? — bkunzi01, Jul 26 '17 at 14:13
You could try using an instance variable for `@b` instead of b — Eric Himmelreich, Jul 26 '17 at 14:16

Vijay Agrawal · Answer 1 · 2017-07-26T15:47:56.140

You should separate file "loading" from function execution. Redefine your file like this.

class AppleLoader

  def self.load(b)
    b.value "apple" do |brand|
      brand.value "6plus"
      brand.value "6s"
      brand.value "6s-plus"
      brand.value "se"
      brand.value "7"
      brand.value "7-plus"
      brand.value "other-model"
    end
  end
end

On top of file you load required classes like this:

require '/catalogues/mobile_phone_brands/apple_loader.rb'

And when you want to load Apple brand in the b object:

AppleLoader.load b

Better Approach: To me it feels like Apple.rb defers from Samsung.rb only in terms of data. If that's the case, and functionality for both of them are same then I would rather:

put that data in a Yaml file(brands.yml) instead of rb file.

    brands:
     apple: ["6plus", "6s"]
     samsung: ["galaxy"]

Have just one common loader called BrandLoader

class BrandLoader

  def self.load(b, brand_name, values)
    b.value brand_name do |brand|
      brand_values.each do |value|
        brand.value value
      end
    end
  end
end

Iterate over brands by reading Yaml

configurations = YAML.load "brands.yml"
configurations["brands"].each do |brand_name, values|
  BrandLoader.load(b, brand_name, values)
end

Agarwal: Can give upvote for your attempt, Which works perfectly but solution below I figured out make it simple — Arun G, Jul 27 '17 at 08:29
I had a look at it, though it works but not a recommended way of doing things in ruby. When you have performance and maintainability concerns, you should switch — Vijay Agrawal, Jul 27 '17 at 15:40

Arun G · Answer 2 · 2017-07-28T05:45:20.737

The way I figured out is by using eval but cautious read up is-eval-supposed-to-be-nasty, eval works for me as I'm not using user input.

To make any code in different file, execute as the code written in the place of call. Use eval(File.read(file_path)

 .....
    c.tree_field :brand, { child_key: :model } do |b|
      Dir[
        File.dirname(__FILE__) + '/catalogues/mobile_phone_brands/*.rb'
      ].each { |brand| eval(File.read(brand)) }

      b.required = true
    end
.....

Include all the ruby files from a directory inside a block as inline code

2 Answers2