Skip to content
This repository has been archived by the owner on Sep 9, 2021. It is now read-only.

Антропова А. и Романов А. 3530901/70201 lab3_1 #76

Closed
wants to merge 9 commits into from

Conversation

Lexcorp3439
Copy link

Отчет лежит в файле README.md

@Lexcorp3439 Lexcorp3439 changed the title Антонова А. и Романов А. 3530901/70201 lab3_1 Антропова А. и Романов А. 3530901/70201 lab3_1 Jan 13, 2021
@wrbbz wrbbz added the Третья лабораторная Задания из методички label Jan 16, 2021
try:
input_str = input(message)

while not isint(input_str):
Copy link
Contributor

Choose a reason for hiding this comment

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

isint(-1), например, вернет True, но это не есть unsisgned int. (Так, просто замечание)

Copy link
Contributor

@wrbbz wrbbz left a comment

Choose a reason for hiding this comment

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

Зачтено

def safe_input(message: str) -> str:
try:
input_str = input(message)
except KeyboardInterrupt:
Copy link
Contributor

Choose a reason for hiding this comment

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

Не уверен, но, как мне кажется, KeyboardInetrrup можно обрабатывать один раз. В основном методе. А в вызывемых - прокидывать по стеку наверх (Просто наблюдение. Не требует исправлений)

@wrbbz
Copy link
Contributor

wrbbz commented Jan 16, 2021

Я уже зачел, конечно. Но, в идеале, добавить отключение клиентов от сервера в случае завершения сервера. Это сделано в какой-то мере, но при отключении сервера - клиент остается висеть (в случае бездействия), что делает недоступным порт, на котором был открыт сервер. Может быть, стоит подумать про отправку какого-нибудь кода завершения всем подключенным клиентам.
Повторюсь, лаба уже зачтена. Так что, если не сделаете - ничего страшного

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Зачтено Третья лабораторная Задания из методички
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants