Skip to content
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

Review 1.0 #1

Open
theaspect opened this issue Apr 17, 2019 · 0 comments

Comments

Projects
None yet
1 participant
@theaspect
Copy link

commented Apr 17, 2019

  1. Нужны скрипты сборки/тестирования
  2. Проект должен быть привязан к Travis CI
  3. Файлы с кодом должны находиться в папке src и в пакете com.автор.проект
  4. Релиз сдается ссылкой на тег (https://git-scm.com/book/ru/v1/Основы-Git-Работа-с-метками)
  5. В main.kt не надо создавать промежуточные переменные user1 и user2
  6. Хэширование и создание соли необходимо делать при создании пользователя а не после инициации бизнес-логики
  7. вот это не понял что такое https://github.com/shaburovda/one/blob/master/Main.kt#L18
  8. не вижу кодов возврата System.exit(код)
  9. зачем эти промежуточные переменные https://github.com/shaburovda/one/blob/master/UserService.kt#L3
  10. должно использоваться хэширование https://github.com/shaburovda/one/blob/master/UserService.kt#L11
  11. для кодов возврата нужно сделать Enum и хранить коды в нём, в коде не должно быть магических числе https://github.com/shaburovda/one/blob/master/BusinessLogic.kt#L5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.