Opened 3 years ago
Closed 3 years ago
#549 closed ожидается проверка (задача сдана)
WW #11
Reported by: | savinov.vladislav | Owned by: | Святослав Власов |
---|---|---|---|
Component: | WW_vector | Version: | 3.0 |
Keywords: | Cc: |
Description
Change History (5)
comment:1 Changed 3 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
- переписал get_closest_2pw на обычный for
- ненужные try-catch-и убрал
- resize пофиксил - упростил разбор случаев
comment:3 Changed 3 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
get_closest_pw
всё равно сложно :) Он же пишется элементарно цикломint res = 1; while (res < n) { res *= 2; } return res;
- С гарантиями исключений всё равно всё плохо. Валгринд ругается на утечки и то, что ты юзаешь неинициализированные переменные.
9/10
comment:4 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 2.0 → 3.0 |
Постарался пофиксить возможные утечки
comment:5 Changed 3 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Все тесты прошли.
Только простыни кода мне твои не нравятся, но ладно, 10/10
Note: See
TracTickets for help on using
tickets.
Тест на ресайз упал. Что-то у тебя там много лишнего намудрено и похоже что неправильно. Как минимум я не понимаю зачем ты освобождаешь элемент, а заместо него конструируешь пустой?
get_closest_2pw
написан ОЧЕНЬ сложно. Он пытается найти нужную степень двойки за логарифмическое время? Если так, то для 4 или 8 байтовых чисел это совершенно не нужно, избыточно и на одном только вызовеstd::log2
сжирает весь потенциальный профит.Если из
new
вылетает исключение, то оно просто пробрасывается дальше. Того же самого эффекта можно было достичь просто убрав этотtry-catch
Пока что 7/10