Conversation
ylyubimov
left a comment
ylyubimov
left a comment
There was a problem hiding this comment.
Решение ок. Но замечания погляди.
| Treap& operator= (const Treap&&) = delete; | ||
| ~Treap(); | ||
|
|
||
| void pre_order_print() const; |
There was a problem hiding this comment.
Если делать такую фукнцию, то с параметром, а что собственно делать с элементами?
There was a problem hiding this comment.
Имеется ввиду, чтобы была не просто функция print? Чтобы был обход дерева, а там уже хотим печатаем, хотим еще что-то делаем?
| void pre_order_print() const; | ||
| void insert(int key, int priority); | ||
| void erase(int key); | ||
| int kth_largest(int idx); |
There was a problem hiding this comment.
Понял, исправил. Но не здесь. Здесь не знаю, как исправить прямо в пулл реквесте.
There was a problem hiding this comment.
Делаешь еще один коммит в эту ветку и все)
| Node* root = nullptr; | ||
| void split(Node* node, int key, Node*& left, Node*& right); | ||
| Node* merge(Node* left, Node* right); | ||
| Node* kth_largest_(int idx, Node* node); |
There was a problem hiding this comment.
Не понимаю, что такое static. Это, вроде, глобальная переменная, но куда ее здесь не понятно.
|
|
||
| private: | ||
| Node* root = nullptr; | ||
| void split(Node* node, int key, Node*& left, Node*& right); |
There was a problem hiding this comment.
Не понимаю, что такое static
| private: | ||
| Node* root = nullptr; | ||
| void split(Node* node, int key, Node*& left, Node*& right); | ||
| Node* merge(Node* left, Node* right); |
There was a problem hiding this comment.
Не понимаю, что такое static
There was a problem hiding this comment.
static Node* merge(Node* left, Node* right)
static означает, что этот метод не использует переменные экземпляра класса. Т.е. this у них не будет.
ylyubimov
left a comment
ylyubimov
left a comment
There was a problem hiding this comment.
Постарался ответить на твои вопросы
2 participants
No description provided.