Ruby Binary Heap插入和发现困境

时间:2017-01-05 20:39:10

标签: ruby-on-rails ruby heap binary-tree

构建我的堆时遇到了一个有趣的问题(没有使用数组的挑战),并想知道是否有人可以提供帮助。到目前为止,我可以插入一些构建的节点,并通过我的输出正确构建我的Heap结构。但是,当我转到#find某个特定节点时,我会收到nil因为看起来我的节点与输出的树没有匹配。保持尽可能浓缩,这就是我所拥有的:

节点构造函数& HeapTree #insert方法

class Node
  attr_accessor :title
  attr_accessor :rating
  attr_accessor :parent
  attr_accessor :left
  attr_accessor :right

  def initialize(title, rating)
    @title = title
    @rating = rating
    @parent = nil
    @left = nil
    @right = nil
  end
end

class HeapSearchTree

  def initialize
    @root = nil
    @heapsize = 1
  end

  def insert(node)
    return nil if node.nil?

    if @root.nil?
      @root = node
    else
      current = @root
      @heapsize += 1 #every insert increases heapsize; used for balancing heap
      until current.left.nil? || current.right.nil?
        if @heapsize % 2 == 0
          current = current.left
        else
          current = current.right
        end
      end

      #after figuring out to go left or right, find the first nil spot
      if current.left.nil? && current.right.nil?
        current.left = node
        node.parent = current
      elsif current.left.nil? && !current.right.nil?
        current.left = node
        node.parent = current
      elsif !current.left.nil? && current.right.nil?
        current.right = node
        node.parent = current
      end

      #heapify by swapping titles and ratings because if I swap parent node for higher node it doesnt stick.
      while node.rating >= node.parent.rating
        if node.parent.rating <= node.parent.left.rating
          temp_title = node.parent.title
          temp_rating = node.parent.rating

          node.parent.title = node.parent.left.title
          node.parent.rating = node.parent.left.rating
          node.parent.left.title = temp_title
          node.parent.left.rating = temp_rating
        elsif node.parent.rating <= node.parent.right.rating
          temp_title = node.parent.title
          temp_rating = node.parent.rating

          node.parent.title = node.parent.right.title
          node.parent.rating = node.parent.right.rating
          node.parent.right.title = temp_title
          node.parent.right.rating = temp_rating
        end
      end
    end
  end
  def find(root=@root, movie_title)
    if root.title == movie_title
      puts "END OF RECURSION"
      puts "movie_title entered: #{movie_title}"
      puts "root.title: #{root.title}"
      return root
    else
      loop = 0
      left = find(root.left, title) if root.left
      right = find(root.right, title) if root.right
      left || right
      loop += 1
      puts loop
    end
  end

问题图片

通过插入martian,您会注意到树正确重新排列。但是,当我tree.find(matrix.title)传入martian.title时,我的回报为零。

enter image description here

我已经对此感到困惑了一段时间,并且无法通过网络找到任何帮助我的东西。如果node.parent的评分低于node.parent,则我会使用node来交换标题和评分。 node ID 正在更改,只是信息。寻找解决方案使这项工作。大多数将显示一个构建的数组,但我不想使用数组用于学习目的。感谢

注意

我发现我的程序在#insert中从 while 循环中冒出来了。我现在正试图移动nodenode.parent,但证明同样困难。

2 个答案:

答案 0 :(得分:0)

这是正确的,因为HeapSearchTree#insert会改变节点而不是正确地交换节点。您的插入代码会改变节点的标题和评级。变量matrix绑定到标题为“The Matrix”的节点,但是通过insert方法将其更改为“The Martian”。

[2] pry(main)> tree = HeapSearchTree.new
[3] pry(main)> matrix = Node.new("The Matrix", 87)
[4] pry(main)> martian = Node.new("The Martian", 92)

[5] pry(main)> tree.insert(matrix)
=> #<Node:0x007f92348c7c10
 @left=nil,
 @parent=nil,
 @rating=87,
 @right=nil,
 @title="The Matrix">
[6] pry(main)> matrix.object_id
=> 70132961787400
[7] pry(main)> matrix.title
=> "The Matrix"

[8] pry(main)> tree.insert(martian)
=> nil

[9] pry(main)> matrix.object_id
=> 70132961787400
[10] pry(main)> matrix.title
=> "The Martian"

[11] pry(main)> tree.find(matrix.title)
END OF RECURSION
movie_title entered: The Martian
root.title: The Martian

这表明绑定到变量matrix的节点正在显示正确的标题,因此搜索该标题会返回正确的结果。正如你所提到的,我会重新实现插入以正确交换节点。

答案 1 :(得分:0)

通过重新创建#insert

解决了我的问题
def insert(root, node)
    root = @root
    current = @root
    @heapsize += 1 #every insert increases heapsize; used for balancing heap

    until current.left.nil? || current.right.nil?
      if @heapsize % 2 == 0
        current = current.left
      else
        current = current.right
      end
    end

    if current.left.nil? && current.right.nil?
      current.left = node
      node.parent = current
    elsif current.left.nil? && !current.right.nil?
      current.left = node
      node.parent = current
    elsif !current.left.nil? && current.right.nil?
      current.right = node
      node.parent = current
    end

  #if rating > (greater than) its parents rating 
    while node.rating >= node.parent.rating
      loop = 1
      temp_parent = node.parent
      temp_parent_right = node.parent.right
      temp_parent_left = node.parent.left
      temp_node_left = node.left
      temp_node_right = node.right
  # if node is greater then its parent and node is to the left of parent
      if node.parent.parent.nil? && node == node.parent.left
        puts "im here on left and parents parent is nil"
        node.right = node.parent.right
        node.parent = node.parent.parent
        node.left = temp_parent

        node.left.parent = node
        node.left.left = temp_node_left
        node.left.right = temp_node_right

        if !node.right.nil?
          node.right.parent = node
        end

        @root = node
        break
  # if node is greater then its parent and node is to the right of parent
      elsif node.parent.parent.nil? && node == node.parent.right
        puts "im here on right and parents parent is nil"

        node.left = node.parent.left
        node.parent = node.parent.parent 
        node.right = temp_parent 

        node.right.parent = node
        node.right.right = temp_node_right
        node.right.left = temp_node_left
        node.left.parent = node

        @root = node
        break


      elsif !node.parent.nil? && node == node.parent.left
        puts "im here on left and my parents parent is not nil"

        if node.parent.parent.left == node.parent
          node.parent.parent.left = node
          node.parent.parent.left.parent = node.parent.parent
          node.left = temp_parent
          node.right = temp_parent_right
          node.left.parent = node
          unless node.right.nil?
            node.right.parent = node
          end
          node.left.left = temp_node_left
          node.left.right = temp_node_right
        elsif node.parent.parent.right == node.parent
          node.parent.parent.right = node
          node.parent.parent.right.parent = node.parent.parent
          node.left = temp_parent
          node.right = temp_parent_right
          node.left.parent = node
          unless node.right.nil?
            node.right.parent = node
          end
          node.left.left = temp_node_left
          node.left.right = temp_node_right
        end

      elsif !node.parent.nil? && node == node.parent.right

        if node.parent.parent.right == node.parent
          node.parent.parent.right = node
          node.parent.parent.right.parent = node.parent.parent
          node.right = temp_parent
          node.left = temp_parent_right
          node.right.parent = node
          unless node.left.nil?
            node.left.parent = node
          end
          node.left.left = temp_node_left
          node.left.right = temp_node_right
        elsif node.parent.parent.left == node.parent
          node.parent.parent.left = node
          node.parent.parent.left.parent = node.parent.parent
          node.left = temp_parent
          node.left = temp_parent_right
          node.right.parent = node
          unless node.right.nil?
            node.right.parent = node
          end
          node.left.left = temp_node_left
          node.left.right = temp_node_right
        end

      end
    end
  end