Skip to content

Initialize UdpClient in UdpClientWrapper#13

Open
Sane4ka126 wants to merge 1 commit into
masterfrom
Sane4ka126-patch-13
Open

Initialize UdpClient in UdpClientWrapper#13
Sane4ka126 wants to merge 1 commit into
masterfrom
Sane4ka126-patch-13

Conversation

@Sane4ka126
Copy link
Copy Markdown
Owner

Refactor error handling and cleanup logic in UdpClientWrapper. image . Дублікат коду в методах StopListening() та Exit() Було (2 ідентичних блоки коду): Метод 1: StopListening() image Метод 2: Exit() image Проблема: Обидва методи робили абсолютно те саме - скасовували токен відміни, закривали UDP клієнт та виводили повідомлення.Крок 1: Створення приватного методу Створили новий приватний метод CleanupResources(), який містить загальну логіку: image Це внутрішня деталь реалізації Не потрібен ззовні класу Інкапсулює логіку очищення ресурсів

Крок 2: Рефакторинг публічних методів Переписали обидва публічні методи, щоб вони викликали CleanupResources(): image Крок 3: Видалення непотрібної змінної У методі StartListeningAsync() знайшли непотрібне оголошення змінної: image змінна ex не використовується image

@sonarqubecloud
Copy link
Copy Markdown

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.

1 participant