Rapter Jul 6 2015 at 11:33

Паттерны проектирования на Ruby

5 min

20K

Ruby*Perfect code*Ruby on Rails*

Comments 11

fuCtor Jul 6 2015 at 12:15

Явный вызов return не нужен, плюс если используете новый синтаксис для хэшей, то используйте его везде:

    def complect_task
     {
          id:          task.gid,
          title:       task.title,
          ts:          task.ts,
          author:      task.author_id,
          holder:       complect_holder
      }
    end

Но в последнем примере в полной мере не избавились от повторений, можно было использовать duck typing и сделать еще компактней.

fuCtor Jul 6 2015 at 12:29

  module CommonOp 
    def common(item)
	  {
          id:          item.gid,
          title:       item.title,
          ts:          item.ts,
          author:      item.author_id
      }
	end
  end
  
  class Event < Operation
    include CommonOp
    def complect_goal
	  common(goal).merge holder: complect_goal_content_header            
    end
  end

  class Task < Operation
    include CommonOp
    def complect_task
	  common(task).merge holder: complect_holder      
    end	
 end

Метод common можно поднять в базовый класс, и использовать его изначально

Rapter Jul 6 2015 at 12:28

Вы правы, сейчас поправлю под «старый» синтаксис. Насчет return, для себя я вынес следующее — если можно написать return, то лучше его написать, ошибок меньше будет.

-3

foxweb Jul 6 2015 at 12:40

Насчет return, для себя я вынес следующее — если можно написать return, то лучше его написать, ошибок меньше будет.

Н — наглядность.

le0pard Jul 6 2015 at 14:29

Жестяк. И это еще и action в controller. Не надо так

def edit
  @med_program = MedProgram.blabla
  redirect_to MedProgramService.new(@med_program).next_step_url
end

sl_bug Jul 6 2015 at 14:54

Я дико извиняюсь, но...

grayhat Jul 7 2015 at 00:05

Это создаёт информационный шум. Имеет смысл только не забывать возвращать nil в методах, которые должны возвращать nil.

grayhat Jul 6 2015 at 23:21

enum в ruby:

ProtocolElementTypeName = [:task, :note, :event].freeze

ProtocolElementTypeName.include?(type) or fail

grayhat Jul 6 2015 at 23:44

Ruby-метод для создания объекта нужного класса

def self.Complect type
  const_get("Complect::#{type.capitalize}").new
rescue NameError
  fail "Неизвестный тип"
end

module Complect
  class Event; end
  class QC; end
  class Tast; end
end

complect = Complect :event

grayhat Jul 6 2015 at 23:49

fixed:

# Ruby-метод для создания объекта нужного класса
def self.Operation type
  const_get("Operation::#{type.capitalize}").new
rescue NameError
  fail "Неизвестный тип"
end

class Operation
  class Event < self; end
  class Task < self; end
end
operation = Operation :event

QuickStudio Jul 7 2015 at 00:46

Интересно, спасибо

Show the best of all time