-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Садовников - 15 #60
base: master
Are you sure you want to change the base?
Садовников - 15 #60
Conversation
В седьмую лабораторную добавил изменённые версии 4, 5 и 6 заданий |
Восьмую задачу-то я поптыался сделать сам, не уверен, что оно было к лучшему! |
В 6 лабе фейлится мой тест. А в ваших тестах нет тестирования негативных чисел |
импортируйте, пожалуйста, сортировки для 7 лабы из предыдущих лаб |
print "empty" | ||
return value | ||
|
||
def _private_calc(self, n): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
мой тестовый код можно было бы убрать
валится тест def test_erofeev(self):
queue = lab8.MaxElementQueue()
queue.push(3)
queue.push(42)
self.assertEqual(42, queue.max())
self.assertEqual(3, queue.pop())
self.assertEqual(42, queue.max())
self.assertEqual(42, queue.pop()) добавьте, пожалуйста, консольный интерфейс |
договорились устно, что на багфикс надо написать тест |
Исправил вообще всё и добавил первую часть девятой лабы! |
Ссылка на сабмит в систему CodeForces! |
def Unite(self, x, y): | ||
x = self.Find(x) | ||
y = self.Find(y) | ||
if randint(1, 2) == 1: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
сделате здесь нормальное сжатие путей, пожалуйста
методы по-прежнему пишутся с маленьйо буквы
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
На Хабре просто заверили меня, что в среднем работает так же быстро! Но обязательно исправлю!
7 ok |
Спасибо! Так я про 9.2 коммент оставил!
|
И 14 я, по-моему, не много исправил :C |
Исправил 17! |
a.add_vertex(0) | ||
a.add_vertex(1) | ||
a.add_vertex(2) | ||
a.add_vertex(3) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А что если узлов будет больше?
We need change!
14 ok |
осталась 9.2 и 15 |
15 сделал! Духота страшная! |
15+, обратите внимание на комменты |
Понял Вас, спасибо! |
No description provided.