Alterar Search para utilizar NamedExec

[lucassabreu]

Hoje o método Search esta mondando a query e manipulando os parâmetros de forma bastante manual. Para diminuir a complexidade do método pode ser utilizado o sqlx.Named ou NamedQuery para diminuir a complexidade da passagem de parâmtros.

vitrine-social/server/db/repo/search.go

Line 27 in f24e129

func (r *SearchRepository) Search(text string, categoriesID []int, organizationsID int64, status string, orderBy string, order string, page int) (needs []model.SearchNeed, count int, err error) {

https://godoc.org/github.com/jmoiron/sqlx#Named
https://godoc.org/github.com/jmoiron/sqlx#Tx.NamedQuery

[marcosinger]

@lucassabreu olhando essa função ela tanto monta o SQL quanto executa ele. Olhando a task eu tinha pensando em separar isso em duas, mesmo porque fica mais fácil testar se o SQL gerado ( com NamedExec ou plain está correto ).

Pensei em fazer assim:

1. Criar uma função (e.g. generateSearchSQL) que só faz a montagem do SQL;
2. Criar testes para essa função;
3. Escrever testes para (r *SearchRepository) Search
4. Migrar a geração do SQL no Search para a generateSearchSQL

Que acha da ideia?

[lucassabreu]

a ideia dessa tarefa é melhorar a legibilidade e facilitar a manutenção, acho que quebrar em duas funções e fazer os testes unitários deve ajudar nesse sentido

pensei de usar as funções Named para ajudar a entender melhor onde estão indo parar os parâmetros

então acho que podemos ir por isso caminho que falou

[marcosinger]

vou começar a dar uma olhada então no final do dia e devo ter algo amanhã para vc dar um primeiro feedback :)