Skip to content

ДЗ 5.1#111

Open
Dimaly123 wants to merge 12 commits into
mainfrom
Lyagin
Open

ДЗ 5.1#111
Dimaly123 wants to merge 12 commits into
mainfrom
Lyagin

Conversation

@Dimaly123
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

Есть существенные замечания.

Comment thread Practice/Lyagin/dim_5.1.py Outdated
Comment thread Practice/Lyagin/dim_5.1.py Outdated
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

Хорошо, но есть вопросы.

Comment thread Practice/Lyagin/dim_5.2.py Outdated
Comment thread Practice/Lyagin/dim_5.1.py Outdated
Comment thread Practice/Lyagin/dim_5.1.py Outdated
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

По dim_6.1.py небольшие замечания по стилю.
dim_5.1.py, как я понимаю, ещё будете доделывать?

Comment thread Practice/Lyagin/dim_6.1.py Outdated
Comment thread Practice/Lyagin/dim_6.1.py
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

6.1 зачтено
5.1 - ОК. Могу пока зачесть, что есть... но только первоначальную редакцию (с функцией min_el). Она, хоть и не соответствует заданию в точности, но всё же является правильным решением.

Comment thread Practice/Lyagin/dim_6.1.py Outdated
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

Всё супер!
Осталось один нюанс прояснить.

Comment thread Practice/Lyagin/dim_5.1.py Outdated
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

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

Всё хорошо. Буквально несколько мелочей осталось.

while j < len(arr):
if arr[j] < arr[m]:
m = j
j = j+1
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

j += 1

Comment thread Practice/Lyagin/dim_5.3.py Outdated
Comment thread Practice/Lyagin/dim_5.3.py Outdated
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.

3 participants