Entrar Ainda não tem acesso?
1
resposta

[Bug] Função sortear não está funcionando e não sei o que tem de errado no código.

Referente ao curso Lógica de programação: praticando com desafios, no capítulo Projeto Amigo Secreto e atividade Realizando o sorteio

por Estudante
| 11.7k xp | 1 posts
  1. let amigos = [];
  3. function adicionar() {
  4. let amigo = document.getElementById('nome-amigo');

  5. let lista = document.getElementById('lista-amigos');

  7. amigos.push(amigo.value);

  9. if (lista.textContent == '') {

  10.     lista.textContent = amigo.value;

  11. } else {

  12.     lista.textContent = lista.textContent + ', ' + amigo.value; 

  13. }

  15. limparInput();
  16. }
  18. function sortear() {
  19. embaralha(amigos);

  20. let sorteio = document.getElementById('lista-sorteio');

  22. for (let i = 0; i > amigos.length; i++) {

  23.     if (i > amigos.length - 1) {

  24.         sorteio.innerHTML = sorteio.innerHTML + amigos[i] + ' --> ' + amigos[0] + '<br>';

  25.     } else {

  26.         sorteio.innerHTML = sorteio.innerHTML + amigos[i] + ' --> ' + amigos[i + 1] + '<br>';

  27.     }

  28. }
  29. }
  31. function embaralha(lista) {
  33. for (let indice = lista.length; indice; indice--) {

  35.     const indiceAleatorio = Math.floor(Math.random() * indice);

  37.     // atribuição via destructuring

  38.     [lista[indice - 1], lista[indiceAleatorio]] = 

  39.         [lista[indiceAleatorio], lista[indice - 1]];

  40. }
  41. }
  43. function limparInput() {
  44. document.getElementById('nome-amigo').value = '';
  45. }
1 resposta
por Rodrigo da Silva Ferreira Caneppele
| 5101.6k xp | 8948 posts
Instrutor Instrutor

Oi! Vamos dar uma olhada no seu código para ver o que pode estar acontecendo com a função de sorteio. Pelo que você descreveu, parece que o problema está na lógica do loop for dentro da função sortear.

No seu código atual, o loop for está assim:

for (let i = 0; i > amigos.length; i++) {
    if (i > amigos.length - 1) {
        sorteio.innerHTML = sorteio.innerHTML + amigos[i] + ' --> ' + amigos[0] + '<br>';
    } else {
        sorteio.innerHTML = sorteio.innerHTML + amigos[i] + ' --> ' + amigos[i + 1] + '<br>';
    }
}

Aqui, o problema está na condição do loop for. Você está usando i > amigos.length, mas deveria ser i < amigos.length. Isso está impedindo o loop de executar, pois i começa em 0, que não é maior que o comprimento da lista.

Além disso, a condição if (i > amigos.length - 1) também está incorreta. Deveria ser if (i == amigos.length - 1), para verificar se i é o último índice do array.

Aqui está a correção:

for (let i = 0; i < amigos.length; i++) {
    if (i == amigos.length - 1) {
        sorteio.innerHTML = sorteio.innerHTML + amigos[i] + ' --> ' + amigos[0] + '<br>';
    } else {
        sorteio.innerHTML = sorteio.innerHTML + amigos[i] + ' --> ' + amigos[i + 1] + '<br>';
    }
}

Com essas alterações, o loop deve funcionar corretamente, percorrendo todos os elementos da lista amigos e exibindo o sorteio de acordo com a lógica esperada.

Espero ter ajudado e bons estudos!