Обсуждение:Дерево поиска, наивная реализация

☑ Нет, не "или равные"

☐ Фееричное название процедуры Tree_inorder - абсолютно ни одному стандарту наименований не соответствует. Ну, и остальные =)

Обоход дерева называется не walk, а traversal

не надо во всех названиях писать tree

почему у методов, которые что-то возвращают, нет типа?

Выдели все операторы жирным.

В treeDelete какое-то полотно кода. Хотя бы напиши комментарии, где какой случай обрабатывается.

☐ « при обнаружении нулевого указателя нужно подвесить на него вставляемый элемент» — подвешивают не на нулевой указатель все-таки.

☐ Есть еще два вида обходов — написать про них.

☐ Картинки низкокачественные. Нет, никто по ним кликать не будет.

☑ Лучше словесное описание размещать до псевдокода, а не после него

☐ Картинка про удаление узла с двумя дочерними узлами дурацкая

слишком здоровенная картинка.

Что за Usún 13? o_O

на картинках разный масштаб у вершин.

Вообще она какая-то странная. Куча разных цветов, непонятно что обозначающих, где-то есть пунктирные стрелки, а где-то нет. Пунктирные рамки то есть, то нет.

Шаги неплохо бы прокомментировать прямо на картинке (например, под стрелками)

☐ Дать ссылки на конспект про упорядоченное множество (это там, где про какое-то «динамическое множество» написано), на википедию