Shorter way to check that value of multiple input is empty for jquery

Question

let a = $('#plustitle').val().trim();  // input type text
let b = $('#plustags').val().trim(); // input type text
let c = $('#plustory').val().trim(); // textarea

I need to check if any of the above variable is empty, i.e. have value "";

Using jquery each loop - there is a lot of code.

Is there a way to do it in a shorter way.

Sorry, you've made me insecure about my logic but it was correct^^ !a || !b is the same as !(a && b) and therefore if(!(a && b && c)) — Andreas
– Andreas, Commented Dec 1, 2018 at 18:15
@Andreas, I see, pls place the answer, I think there is no shorter way — user7461846
– user7461846, Commented Dec 1, 2018 at 18:19

Andreas · Accepted Answer · 2018-12-01 18:37:40Z

1

If we use the fact that an empty string will be falsy we could achieve your requirement with

if (!(a && b && c)) {
  // one of them is empty
}

answered Dec 1, 2018 at 18:37

Andreas

21.9k7 gold badges52 silver badges58 bronze badges

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

Comments

Ramesh · Accepted Answer · 2018-12-01 18:06:49Z

0

considering "let" is being used, I assume you have ES6 support. You can use below code after pushing these values to an array:

   let a = $('#plustitle').val().trim();  // input type text
   let b = $('#plustags').val().trim(); // input type text
   let c = $('#plustory').val().trim(); // textarea 
   let someValues = [a, b, c]; // if separate variables are not required, directly push to array   
   someValues.forEach(x =>{ if(!x) {
       //do operation when empty
   }});

answered Dec 1, 2018 at 18:06

Ramesh

6393 gold badges8 silver badges17 bronze badges

2 Comments

Andreas Over a year ago

if ([a, b, c].some(x => !x)) {...}

Mohammad Over a year ago

OP doesn't want to use loop as mentioned

Mohammad · Accepted Answer · 2018-12-01 19:00:55Z

0

Join all variables and check length of result

let a = $('#plustitle').val().trim();
let b = $('#plustags').val().trim();
let c = $('#plustory').val().trim(); 

if ((a+b+c).length == 0)
  console.log("empty");

<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
<input type="text" id="plustitle" value="">
<input type="text" id="plustags" value="">
<input type="text" id="plustory" value="">

Also you can simplify your code and use one selector

var values = $('#plustitle, #plustags, #plustory').map(
  (i, ele) => ele.value.trim()
).toArray().join('');

if (values.length == 0)
  console.log("empty");

var values = $('#plustitle, #plustags, #plustory').map(
  (i, ele) => ele.value.trim()
).toArray().join('');

if (values.length == 0)
  console.log("empty");

<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
<input type="text" id="plustitle" value="">
<input type="text" id="plustags" value="">
<input type="text" id="plustory" value="">

edited Dec 1, 2018 at 19:00

answered Dec 1, 2018 at 18:50

Mohammad

21.5k16 gold badges58 silver badges86 bronze badges

1 Comment

Andreas Over a year ago

"Also you can simplify your code" - Only if puerto doesn't need the values. Otherwise you would have to get them twice from the DOM

Nico Haase · Accepted Answer · 2018-12-02 17:30:33Z

0

You could abstract it the following way:

function isEmpty(id) {
  return ($('#' + id).val().trim() == '');
}

if(!isEmpty('plusTitle') && !isEmpty('plustags') && !isEmpty('plustory')) {
  console.log('none is empty');
}

edited Dec 2, 2018 at 17:30

answered Dec 1, 2018 at 17:52

Nico Haase

12.1k39 gold badges48 silver badges96 bronze badges

1 Comment

charlietfl Over a year ago

Not using comparison operator

Collectives™ on Stack Overflow

Shorter way to check that value of multiple input is empty for jquery

4 Answers 4

Comments

2 Comments

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

2 Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related