8000 HR-1 Рефакторинг кода by ioramishvili · Pull Request #367 · NodaSoft/hr · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

HR-1 Рефакторинг кода #367

8000
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from

Conversation

ioramishvili
Copy link

Код представляет собой PHP-класс TsReturnOperation, который наследуется от ReferencesOperation и предназначен для обработки уведомлений. В зависимости от типа уведомления он выполняет различные действия, например, отправка электронных писем и SMS.

Основные замечания по коду:

  • Отсутствие валидации и проверки данных
  • Некорректный нейминг класса и переменных
  • Нарушение принципов SOLID (для примера реализован отдельный класс для отправки письма, можно выносить и другой функционал)
  • Не настроены исключения (всегда один тип исключений, создал пару примеров новых исключений)
  • Спагетти-код, вынес отдельные логики в свои функции
  • Некорректное указания типов возвращаемых значений
  • Переопределение переменных или отсутствие их инициализации

Для дальнейшего улучшения:

  • Продолжить выносить логику в отдельные классы
  • Упрощение функций (уменьшение передаваемых переменных)
  • Использование DTO

@groall groall closed this Jan 23, 2025
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.

2 participants
0