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

Conversation

GolzitskyNikolay
Copy link

@GolzitskyNikolay GolzitskyNikolay commented Jan 15, 2021

Лабораторная делалась совместно с Антоном Натурой, отчёт в readme

@wrbbz wrbbz added the Третья лабораторная Задания из методички label Jan 16, 2021
@wrbbz
Copy link
Contributor

wrbbz commented Jan 16, 2021

Т.к. в одной репе и сервер, и клиент (задачи разных людей), добавлять все файлы одним коммитом - не очень хорошо. Ибо непонятно, кто что делал. Можно было двумя коммитами. Один - сервер, второй - клиент. За авторством ответственного (Это просто замечание на будущее. Его только принять и осознать)

Новостной клиент:

  • Не работает Backspace. Только Del (Править не обязательно).

По новостям все ок. Зачтено

Что касается кошельков:

  • При вводе в клиенте отрицательной суммы все виснет. При отключении сервера (в данном случае, порт не освобождается)
  • Некоторые исключения на сервере не обрабатываются (виесто определенного сообщения - стэк трейс)
  • Гонять пароли по сети в таком виде, как сейчас - не есть хорошая практика. Можно хэшировать. Как самый простой вариант (На будущее. Сейчас не так важно)
  • Не работает Backspace. Только Del (Править не обязательно).

Итого: поправить первый пункт (в кошельке) и лаба зачтена

@GolzitskyNikolay
Copy link
Author

Клиент обновлён

@GolzitskyNikolay
Copy link
Author

Здравствуйте, вы мне работу зачли в общей таблице, но здесь реквест всё ещё открытый, хотя у моего напарника (#83) он закрыт.

@wrbbz
Copy link
Contributor

wrbbz commented Jan 18, 2021

Моя ошибка. Спасибо

@wrbbz wrbbz closed this Jan 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants