Ошибка исправления вставки красно-черного дерева

Question

У меня проблема с домашней работой по вставке в красно-черные деревья в c #.Я написал код ниже, и программа работает без каких-либо проблем для добавления первых 3 чисел.Когда я пытаюсь добавить 4-е число, я получаю исключение NullReferenceException.Я пытаюсь устранить ошибку в течение 2 дней, но не могу понять.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace Algoritmalar3b
{
class rbtNode
{
    public int? key;
    public char? color;
    public rbtNode left, right, p;
    public rbtNode(int? key, char? color, rbtNode left, rbtNode right, rbtNode p)
    {
        this.key = key;
        this.color = color;
        this.left = left;
        this.right = right;
        this.p = p;
    }
}

class Program
{
    static rbtNode Tnil = new rbtNode(null, 'B', null, null, null);

    static void Main(string[] args)
    {
        rbtNode root = new rbtNode(null, 'B', Tnil, Tnil, null);

        RB_Insert(root, 7);
        RB_Insert(root, 3);
        RB_Insert(root, 89);
        RB_Insert(root, 4);
        RB_Insert(root, 9);
        RB_Insert(root, 15);
        RB_Insert(root, 35);
        RB_Insert(root, 8);
        RB_Insert(root, 24); 
        preOrderWalk(root);
    }

    static void RB_Insert(rbtNode T, int? deger)
    {
        rbtNode z = new rbtNode(deger, null, Tnil, Tnil, null);

        if (T.key == null)
            T.key = deger;
        else
        {
            rbtNode y = new rbtNode(null, null, Tnil, Tnil, null);
            y = Tnil;
            rbtNode x = new rbtNode(null, null, Tnil, Tnil, null);
            x = T;
            while (x != Tnil)
            {
                y = x;
                if (z.key < x.key)
                    x = x.left;
                else
                    x = x.right;
            }
            z.p = y;
            if (y == Tnil)
                T = z;
            else if (z.key < y.key)
                y.left = z;
            else
                y.right = z;
            z.left = Tnil;
            z.right = Tnil;
            z.color = 'R';
            RB_Insert_Fixup(T, z);
        }
    }

    static void RB_Insert_Fixup(rbtNode T, rbtNode z)
    {
        rbtNode y = new rbtNode(null, null, Tnil, Tnil, null);

        while (z.p.color == 'R')
        {
            if (z.p == z.p.p.left)
            {
                y = z.p.p.right;
                if (y.color == 'R')
                {
                    z.p.color = 'B';
                    y.color = 'B';
                    z.p.p.color = 'R';
                    z = z.p.p;
                }
                else if (z == z.p.right)
                {
                    z = z.p;
                    Left_Rotate(T, z);
                    z.p.color = 'B';
                    z.p.p.color = 'R';
                    Right_Rotate(T, z.p.p);
                }
            }
            else
            {
                y = z.p.p.left;
                if (y.color == 'R')
                {
                    z.p.color = 'B';
                    y.color = 'B';
                    z.p.p.color = 'R';
                    z = z.p.p;
                }
                else if (z == z.p.left)
                {
                    z = z.p;
                    Left_Rotate(T, z);
                    z.p.color = 'B';
                    z.p.p.color = 'R';
                    Right_Rotate(T, z.p.p);
                }
            }
        }
        T.color = 'B';
    }

    static void Left_Rotate(rbtNode T, rbtNode x)
    {
        rbtNode y = new rbtNode(null, null, Tnil, Tnil, null);
        y = x.right;
        x.right = y.left;
        if (y.left != Tnil)
            y.left.p = x;
        y.p = x.p;
        if (x.p == Tnil)
            T = y;
        else if (x == x.p.left)
            x.p.left = y;
        else
            x.p.right = y;
        y.left = x;
        x.p = y; 
    }

    static void Right_Rotate(rbtNode T, rbtNode x)
    {
        rbtNode y = new rbtNode(null, null, Tnil, Tnil, null);
        y = x.left;
        x.left = y.right;
        if (y.right != null)
            y.right.p = x;
        y.p = x.p;
        if (x.p == null)
            T = y;
        else if (x == x.p.right)
            x.p.right = y;
        else
            x.p.left = y;
        y.right = x;
        x.p = y;
    }

    static void preOrderWalk(rbtNode T)
    {
        if (T.color == 'B')
            Console.WriteLine("-{0}", T.key);
        else
            Console.WriteLine("{0}", T.key);
        if (T.left != null)
            preOrderWalk(T.left);
        if (T.right != null)
            preOrderWalk(T.right);
    }
}

}

Answer 1

Кажется, у вас проблемы в 3 областях:

• Алгоритм дерева RB.Но похоже, что вы почти там
• Отладка.Чтобы отозвать такую ​​ошибку, потребуется всего 2 минуты, может быть, 2 часа, если вы новичок.Но не 2 дня.
• Задать хороший вопрос.У вас есть nullref, но по какой линии?и каковы значения переменных / полей в этой строке?

---

Когда я бегло взгляну на метод Fixup и остальную часть кода, я подозреваю, что у вас может быть ap (Parent) ref, являющийся нулем в неправильной точке.

В качестве инструмента диагностики измените элемент .p в обычном свойстве:

class rbtNode
{    
    private rbtNode _parent;

    public rbtNode Parent
    {
         get { return _parent; }
         set
         {
             System.Diagnostics.Debug.Assert(value != null);
             _parent = value;
         }
    }
    ....

Я думаю, вам придется разрешить _parent = null, но только в конструкторе.

Члены get / set также дают вам хорошее место для установки (условных) точек останова.

Answer 2

Я полагаю, вы ошибаетесь в своей функции fix_up .Добавьте определенные условия в цикл while, то есть

while (z != null && z.Getparent() != null && z.Getparent().Getcolor() == 'R')

Кроме того, вы делаете большую ошибку в остальной части цикла while.Измените порядок функций LeftRotate и RightRotate следующим образом:

               else if (z == z.Getparent().Getleft())
                    {
                        z = z.Getparent();
                        **RightRotate(T, z);**
                        z.Getparent().Setcolor('B');
                        z.Getparent().Getparent().Setcolor('R');
                        **LeftRotate(T, z);**
                    }

РЕДАКТИРОВАТЬ: вам также необходимо добавить еще несколько условий в обе функции поворота.