Инициализация объекта DRY'er в Ruby

Question

Есть ли более "сухой" способ сделать следующее в ruby?

#!/usr/bin/env ruby

class Volume
    attr_accessor :name, :size, :type, :owner, :date_created, :date_modified, :iscsi_target, :iscsi_portal

    SYSTEM = 0
    DATA = 1

    def initialize(args={:type => SYSTEM})
      @name = args[:name]
      @size = args[:size]
      @type = args[:type]
      @owner = args[:owner]
      @iscsi_target = args[:iscsi_target]
      @iscsi_portal = args[:iscsi_portal]
    end

    def inspect
      return {:name => @name,
              :size => @size,
              :type => @type,
              :owner => @owner,
              :date_created => @date_created,
              :date_modified => @date_modified,
              :iscsi_target => @iscsi_target,
              :iscsi_portal => @iscsi_portal }
    end

    def to_json
      self.inspect.to_json
    end

конец

Answer 1

Всякий раз, когда вы видите длинный список подобных вещей, обычно вы можете свернуть все это в отдельный массив:

class Volume
  ATTRIBUTES = [
    :name, :size, :type, :owner, :date_created, :date_modified,
    :iscsi_target, :iscsi_portal
  ].freeze

  ATTRIBUTES.each do |attr|
    attr_accessor attr
  end

  SYSTEM = 0
  DATA = 1

  DEFAULTS = {
    :type => SYSTEM
  }.freeze

  def initialize(args = nil)
    # EDIT
    # args = args ? DEFAULTS : DEFAULTS.merge(args) # Original
    args = args ? DEFAULTS.merge(args) : DEFAULTS

    ATTRIBUTES.each do |attr|
      if (args.key?(attr))
        instance_variable_set("@#{attr}", args[attr])
      end
    end
  end

  def inspect
    ATTRIBUTES.inject({ }) do |h, attr|
      h[attr] = instance_variable_get("@#{attr}")
      h
    end
  end

  def to_json
    self.inspect.to_json
  end
end

Управлять переменными экземпляра довольно просто после этого.

Answer 2

 class Volume

  FIELDS = %w( name size type owner iscsi_target iscsi_portal date_create date_modified)
  SYSTEM = 0
  DATA = 1
  attr_accessor *FIELDS

  def initialize( args= { :type => SYSTEM } )
    args.each_pair do | key, value |
      self.send("#{key}=", value) if self.respond_to?("#{key}=")
    end
  end

  def inspect
    FIELDS.inject({}) do | hash, field |
      hash.merge( field.to_sym => self.send(field) )
    end.inspect
  end

 end

Answer 3

Вот немного другой ответ на вопрос Тадмана:

class Volume
  ATTRIBUTES = [
    :name, :size, :type, :owner, :date_created, :date_modified,
    :iscsi_target, :iscsi_portal
  ].freeze

  ATTRIBUTES.each do |attr|
    attr_accessor attr
  end

  SYSTEM = 0
  DATA = 1

  DEFAULTS = {
    :type => SYSTEM
  }.freeze

  def initialize(&block)
    ATTRIBUTES.each do |attr|
      self.__send__ "#{attr}=", DEFAULTS[attr]
    end
    yield(self)
  end

  def inspect
    ATTRIBUTES.inject({}) { |h,attr| h[attr] = self.__send__ attr; h }
  end
  def to_json
    self.inspect.to_json
  end
end

Это позволяет сделать это:

vol = Volume.new do |v|
  v.name = 'myVolume'
end

Мне это нравится, потому что преимущество заключается в том, что вы сразу же получаете ошибки, если кто-то сделал опечатку для атрибута.

Кроме того, в отличие от его ответа, он инициализирует значения по умолчанию, когда они не поставляются.

или если вы в конечном итоге делаете это много и действительно нуждаетесь в СУХОЙ:

module Attributable
  @@ATTRIBUTES = []
  @@DEFAULTS = {}

  def initialize(&block)
    @@ATTRIBUTES.each do |attr|
      self.class.__send__ :attr_accessor, attr
      self.__send__ "#{attr}=", @@DEFAULTS[attr]
    end
    yield(self)
  end
end

class Volume
  include Attributable
  @@ATTRIBUTES = [ :name, :size, :type, :owner ]
  @@DEFAULTS = { :type => 0 }
end

не мог понять, как это сделать с константами, поэтому я сделал это с переменными класса.

Answer 4

Рассеяние Тэдман х ответ

Я бы #inspect вернул String (как большинство #inspect методов) и, возможно, фактор вместо преобразования в метод хеширования в метод #to_hash.

Ерунда args.merge(DEFAULTS).merge(args) позволяет args переопределять DEFAULTS, но сохраняет любое поведение по умолчанию для args (скажем, если args == Hash.new(3) или args == Hash.new { |h,k| h[k] = h.to_s.length }

class Volume
  ATTRIBUTES = %w{
    name size type owner date_created date_modified
    iscsi_target iscsi_portal
  }.map! { |s| s.to_sym }.freeze

  attr_accessor *ATTRIBUTES

  SYSTEM = 0
  DATA = 1

  DEFAULTS = { :type => SYSTEM }.freeze

  def initialize(args = nil)
    args = args ? args.merge(DEFAULTS).merge(args) : DEFAULTS

    ATTRIBUTES.each do |attr|
      instance_variable_set("@#{attr}", args[attr])
    end
  end

  def to_hash
    Hash[ *ATTRIBUTES.map { |attr| [ attr, instance_variable_get("@#{attr}") ] }.flatten ]
  end

  def inspect
    to_hash.inspect
  end

  def to_json
    self.to_hash.to_json
  end
end