Skip to content

Lab 2: Code Smells#13

Open
KonstantinDanger wants to merge 16 commits intolenagrin:masterfrom
KonstantinDanger:lab-2
Open

Lab 2: Code Smells#13
KonstantinDanger wants to merge 16 commits intolenagrin:masterfrom
KonstantinDanger:lab-2

Conversation

@KonstantinDanger
Copy link
Copy Markdown

Lab 2: Code Smells

Мета роботи

Виправити 5-10 зауважень Sonar (bugs/smells) без зміни поведінки.


Необхідні умови

  • Дрібними комітами виправити знайдені Sonar-проблеми у NetSdrClientApp.

Виконав студент:
Яцюк Костянтин
групи ПЗС-1


Хід роботи

1. Code smells, які будуть виправлені

• Make '_cancellationTokenSource' 'readonly'.
• Declare types in namespaces (EchoServer)
• Use a constructor overloads that allows a more meaningful exception message to be provided / Call the ArgumentOutOfRangeException constructor that contains a message and/or paramName parameter
• Make '_tcpClient' 'readonly'.
• Make '_udpClient' 'readonly'.
• Remove empty statement.
• Make '_udpClient_MessageReceived' a static method.
• Possible null reference return.
• Cannot convert null literal to non-nullable reference type.


2. Метрики SonarCloud до виправлень

image image image image image

3. Створення пулл-реквеста з мерджем в головну гілку після виправлень

image

4. Метрики SonarCloud після виправлень

image image image image image

Як перевірити

  1. Відкрити Pull Request: Lab 2: Code smell fixes KonstantinDanger/reengineering-course-fork#3
  2. Перейти у вкладку "Checks"
  3. Переконатися, що SonarCloud analysis виконано
  4. Перевірити статус Quality Gate
  5. Перевірити коміти виправлень code smells

Висновки

Під час виконання даної лабораторної роботи було виправлено Code smells та проаналізовано відповідні метрики на SonarCloud.

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