Skip to content

Небольшой рефакторинг#2

Open
korolev-d-l wants to merge 1 commit intospatecon:mainfrom
korolev-d-l:feature-refactoring
Open

Небольшой рефакторинг#2
korolev-d-l wants to merge 1 commit intospatecon:mainfrom
korolev-d-l:feature-refactoring

Conversation

@korolev-d-l
Copy link

Немного причесал код, оставив заложенную "философию".
Не стал создавать сервисный слой, а так вообще сделал бы иначе, мне по душе больше компонентный подход.

  • Вынес логику в функции с понятным названием
  • При системных ошибках не выдаём их в ответ, а пишем в консоль или лог
    Пример:
{"error":"failed to begin transaction: failed to connect to `user=hts-user database=hts`: 127.0.0.1:5432 (localhost): dial error: dial tcp 127.0.0.1:5432: connect: connection refused"}
  • Добавил тип ошибок serviceError и пару конструкторов помощников.

Вроде как стало понятней, что есть что. Но это уже на ваше усмотрение)

- При системных ошибках не выдаём их в ответ, а пишем в консоль или лог
- Добавил тип ошибок serviceError и пару конструкторов помощников.
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