2

Estou fazendo, com google script, um sistema bem simples que monitora o andamento de atividades do dia a dia, tenho uma tabela com as atividades solicitadas, uma tabela com as interações e uma tabela com o status (aberto ou fechado). preciso comparar a lista com os ID's da atividades solicitadas com a lista com os ID's da atividades fechadas. Os ID's que sobrarem eu assumo que estão abertos. Obs.: sou iniciante em programação, podem me corrigir a vontade!

function arrayabertas(){
var todos = [08072020105842, 08072020110953, 8072020104528];
var fechado = [8072020104528];
var abertos = [];

// Verifica se cada número jogado
// está na lista dos sorteados
for(var i=0; i<fechado.length; i++) {
    if(todos.indexOf(fechado[i]) > -1) {
        abertos.push(fechado[i]);
    }
}
    return(abertos);
}

4
  • 1
    Júlio, vc não pode ficar mudando a pergunta porque isso invalida as respostas já existentes. Se tem outra pergunta, por favor faça-a aqui. E eu sugiro que nessa nova pergunta vc coloque um exemplo da planilha para que as pessoas possam testar (pois só com imagens não dá)
    – hkotsubo
    Commented 13/07/2020 às 13:50
  • 1
    E se uma das respostas resolveu o seu problema (o problema específico que está na sua pergunta, da forma que vc colocou, não o problema da planilha), vc pode escolher a que melhor resolveu e aceitá-la, veja aqui como e porque fazê-lo. Não é obrigatório, mas é uma boa prática do site, para indicar a futuros visitantes que aquilo resolveu o problema. Não esqueça que vc também pode votar em todas as respostas que achou úteis.
    – hkotsubo
    Commented 13/07/2020 às 13:52
  • 1
    Não leve a mal, é que a ideia do site é ter um problema específico por pergunta. Se tiver outra pergunta (mesmo que esteja relacionada), então o ideal é fazer outra. Ficar mudando a pergunta vai contra a ideia do site e deixa-o mais desorganizado: pt.meta.stackoverflow.com/a/1684/112052
    – hkotsubo
    Commented 13/07/2020 às 13:56
  • 1
    Ok, tudo bem, eu agradeço pelos comentários e ajuda, eu ainda sou iniciante nisso, vou me atentar na forma de usar o site, obrigaduuu! Commented 13/07/2020 às 14:01

3 Respostas 3

3

Se já estiver usando ambientes compatíveis com ECMAScript 2015, uma alternativa é usar Set, que já possui bom suporte dos browsers atuais.

O Set garante que os elementos dele serão únicos (que é um dos seus requisitos), e também possui algumas operações que facilitam o algoritmo para o que você precisa (além de ser mais eficiente, como veremos ao final).

Por exemplo, para o seu caso, bastaria criar um Set com todos os valores e ir removendo os que estão fechados. O que sobrar são os itens abertos:

let todos = [ 08072020105842, 08072020110953, 8072020104528 ];
let fechado = [ 8072020104528 ];

let abertos = new Set(todos);
for (let e of fechado) { // remove todos os fechados
    abertos.delete(e);
}

// converter para array
abertos = Array.from(abertos);
console.log(abertos);


As outras respostas sugerem usar indexOf ou includes, que de fato funcionam (exceto que com includes você deveria verificar quais elementos não estão em fechado, ou seja, abertos = todos.filter(x => ! fechado.includes(x))), mas são mais ineficientes porque tanto indexOf quanto includes percorrem o array todos desde o início.

Ou seja, para cada elemento de fechado, você tem que percorrer o array todos desde o início, até encontrar o elemento. E quando você percorre o mesmo array várias e várias vezes, está criando uma variação do chamado Shlemiel the painter’s algorithm, que é extremamente ineficiente.

Claro que para arrays pequenos - como é o seu caso - a diferença será insignificante e talvez isso não passe de micro-otimização. Mas para arrays maiores a diferença é significativa.

Obviamente que, se isso for rodar em browsers antigos que não suportam Set, aí o jeito é usar indexOf ou includes mesmo.


Outro detalhe é que, se você está usando uma função para verificar os itens abertos, é interessante receber os arrays todos e fechado como parâmetros, pois assim a função fica mais genérica e reusável, podendo receber quaisquer arrays (da forma que você fez, ela só funciona para os arrays informados). Ficaria assim:

function verificarAbertos(todos, fechado){
    let abertos = new Set(todos);
    for (let e of fechado) {
        abertos.delete(e);
    }
    
    // converter para array
    return Array.from(abertos);
}

let todos = [08072020105842, 08072020110953, 8072020104528];
let fechado = [8072020104528];
let abertos = verificarAbertos(todos, fechado);
console.log(abertos);

// posso passar outros arrays também
console.log(verificarAbertos([1,2,3,4,5,6], [3,4,5])); // [1, 2, 6]

2

Utilize filter com includes, onde esse filtro só será satisfatório quando tiver contido o valor testado, exemplo:

var todos = [08072020105842, 08072020110953, 8072020104528];
var fechado = [8072020104528];
var abertos = todos.filter(x => fechado.includes(x)); 

console.log(abertos);

Referencias:

1
  • 1
    A resposta mais simples e pratica sem fazer muito malabarismo com os arrays, única coisa pra ajustar seria negar fechado.includes(x) visto que abertos deve conter os elementos que não existem em fechado, de resto perfeito.
    – Leo Letto
    Commented 13/07/2020 às 13:54
1

Seu algoritmo está errado, ele verifica quais elementos em todos também estão em fechados, quando na verdade deveria verificar quais NÃO estão em fechados.

function arrayabertas(){
  var todos = [08072020105842, 08072020110953, 8072020104528];
  var fechado = [8072020104528];
  var abertos = [];

  // Verifica se cada número jogado
  // está na lista dos sorteados
  for(var i=0; i<todos.length; i++) {
    // verifica qual em "todos" não está fechado
    if(fechado.indexOf(todos[i]) === -1) {
      abertos.push(todos[i]);
    }
  }
  return(abertos);
}

Você pode utilizar a função filter para o código ficar mais limpo

function arrayabertas(){
  var todos = [08072020105842, 08072020110953, 8072020104528];
  var fechado = [8072020104528];
  var abertos = [];

  abertos = todos.filter((x) => fechado.indexOf(x) === -1)

  return(abertos);
}

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .