Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Algorithms&data structures 6#11

Open
SRytv wants to merge 14 commits into
masterSRytv/SR_Python_HWork:masterfrom
Algorithms&Data_Structures_6SRytv/SR_Python_HWork:Algorithms&Data_Structures_6Copy head branch name to clipboard
Open

Algorithms&data structures 6#11
SRytv wants to merge 14 commits into
masterSRytv/SR_Python_HWork:masterfrom
Algorithms&Data_Structures_6SRytv/SR_Python_HWork:Algorithms&Data_Structures_6Copy head branch name to clipboard

Conversation

@SRytv

@SRytv SRytv commented Oct 14, 2018

Copy link
Copy Markdown
Owner

Мне очень стыдно за подобное выполнение ДЗ, но, к сожалению ничего более достойного я из себя выжать не смог. Не хватает элементарного понимания, каким образом кроме как вручную можно посчитать расход памяти на отдельные переменные. С крупными массивами еще, Вы более или менее разъяснили. Но все равно - это все "ручная работа".

@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.

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


if __name__ == "__main__":
def show_size(x, level=0):
print('\t'*level, 'type = {:s}'.format(x.__class__), 'size = {:d}'.format(sys.getsizeof(x)), 'object = ', x)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Возможно из-за разницы в версия python, но у меня программа не заработала. Пришлось исправить строку на:
print('\t'*level, 'type = {}'.format(x.__class__), 'size = {}'.format(sys.getsizeof(x)), 'object = ', x)

show_size(y, level+1)


print(show_size([[], [], []])) #первоначальный размер матрицы

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Интересно, вы обратили внимание на None в результатах вывода?
Это ошибка, т.к. вы распечатывает функцию, которая не имеет return.

#очень большой объем занимают строки взаимодействия с пользователем около 870 байт
#можно предположить, что на этом фоне память, занимаемая переменными программы,
#врядли окажется сколь-нибудь существенной
#и того в конечном итоге программа расходовала около 1000 байт памяти No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Как же вы посчитали сумму? Если в уме - мой респект и уважение. А если вручную складывали, например на калькуляторе - ужас и позор.
Вы же программист - программа должна находить сумму сама.

answer = 's'

matrix = [[], [], [], []]
columns_numb = 5

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.