[Bug] aula Corrigindo erro ao filtrar por companhias

Oi

Isso ocorre devido à maneira como o método converterParametrosParaString está tratando o parâmetro companhiasId. Quando você seleciona duas companhias, ele está gerando duas entradas companhiasId na string de consulta, o que não é o comportamento desejado.

Para resolver esse problema, você pode ajustar o método converterParametrosParaString para tratar o parâmetro companhiasId como um array e criar a string de consulta corretamente. Aqui está como você pode fazer isso:

converterParametrosParaString(busca: DadosDeBusca) {
  const { companhiasId, ...rest } = busca; // Separar companhiasId do restante dos parâmetros

  const query = Object.entries({ ...rest, companhiasId: companhiasId.join(',') }) // Juntar as companhiasId em uma única entrada
    .map(([key, value]) => {
      if (!value) {
        return '';
      }
      return `${key}=${value}`;
    })
    .join('&');
  return query;
}

Assim separando o companhiasId do restante dos parâmetros, juntando os IDs das companhias em uma única entrada (separados por vírgulas) e depois continuando a criar a string de consulta como antes.

Isso deve resolver o problema de passar companhiasId duas vezes na consulta quando você seleciona duas companhias diferentes. Tenha certeza que ao chamar a função converterParametrosParaString, o parâmetro companhiasId seja um array quando houver mais de uma companhia selecionada.

Valeu!

Olá Ariel!

Nosso colega deu uma ótima dica, acredito que ela possa te ajudar!

No mais deixo o código da aula para fins de revisão :

• Caso precise alinhar seu código com o do instrutor

Reforço que em casos de dúvidas, conte sempre com o fórum da comunidade Alura! Bons estudos!

Sucesso ✨

Um grande abraço e até mais!

Oi! tentei resolver dessa forma, mas nao obtive sucesso

  converterParametrosParaString(busca: DadosDeBusca) {
    const { companhiasId, ...rest } = busca; // Separar companhiasId do restante dos parâmetros

    const query = Object.entries({
      ...rest,
      companhiasId: companhiasId?.join(','),
    }) // Juntar as companhiasId em uma única entrada
      .map(([key, value]) => {
        if (!value) {
          return '';
        }
        return `${key}=${value}`;
      })
      .join('&');
    return query;
  }

sobre o seu comentário

 Tenha certeza que ao chamar a função converterParametrosParaString, o parâmetro companhiasId seja um array quando houver mais de uma companhia selecionada.

eu pensei isso também, mas não consegui descobrir como fazer isso, o que me fez não avançar nada do estudo hj :'(

Oi Victor, tudo bem? agradeço por me enviar o repositório, mas eu já havia dado uma olhada nele ele mostra no fim do código dele, tirando as aulas de ajuste de valor, o seguinte código:

  converterParametroParaString(busca: DadosBusca){
    const query = Object.entries(busca)
      .map( ([key, value]) => {
        if(!value){
          return ''
        }
        return `${key}=${value}`
      })
      .join('&')
      return query
  }

que é o mesmo que o meu que postei na abertura desse post

converterParametrosParaString(busca: DadosDeBusca) {
    const query = Object.entries(busca)
      .map(([key, value]) => {
        if (!value) {
          return '';
        }
        return `${key}=${value}`;
      })
      .join('&');
    return query;
  }

Eu encontrei o meu erro, foi falta de atenção em ter feito um desafio e ter criado um service, por falta de tempo, qnd voltei, nao lembrava o q tinha feito e fiz o desafio mas copiando o código pronto por precisar andar com o projeto acontece q existiam dois lugares que chamavam a função buscar, e a forma q estava fazendo nao era pelo passagens.service.ts

deu tudo certo agora, foi falta de atenção mesmo, como 98% das vezes kkk