Skip to content

Refact dev g5#51

Open
OleksandrLysak1 wants to merge 6 commits intogoitProjects:mainfrom
OleksandrLysak1:refactDEV_G5
Open

Refact dev g5#51
OleksandrLysak1 wants to merge 6 commits intogoitProjects:mainfrom
OleksandrLysak1:refactDEV_G5

Conversation

@OleksandrLysak1
Copy link
Copy Markdown

/*

  • Замінив назви змінних на більш зрозумілі та описові.
  • Додав константи, такі як BOARD_SIZE, замість використання магічних чисел.
  • Виніс перевірку кожної лінії в окремий метод checkLine.
  • Спрощено метод checkVictory.
  • Логіка роботи циклів була оптимізована через використання логічних змінних (validMove, gameFinished)
    замість використання декількох break або continue.
  • Перевірив і відкоригував форматування коду відповідно до стандартів.
  • Збільшив читабельність коду через додаткову структуру методів.
  • Скоротив кількість дубльованих умов для перевірки перемоги.
  • Оптимізував цикли шляхом винесення загальної логіки в окремі методи.
  • Проаналізував проект з використанням SonarLint.
    */

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