Skip to content

Feedback#2

Open
ancavar wants to merge 135 commits intofeedbackfrom
add-ui
Open

Feedback#2
ancavar wants to merge 135 commits intofeedbackfrom
add-ui

Conversation

@ancavar
Copy link
Owner

@ancavar ancavar commented May 1, 2023

No description provided.

github-classroom bot and others added 30 commits March 23, 2023 05:28
linter action have to run on every push and pull_request.
* feat: Add AVL tree with own BST implementation
* Create LICENSE

* fix: change insert method

* fix: change tests

* repo: add CI

* feat: implement remove method and test for it
SurfaceYellowDuck and others added 29 commits May 1, 2023 00:08
Co-authored-by: Nikita Lukonenko <68463602+surfaceyellowduck@users.noreply.github.com>
the tree is invalid red-black tree, and any other number is the black height
of the entire tree.
*/
private fun <K : Comparable<K>, V> isRbt(root: RBTNode<K, V>?): Int {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Данная функция должна возвращать Boolean значение.


import bst.nodes.BSTNode

class BSTree<K : Comparable<K>, V>(val key: K? = null, val value: V? = null) :
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Такое ощущение, что в конструкторе не нужны key и value. У пользователя есть функции вставки.

import org.junit.jupiter.api.Nested
import org.junit.jupiter.api.Test

class RbtTest {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Кажется еще нужны тесты для проверки корректности поворотов в различных кейсах вставок/удаления

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants