Всем привет!
Сделал тестовое в качестве обучения.
Кому не лень проведите ревью, пожалуйста 🙂
https://github.com/zvrvdmtr/advertising

это надо написать больше комментов, что не так и как должно быть, чем там кода есть

Я тут какое-то время назад закидывал опрос для бэкендеров, обещал ответы скинуть. Кому интересно, тут сводка и отдельные ответы в таблице:
https://docs.google.com/forms/d/e/1FAIpQLSf0cXdf_XJKf-Bz6jAtYhdm-H-vxXnfJNBS5WZdrFMgI2wRQw/viewanalytics
https://docs.google.com/spreadsheets/d/1-q76IiQXupMvSyQKfqmlX4Pg0YLxtWVJunnLtDLdXyE/edit?usp=sharing

Добрый день, на Go можно как-то очистить вывод, во время исполнения программы - типа CLS из доса (в IntelliJ Idea)

Спасибо за доклад сегодня на мите, и снято было хорошо:  приятно смотреть и интересно все рассказал.

🙏

в коде программы? или в терминале, который ide запустила для запуска программы?

fmt.Println("\033[2J")

Что делать, если хочется улучшить проект, форкать, а автор забросил проект и не смотрит ни issues, ни pull requests?
Я ему написал, месяц прошел, а ответа не получил

форк свой, да

норма жизни

Fork it 😂

Это не работает в Идее. На https://play.golang.org/  работает такое fmt.Print("\x0c") , но в идее тоже не пашет

https://github.com/zvrvdmtr/advertising/blob/0972b1e59fd747533d9054827af99a7026c6159c/pkg/services/services.go#L26

ошибку лучше сразу проверять

https://github.com/zvrvdmtr/advertising/blob/0972b1e59fd747533d9054827af99a7026c6159c/pkg/services/services.go#L36

тут более идиоматично было бы return nil, err
как и в других местах —  возвращать ссылку на структуру, а не ее целиком

https://github.com/zvrvdmtr/advertising/blob/ceedac2891ff60766fb84dcb580df4132fd70ad7/pkg/api/api.go#L27

 strconv.Atoi(page)

https://github.com/zvrvdmtr/advertising/blob/ceedac2891ff60766fb84dcb580df4132fd70ad7/pkg/api/api.go#L97

не подавлять ошибку, проверять

https://github.com/zvrvdmtr/advertising/blob/719370c2754dd27b42d416f147935a229df49799/cmd/main.go#L22

при повторном запуске с той же базой упадет,  нужен IF NOT EXIST  

handler := handler.NewHandler()

так лучше не делать

хотя бы newHandler:=handler.NewHandler()

https://github.com/zvrvdmtr/advertising/blob/719370c2754dd27b42d416f147935a229df49799/cmd/main.go#L15

проверять ошибку. ну это много где вообще встречается.

http://localhost:8000/ad/8?fields=description&fields=photos

fields=description,photos
выглядит более разумней

https://github.com/zvrvdmtr/advertising/blob/719370c2754dd27b42d416f147935a229df49799/pkg/models/models.go#L53
тут наверное лучше скопировать массив фоток. так тоже прокатит, но грязновато

в целом, не совсем понял зачем аж два с половиной типа на одну и ту же сущность, есть подозрение, что, с учетом  того что выгружается через Scan, можно обойтись одной, но это не точно )

Спасибо!

в Go нет stdlib, есть standard library

ну и число опечаток в readme — это пиздец.

Ну, это я хотя бы знаю, как исправить 😄
Благодарю

Более правильно иметь не /create, а  POST /ad для создания, GET /ad для получения списка, GET /ad/{id} - получение одного объявления

Справедливо
Спасибо!