Skip to content

Lesson6 - #6

Open
iisat wants to merge 3 commits into
lesson5from
Lesson6
Open

iisat wants to merge 3 commits into
lesson5from
Lesson6

Conversation

@iisat

@iisat iisat commented Nov 14, 2018

Copy link
Copy Markdown
Owner

No description provided.

@Dzhoker1 Dzhoker1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Оставил комментарии по коду

Comment thread Lesson6/task1.py
print(mod_size(maxpos))
print(mod_size(array))
print(mod_size(countarray))
print(f'Summary: {mod_size(maxpos) + mod_size(array) + mod_size(countarray)}')

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Константы решили не добавлять? )))

Comment thread Lesson6/task1.py
#14
#Summary: 183858

# Вывод - функция 1 - жертвуем памятью ради быстродействия, функция 2 - жертвуем быстродействием - экономим память

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Отлично

Comment thread Lesson6/task1.py
for i in a:
if a[i] != 0:
b.append(a[i])
#del a

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Хорошо, что закомментировали. Даже успел подумать об ошибке при замерах удалённой переменной

Comment thread Lesson6/task1.py
#findsimple(100)
#Summary: 1868

# В данном случае можем сделать точно такой же вывод как и в первом случае - алгоритм, экономнее расходующий память

Copy link
Copy Markdown

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.

2 participants