2016-10-08 2 views
1

Я хотел бы высушить этот код. Кажется, я не могу понять, как это сделать.Как высушить этот код ruby ​​

def get_all_verb_nodes 
    all_verb_nodes = @all_nodes_ins_del_nodes.select { |node| 
    previous_node = node.xpath('preceding-sibling::w:r').last 
    base_word = previous_node.text.split.last.strip.delete('.!?,:') if previous_node 
    words = get_node_text(node) 
    next_node = node.next_element 
    next_node_is_insert_or_delete = is_insert_or_delete?(next_node.name) if next_node 
    next_node_word = next_node.text.strip if next_node 
    words.length <= 2 && words.any? { |word| is_a_verb?(base_word+word) || is_a_verb?(word) && !is_pluralized?(base_word+word, base_word+next_node_word) } && !next_node_is_insert_or_delete 
    } 
end 


def get_all_article_nodes 
    all_article_nodes = @all_nodes_ins_del_nodes.select { |node| 
    previous_node = node.xpath('preceding-sibling::w:r').last 
    base_word = previous_node.text.split.last.strip.delete('.!?,:') if previous_node 
    words = get_node_text(node) 
    next_node = node.next_element 
    next_node_is_insert_or_delete = is_insert_or_delete?(next_node.name) if next_node 
    next_node_word = next_node.text.strip if next_node 
    words.length <= 2 && words.any? { |word| @articleset.include?(word) || (@articleset.include?(base_word) if word == 'n') } && [email protected]?(next_node_word) && !next_node_is_insert_or_delete 
    } 
end 

Оба они почти идентичны, за исключением последней строки, которая определяет конкретное требование функции.

Любые идеи оценили.

+0

Это выглядит довольно процедурно и довольно сложно читать с комментариями, но блоки и вызов 'yield' - это то, что вам нужно здесь. –

ответ

2

Здесь первый разрез. Я перемещал вещи, чтобы они были сгруппированы логически и чтобы облегчить выход.

def get_all_nodes 
    @all_nodes_ins_del_nodes.select do |node| 
    previous_node = node.xpath('preceding-sibling::w:r').last 
    base_word = previous_node.text.split.last.strip.delete('.!?,:') if previous_node 

    next_node = node.next_element 
    next_node_is_insert_or_delete = is_insert_or_delete?(next_node.name) if next_node 
    next_node_word = next_node.text.strip if next_node 

    words = get_node_text(node) 
    words.length <= 2 && 
    !next_node_is_insert_or_delete && 
    yield(words, base_word, next_node_word) 
    end 
end 

all_verb_nodes = get_all_nodes do |words, base_word, next_node_word| 
    words.any? do |word| 
    is_a_verb?(base_word + word) || 
    is_a_verb?(word) && 
    !is_pluralized?(base_word + word, base_word + next_node_word) 
    end 
end 

all_article_nodes = get_all_nodes do |words, base_word, next_node_word| 
    [email protected]?(next_node_word) && 
    words.any? do |word| 
    @articleset.include?(word) || (@articleset.include?(base_word) if word == 'n') 
    end 
end 
+0

Благодарим вас за ваши идеи и код. Я попробую и вернусь к вам. Из того, что я вижу, это дает мне правильный путь и отвечает на мой вопрос. – chell

Смежные вопросы