Привет Наско!
И аз да отбележа, че имах доста проблеми със някои от тестовете, и сега като сверих някои стойности с тези на Краси и тези от от упражнението и ми се стори странно следното :

1) За  GetQuadrant функциите :
Заместих Top/Bottom quadrant стойностите, с тези от лаба ... и фейлнаха тестовете?

2) За Split-a функцията - Children[0] и Children[1] (top right/top left)

Заместих стойностите с тези от лаба и фейлнаха тестовете.

3) За Split функцията : 

Не разбирам разликата между :

    for (int i = 0; i < currentNode.Items.Count;)
            {
                var item = currentNode.Items[i];
                int quadrant = this.GetQuadrant(currentNode, item.Bounds);
                if (quadrant != -1)
                {
                    currentNode.Items.Remove(item);
                    currentNode.Children[quadrant].Items.Add(item);
                }
                else
                {
                    i++;
                }

и слагането на итерацията във for statement-a :

    for (int i = 0; i < currentNode.Items.Count;i++)
            {
                var item = currentNode.Items[i];
                int quadrant = this.GetQuadrant(currentNode, item.Bounds);
                if (quadrant != -1)
                {
                    currentNode.Items.Remove(item);
                    currentNode.Children[quadrant].Items.Add(item);
                }

понеже фейлват тестовете ако не се изкара итератора в Else statement-a?

Ако правилно отгатвам е че колекцията се променя като се трият елементи и се пренареждат индексите.

Но тогава не разбирам защо примера във лаба е даден по този начин? Или аз нещо се бъркам?

Иначе, към @krasi

Имаше парче код за Insert функцията, може лесно да си спестиш код с това :

if (quadrant != -1)
                {
                    currentNode = currentNode.Children[quadrant];
                    depth++;
                }

Поздрави,

Владо

Не разбрах кое си заместил и са фейлнали тестовете. Най-добре качи линк към кода и да видим.

Иначе вцикъла на Split както е в лаба трябва при триене i да намаля. Честно казано твоят вариант е по-умен, няма причина да е така.

Здравей!

И на мен този лаб ми се оказа най-проблематичен. GetQuadrant и Split  от лаба са коректни.

Колкото до for-цикъла аз го въртях от (currentNode.Items.Count -1)  до 0, за да не излизам от range-a на индексите и дълго се чудих възможно ли е да стане по начинът написан в лаба.

И аз бях пропуснала в ShouldSplit да проверя дали node-a няма вече  деца.

Като цяло курсът много ми хареса и обогати знанията ми. Искам да благодаря на Наско за положените усилия.

Успех на изпита на всички!