Iterate and checking an array with for of loop and if..else statement

Question

I am trying to make an array with names, then to iterate them using a for of loop and then to check the iteration using a if else statement. My code is below, but it did not work, I saw no result. I tried with for in, the same bad result. In Html file I have a paraghraph with demo id and a button with function for iterate and checking array

function availableName() {

 const check = document.getElementById('userID');
 const takenNames=['manuel','remus','manuel-remus','tomi','mirel'];
 const reservedNames=['cristina','angela','rusanda','costantin'];
 const givenNames=[...takenNames, ...reservedNames];

  for (givenName of givenNames) {
  if (givenName === check) {
      sayMessage=givenName +"Not available!";
 }
 else {
   sayMessage= givenName +"Available";
 }

 }
document.getElementById("available").innerHTML= sayMessage;
 };

it document.getElementById('userID') should be document.getElementById('userID').value if userId is an input — Sagar Jajoriya
– Sagar Jajoriya, Commented Mar 26, 2018 at 16:16
Try if ( givenName === check.value ), as check is actually a dom element not the value of element itself. — palaѕн
– palaѕн, Commented Mar 26, 2018 at 16:17
the question is not clear.. give js fiddle demo, what exactly is givenNames? what is desired output of sayMessage for above example. I have created sample js fiddle demo here:- jsfiddle.net/rbb1af2o/8 Check and modify this — Sahil Sharma
– Sahil Sharma, Commented Mar 26, 2018 at 16:38
@maverick givenNames is a combination of both takenNames and resrevedNames. — cнŝdk
– cнŝdk, Commented Mar 26, 2018 at 16:43
Check my fiddle- jsfiddle.net/rbb1af2o/8 I am able to print it — Sahil Sharma
– Sahil Sharma, Commented Mar 26, 2018 at 16:51

cнŝdk · Accepted Answer · 2018-03-26 16:35:12Z

First of all you need to get the value of the inputusing .value as stated in comments.

Then to check its existence in the array you can use one of Array's built-in methods, such as includes() or indexOf().

This is how should be your code:

function availableName() {
  const check = document.getElementById('userID').value;
  const takenNames = ['manuel', 'remus', 'manuel-remus', 'tomi', 'mirel'];
  const reservedNames = ['cristina', 'angela', 'rusanda', 'costantin'];
  const givenNames = [...takenNames, ...reservedNames];

  if (givenNames.indexOf(check) > 0) {
    sayMessage = check + " Not available!";
  } else {
    sayMessage = check + " Available";
  }
  document.getElementById("available").innerHTML = sayMessage;
};

Demo:

This is a Demo:

function availableName() {
  //const check = document.getElementById('userID').value;
  const check = "cristina";
  const takenNames = ['manuel', 'remus', 'manuel-remus', 'tomi', 'mirel'];
  const reservedNames = ['cristina', 'angela', 'rusanda', 'costantin'];
  const givenNames = [...takenNames, ...reservedNames];

  if (givenNames.indexOf(check) > 0) {
    sayMessage = check + " Not available!";
  } else {
    sayMessage = check + " Available";
  }
  document.getElementById("available").innerHTML = sayMessage;
};

availableName();

<div id="available"></div>

Eduardo Carvajal · Accepted Answer · 2018-03-26 16:22:24Z

1

Why not use the function isInArray?

function isInArray(value, array) {
    return array.indexOf(value) > -1;
}

function availableName() {
const check = document.getElementById('userID');
 const takenNames=['manuel','remus','manuel-remus','tomi','mirel'];
 const reservedNames=['cristina','angela','rusanda','costantin'];
 const givenNames=[...takenNames, ...reservedNames];
for (givenName of givenNames) {
  if (isInArray(check, givenNames)) {
      sayMessage=givenName +"Not available!";
 }
 else {
   sayMessage= givenName +"Available";
 }

 }
document.getElementById("available").innerHTML= sayMessage;
 };

answered Mar 26, 2018 at 16:22

Eduardo Carvajal

615 bronze badges

Collectives™ on Stack Overflow

Iterate and checking an array with for of loop and if..else statement

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related