[WIP] Add QR-generator #1
Conversation
TOsmanov
force-pushed
the
add-qr-gen
branch
2 times, most recently
from
d1d9972 to
9c98986
Compare
TOsmanov
force-pushed
the
add-qr-gen
branch
2 times, most recently
from
54eda0e to
aa58aba
Compare
TOsmanov
force-pushed
the
add-qr-gen
branch
2 times, most recently
from
e1f4fac to
c8145a6
Compare
vladislavmass
left a comment
vladislavmass
left a comment
There was a problem hiding this comment.
Хорошая работа. Принято. Проект действительно теперь работает без явных паник и просто ошибок. Из минусов проекта могу отметить следующее
1)Нет четкого разделения на слои. По сути все работа идет в веб хэндлерах
2)Нет интеграционных тестов
3) Используется шаблонизатора, хотя он тут не нужен
| func Index(log *slog.Logger, w http.ResponseWriter, | ||
| _ *http.Request, | ||
| ) { | ||
| tpl := template.Must(template.ParseFiles("site/index.html")) |
There was a problem hiding this comment.
Два вопроса по этому месту. Почему мы с этим работаем как с шаблоном, а не со статикой? И почему мы на каждый запрос читаем файл, а не прочитали его один раз и положили в мапу?
| r *http.Request, | ||
| ) { | ||
| const op = "handlers.OrderHandler.GetPreview" | ||
| buf, err := os.ReadFile("site/preview.jpg") |
| log.Info("Starting server", slog.String("address", srv.Addr)) | ||
|
|
||
| go func() { | ||
| if err = srv.ListenAndServe(); err != nil { |
There was a problem hiding this comment.
Подумайте что произойдет если тут будет ошибка. По текущей логике когда ошибка выведится потом будет написано что севере успешно запущен и после этого будет идти ожидание сигналы от системы. Обычно при ошибке дергается cancel
| func Index(log *slog.Logger, w http.ResponseWriter, | ||
| _ *http.Request, cfg *config.Config, | ||
| ) { | ||
| tpl := template.Must(template.ParseFiles(cfg.MainPage)) |
There was a problem hiding this comment.
По сути это функционал сервера статики. Такая обработка имеет смысл только если мы хотим динамический заполнять шаблон
For use, run:
after open in browser http://127.0.0.1:10001