Привет ребят, пишу на Go чуть более пол года) К

Question

Привет ребят, пишу на Go чуть более пол года) К

сожалению, у меня нет знакомых, которые пишут на Go, да и в целом наставника или тех кто может дать конструктивный фидбек.
К слову писал ТЗ авито, не все доп задания сделал. Мне хотелось бы чтобы вы помогли понять мне мой уровень(если такое возможно) дали в целом вектор направления движения(указали косяки и возможные улучшения в моем проекте)
Я понимаю, что многие из вас уже работают и не хочу отнимать у вас время и все же если у вас будет желание помочь и глянуть тз:
https://github.com/Angstreminus/avito_intern_backend_2023
в целом пока этот проект мой Magnum Opus.
PS: хотел бы я узнать возможно ли с таким уровнем подаваться на стажировку?

#backend #go #programming #russian

0

06.11.2023

13 ответов

25 просмотров

Daniel Podolsky

я вот не вижу ни одного теста. это show stopper, однозначно

0

06.11.2023

Valery Zakharov

для работы с БД можно взять https://github.com/jackc/pgx и посмотреть на https://docs.sqlc.dev/ для генерации запросов туда

0

06.11.2023

Sir Minus Автор вопроса

Valery Zakharov
для работы с БД можно взять https://github.com/ja...

В целом я тоже думал взять что-то из 3rd party, но решил ограничиться gin и стандартом, но да)

0

06.11.2023

Roman Sharkov

интересно почему host address захардкоден а не передаётся через конфиг

0

06.11.2023

Sir Minus Автор вопроса

Roman Sharkov
интересно почему host address захардкоден а не пер...

недоглядел потому что)

0

06.11.2023

Roman Sharkov

Sir Minus
недоглядел потому что)

NewDatabaseHandler вызывает os.Exit(1) что из названия неочевидно. Оно должно либо вернуть ошибку либо добавить префикс Must в имя

0

06.11.2023

Roman Sharkov

https://github.com/Angstreminus/avito_intern_backend_2023/blob/e592ab0ddb6b5207b4600d06f638fa8b4dbb70e2/internal/AppErrors/error.go#L18 почему не сохранить оригинальное значение error вместо Message string? Что если мне интересно понять какая именно это была DB ошибка? Парсить сообщения ошибок это плохо.

0

06.11.2023

Roman Sharkov

кажется у вас тут и data race и бутыльное горлышко: https://github.com/Angstreminus/avito_intern_backend_2023/blob/e592ab0ddb6b5207b4600d06f638fa8b4dbb70e2/internal/repository/segmentsUserRepository.go#L13 что если две горутины (2 параллельных запроса) одновременно захотят выполнить транзакцию?

0

06.11.2023

Аноним

Хехе, авито АБшницу на тестовое дает сделать 😁

0

06.11.2023

Denis

Аноним
Хехе, авито АБшницу на тестовое дает сделать 😁

тестовое на стажера причем

0

06.11.2023

leejoys Joys