Обсуждение:Дерево поиска, наивная реализация — различия между версиями
(Новая страница: «Исправь вместо hull - NULL») |
|||
| (не показана 1 промежуточная версия этого же участника) | |||
| Строка 1: | Строка 1: | ||
| − | + | :{{tick|ticked=1}} Нет, не "или равные" | |
| + | :{{tick|ticked=1}} Фееричное название процедуры Tree_inorder - абсолютно ни одному стандарту наименований не соответствует. Ну, и остальные =) | ||
| + | :: Обоход дерева называется не walk, а traversal | ||
| + | :: не надо во всех названиях писать tree | ||
| + | :: почему у методов, которые что-то возвращают, нет типа? | ||
| + | :: Выдели все операторы жирным. | ||
| + | :: В treeDelete какое-то полотно кода. Хотя бы напиши комментарии, где какой случай обрабатывается. | ||
| + | :{{tick|ticked=1}} « при обнаружении нулевого указателя нужно подвесить на него вставляемый элемент» — подвешивают не на нулевой указатель все-таки. | ||
| + | :{{tick|ticked=1}} Есть еще два вида обходов — написать про них. | ||
| + | :{{tick|ticked=1}} Картинки низкокачественные. Нет, никто по ним кликать не будет. | ||
| + | :{{tick|ticked=1}} Лучше словесное описание размещать до псевдокода, а не после него | ||
| + | :{{tick|ticked=1}} Картинка про удаление узла с двумя дочерними узлами дурацкая | ||
| + | :: слишком здоровенная картинка. | ||
| + | :: Что за Usún 13? o_O | ||
| + | :: на картинках разный масштаб у вершин. | ||
| + | :: Вообще она какая-то странная. Куча разных цветов, непонятно что обозначающих, где-то есть пунктирные стрелки, а где-то нет. Пунктирные рамки то есть, то нет. | ||
| + | :: Шаги неплохо бы прокомментировать прямо на картинке (например, под стрелками) | ||
| + | :{{tick|ticked=1}} Дать ссылки на конспект про упорядоченное множество (это там, где про какое-то «динамическое множество» написано), на википедию | ||
Текущая версия на 19:29, 10 июня 2012
- ☑ Нет, не "или равные"
- ☑ Фееричное название процедуры Tree_inorder - абсолютно ни одному стандарту наименований не соответствует. Ну, и остальные =)
- Обоход дерева называется не walk, а traversal
- не надо во всех названиях писать tree
- почему у методов, которые что-то возвращают, нет типа?
- Выдели все операторы жирным.
- В treeDelete какое-то полотно кода. Хотя бы напиши комментарии, где какой случай обрабатывается.
- ☑ « при обнаружении нулевого указателя нужно подвесить на него вставляемый элемент» — подвешивают не на нулевой указатель все-таки.
- ☑ Есть еще два вида обходов — написать про них.
- ☑ Картинки низкокачественные. Нет, никто по ним кликать не будет.
- ☑ Лучше словесное описание размещать до псевдокода, а не после него
- ☑ Картинка про удаление узла с двумя дочерними узлами дурацкая
- слишком здоровенная картинка.
- Что за Usún 13? o_O
- на картинках разный масштаб у вершин.
- Вообще она какая-то странная. Куча разных цветов, непонятно что обозначающих, где-то есть пунктирные стрелки, а где-то нет. Пунктирные рамки то есть, то нет.
- Шаги неплохо бы прокомментировать прямо на картинке (например, под стрелками)
- ☑ Дать ссылки на конспект про упорядоченное множество (это там, где про какое-то «динамическое множество» написано), на википедию
