How to check entered value is exist or not using Jquery or javascript

Question

I have one textbox and one button and on button I have written below code. problem is suppose first I have entered in textbox 10 than its worked but when another time I enter 10 than also it prints value is not in array. so pls help me whats the issue...

jQuery(document).ready(function()
{
 jQuery("#mybutton").live('click',function () 
 {
    var sel_fam_rel=jQuery("#my_textbox").val();
    var ids = [];
    code =sel_fam_rel;
    if($.inArray(code,ids) >= 0)
    {
      alert("Value is in array");
    }
    else
    {
      alert("Value is not in array");
      ids.push(code);
    }
  });
});

You are re-initializing ids, So it will always go to else section. Show us your compete JS code. — Satpal
– Satpal, Commented Mar 25, 2015 at 6:25

Jai · Accepted Answer · 2015-03-25 06:34:24Z

2

This line:

if($.inArray(code,ids) >= 0)

should be changed to:

if($.inArray(code,ids) != -1)

and put your ids var outside of click.

Try the snippet below.

var ids = [];
jQuery("button").on('click', function() {
  var sel_fam_rel = jQuery("#my_textbox").val();

  code = sel_fam_rel;
  if ($.inArray(code, ids) != -1) {
    alert("Value is in array");
  } else {
    alert("Value is not in array");
    ids.push(code);
  }
});

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<input type='text' id='my_textbox'>
<button>check</button>

answered Mar 25, 2015 at 6:34

Jai

74.8k12 gold badges77 silver badges104 bronze badges

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

4 Comments

charlietfl Over a year ago

What difference does it make if it is >=0 or !=-1?

Wesley Smith Over a year ago

@charlietfl !=-1 , seems to better convey the intent to me, but good point

charlietfl Over a year ago

@DelightedD0D that is purely subjective. The only difference here that matters is storing the array outside the click handler

Wesley Smith Over a year ago

Should the OP not also be encouraged to use on() or .delegate() over live()?

Navneeth · Accepted Answer · 2015-03-25 06:34:19Z

1

Create your array var ids=[];global outside button event, as whenever you click button it is creating new empty array. It will fix your problem.

answered Mar 25, 2015 at 6:34

Navneeth

9883 gold badges11 silver badges29 bronze badges

Comments

Wesley Smith · Accepted Answer · 2015-03-25 06:47:33Z

A few changes are needed:

var ids = []; // `ids` needs to be in the global scope to work as you want it, 
              //  or you could use a different method like localstorage 
jQuery(document).ready(function()
{
 jQuery("#mybutton").on('click',function () // use `on` not `live` which is deprecated
 {
    var sel_fam_rel=jQuery("#my_textbox").val();
    code =sel_fam_rel;
    if($.inArray(code,ids) != -1)  // inArray() returns -1 if the value is not in the array, you can use it the way you have it, IMO (purely subjective), using `!=-1` is preferable as it's more clear what the code in intend to do
    {
      alert("Value is in array");
    }
    else
    {
      alert("Value is not in array");
      ids.push(code);
    }
  });
});

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<input type="text" id="my_textbox" value="10"/><br>
<input type="button" id="mybutton" value="Click me"/>

Nishit Maheta · Accepted Answer · 2015-03-25 07:19:28Z

1

use below code . take your ids out side of click event . as per your code each time when you click button ids reset .

var ids = [];  // declare as global variable
jQuery(document).ready(function()
{
  jQuery("#mybutton").live('click',function () 
  {
    var sel_fam_rel=jQuery("#my_textbox").val();
    code =sel_fam_rel;
    if($.inArray(code,ids) >= 0)
    {
     alert("Value is in array");
    }
   else
   {
     alert("Value is not in array");
     ids.push(code);
    }
 });
});

edited Mar 25, 2015 at 7:19

answered Mar 25, 2015 at 6:33

Nishit Maheta

6,0413 gold badges20 silver badges32 bronze badges

3 Comments

Wesley Smith Over a year ago

shouldn't the OP be encouraged to use on() over live()?

Mahendra Pumbhadiya Over a year ago

thanks... in above code if i change this line of code if($.inArray(code,ids) != -1) instead of if($.inArray(code,ids) >= 0) than only it will be worked..

Nishit Maheta Over a year ago

@MahendraPumbhadiya its not like you change it >= 0 to != -1 this both condition is same. it work because you put ids = [] out side of click event .

Manoz · Accepted Answer · 2015-03-25 06:36:50Z

-1

I made a fiddle to your problem, Use indexOf

http://jsfiddle.net/go8o34fq/

jQuery-

var array=["A","B","C","D"];

$('button').click(function(){
    var code=$('input').val();
    if(array.indexOf(code)==-1)
    {
        array.push(code);
       console.log("if "+array)
    }
    else
    {
      console.log("else "+array)
    }
});

Just a bit requirement if your need is case-sensitive then use- code.toUpperCase()

answered Mar 25, 2015 at 6:36

Manoz

6,62713 gold badges72 silver badges117 bronze badges

1 Comment

Wesley Smith Over a year ago

-1 because you tell the OP to use indexOf (which is not really the issue). and while you do change the scope of the array (which was the problem), you dont mention that in your answer

Collectives™ on Stack Overflow

How to check entered value is exist or not using Jquery or javascript

5 Answers 5

4 Comments

Comments

Comments

3 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

4 Comments

Comments

Comments

3 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related