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

Гуляев Дмитрий, 3530901/70201, Lab1 #50

Closed
wants to merge 9 commits into from

Conversation

Mr-Shpinks
Copy link

Описание

Первая лабораторная. Реализация на Python при помощи стандартной библиотки socket (BSD socket interface). Визуальный интерфейс создан при помощи библиотеки tkinter.

Инструкция по использованию

После запуска сервера можно запускать несколько клиентов, сначала требуется ввести имя в специальном окне, далее перейти к самому чату и отправке сообщений.

Инструкция по сборке/установке

Требования:

Python 3, tkinter

Запуск скриптов производится из командной строки/терминала

@wrbbz
Copy link
Contributor

wrbbz commented Jan 4, 2021

  • Протокол отсутствует как таковой (по сети гоняются строки)
  • Отсутствует описание протокола
  • В отчете, было бы полезно, указать способ запуска софта (python server.py и т.п.)
  • bind на локалхост. Не есть хорошая практика. В идеале - предоставлять пользователю возможность задать IP на котором он хочет запускать сервер и клиент. А если такой аргумент не поступил - оставлять localhost в качестве дефолта
  • Захардкожен номер порта. Такой порт может быть занят чем-то другим. Опять же - предоставление выбора пользователю
  • GUI, конечно, хорошо. Но в данной лабе не обязательно (лишняя работа. Не требует исправлений)
  • Способ выхода из клиента (помимио "KeyboardInterrupt")
  • Чем обусловленно ограничение длины ника в 50 символов (описать в отчете)
  • При завершении сервера - в клиентах ничего не выводится/не завершаются. В фоне сыпятся исключения (там, где открыт терминал)

@wrbbz wrbbz added Первая лабораторная TCP чат Требуется доработка Доработать отчёт Отчёта нет или он не соответствует требованиям labels Jan 4, 2021
@Mr-Shpinks
Copy link
Author

А что должно передаваться по сети и по каким принципам должен формироваться протокол? Я не видел подобных требований. Уточните, пожалуйста.

@wrbbz
Copy link
Contributor

wrbbz commented Jan 14, 2021

Суть данной лабораторной - реализация чата с использованием TCP сокетов и протокола. Это не про передачу голых строк по сети. Однако, данная ошибка весьма распространена. Если посмотреть на уже принятые PR, то во многих можно увидеть такое замечание. В идеале - гонять какой-то пакет по сети. И парсить его. Описание данного пакета можно вставить в отчет в качестве описания протокола

@Mr-Shpinks
Copy link
Author

Я решил переписать работу с нуля без графического интерфейса, теперь в корне репозитория находятся новые файлы для lab1a (блокирующие сокеты), полный отчёт в readme. Решил не создавать новый PR. Надеюсь, путанницы не возникнет.

@Mr-Shpinks
Copy link
Author

Сюда же залил неблокирующие сокеты

@wrbbz wrbbz added Зачтено and removed Доработать отчёт Отчёта нет или он не соответствует требованиям Требуется доработка labels Feb 24, 2021
@wrbbz wrbbz closed this Feb 24, 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