Loop through many arrays in javascript

Question

I'm trying to make a search through three different arrays of strings in Javascript, looking for a name (submitted by user), and if this name is found, I have to return in which array it is located.

Something like that: HTML

let users = ['mario', 'gianni', 'pinotto'];
let admins = ['moana', 'cicciolina', 'selen'];
let mods = ['frodo', 'sam', 'bilbo'];

const form = document.querySelector('form');
const btnNome = document.querySelector('#nome');
let risp = document.querySelector('#risposta');

function search() {

  risp.innerText = '';
  let nome = btnNome.value.trim();

  for (i = 0; i < mods.length; i++) {
    if (nome == mods[i]) {
      risposta.innerText += `${nome} is a moderator`;
      break;
    } else if (i == users.length - 1) {
      for (i = 0; i < admins.length; i++) {
        if (nome == admins[i]) {
          risposta.innerText += `${nome} is an admin`;
          break;
        } else if (i == users.length - 1) {
          for (i = 0; i < users.length; i++) {
            if (nome == users[i]) {
              risposta.innerText += `${nome} is a registered user`;
              break;
            } else if (i == users.length - 1) {
              risposta.innerText += `${nome} NON è registrato`;
              break;
            }
          }
        }
      }
    }
  }
  form.reset();
};

<form>
  <label for="text">Insert name</label>
  <input id="name" type="text" name="text" required/>
  <input type="button" onClick="search()" value="search">
</form>

BUT it doesn't work, and freezes the browser. I think I have mistaken something creating a infinite loop... any ideas? thanks

THANK you everyone for your answers, which were all very useful.

Can you please fix the indentation. That's absolutely terrible to read and understand... — Andreas
– Andreas, Commented Mar 18, 2020 at 11:35
There is no element with id risposta. Also the variable is risp not risposta — adiga
– adiga, Commented Mar 18, 2020 at 11:37
You have to define a variable in JavaScript with var, let or const otherwise they are global. You should go with let in this case to get a writable variable with block scope. — Andreas
– Andreas, Commented Mar 18, 2020 at 11:39
Does this answer your question? Check if string inside an array javascript — A. Meshu
– A. Meshu, Commented Mar 18, 2020 at 11:45

Alex - Tin Le · Accepted Answer · 2020-03-18 11:44:09Z

0

Try this

let users = ['mario', 'gianni', 'pinotto'];
let admins = ['moana', 'cicciolina', 'selen'];
let mods = ['frodo', 'sam', 'bilbo'];

const form = document.querySelector('form');
const btnNome = document.querySelector('#nome');
let risp = document.querySelector('#risposta');

function search() {

  let searchName = btnNome.value;
  risp.innerHTML = 
    checkWithArray(searchName, 'user', users) +
    checkWithArray(searchName, 'admin', admins) +
    checkWithArray(searchName, 'mod', mods);
  console.log(searchName);

  form.reset();
};

function checkWithArray(searchName, title, arr) {
  if (arr.indexOf(searchName) > -1) {
    return `${searchName} is ${title}. `;
  }
  return '';
}

<form>
  <label for="text">Insert name</label>
  <input id="nome" type="text" name="text" required/>
  <input type="button" onClick="search()" value="search">
  <label id="risposta"></label>
</form>

answered Mar 18, 2020 at 11:44

Alex - Tin Le

2,0101 gold badge8 silver badges11 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Andreas Over a year ago

Why should OP do this? What did you change?

Alex - Tin Le Over a year ago

I just refactor the code to be shorter. Hence less bug.

Siva Kondapi Venkata · Accepted Answer · 2020-03-18 13:13:30Z

0

User Object.entries, find and includes methods

let users = ["mario", "gianni", "pinotto"];
let admins = ["moana", "cicciolina", "selen"];
let mods = ["frodo", "sam", "bilbo"];

const data = { users, admins, mods };

const search = str => {
  const found = Object.entries(data).find(([key, arr]) => arr.includes(str));
  return found ? found[0] : "";
};

const key = search("sam");
const row = data[key];

console.log(`sam found in ${key} and array is ${row}`);

answered Mar 18, 2020 at 13:13

Siva Kondapi Venkata

11.1k2 gold badges20 silver badges32 bronze badges

Comments

Sushmit Sagar · Accepted Answer · 2020-03-19 08:23:54Z

You can use Array's includes method to find if element exist in an array:

let users = ["mario", "gianni", "pinotto"];
let admins = ["moana", "cicciolina", "selen"];
let mods = ["frodo", "sam", "bilbo"];

const form = document.querySelector("form");
const btnNome = document.querySelector("#nome");
let risp = document.querySelector("#risposta");

function search() {
  risp.innerText = "";
  let nome = btnNome.value.trim();

  checkWithArray(nome, "user", users);
  checkWithArray(nome, "admin", admins);
  checkWithArray(nome, "moderator", mods);

  form.reset();
}

function checkWithArray(searchName, title, arr) {
  const isFound = arr.includes(searchName);

  if (isFound) {
    let text;

    switch (title) {
      case "user":
        text = `${searchName} is a registered user`;
        break;
      case "admin":
        text = `${searchName} is an admin`;
        break;
      case "moderator":
        text = `${searchName} is a moderator`;
        break;
    }

    risp.innerText += text;
  } else {
    risp.innerText += `${searchName} NON è registrato`;
  }
}

Collectives™ on Stack Overflow

Loop through many arrays in javascript

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related