하나의 방법까지 두 가지 방법에서이 코드를 리팩토링하는 방법

Question

나는 매우 유사 다음 두 가지 방법이 있습니다

  def space_before_element?(start_element) 
       element = start_element.previous_element 
       until element.nil? || 
       (element.name == start_element.name || "r" && !element.text.empty?) 
        element = element.previous_element 
       end 
       character = element 
          .text 
          .split(/(\W)/) 
          .compact 
          .reject(&:empty?) 
          .last 
          .last_character \ 
          unless element.nil? 

       element.nil? || 
       (character.punctuation? && !character.hyphen? && !character.apostrophe?) || 
       character.spaces? 
       end 


      def space_after_element?(start_element) 
       element = start_element.next_element 
       until element.nil? || 
       (element.name == start_element.name || "r" && !element.text.empty?) 
        element = element.next_element 
       end 
       character = element 
          .text 
          .split(/(\W)/) 
          .compact 
          .reject(&:empty?) 
          .last 
          .first_character \ 
          unless element.nil? 

       element.nil? || 
       (character.punctuation? && !character.hyphen? && !character.apostrophe?) || 
       character.spaces? 
       end 

내가 허용 할 필요한 사항을 변경하는 방법을 알아낼 수없는 것을 나 한 가지 방법으로 이것을 줄이기 위해.

아직 내 루비 기술을 계속 사용하고 있습니다.

도움을 주시면 감사하겠습니다. 루비 2.2.3

Answer 1

세 가지 차이가 ​​있습니다. 따라서, 이러한 차이는 명시 적으로 선언해야합니다

METHODS = { 
    prev: [:previous_element, %i|last last_character|], 
    next: [:next_element, %i|first first_character|] 
}.freeze 

지금 우리는 단지 메서드에 매개 변수를 전달 :

def space_around_element?(start_element, prev_or_next = :prev) 
    element = start_element.public_send(METHODS[prev_or_next].first) 
    # same code 
    character = element 
       .text 
       .split(/(\W)/) 
       .reject(&:empty?) 
       .public_send(METHODS[prev_or_next].last.first) 
       .public_send(METHODS[prev_or_next].last.last) \ 
     unless element.nil? 
    # same code 
    end