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

Есть ли более "СУХОЙ" способ сделать следующее в 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

конец

Ответы

Ответ 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

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

Ответ 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

Ответ 3

Отмена tadman ответ

Я бы #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

Ответ 4

Здесь немного отличается от ответа тадманом:

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

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