Javascript check if string if true or false and convert to Boolean

Question

Is there a better way to improve the below statement to check if the val() is 'true' or 'false' and if it is then it will change it to a Boolean. The reason being, some of the values may be a word.

var thisval = $(this).val();
if (thisval == "true"){
    ao[id] = true;
} else if (thisval == "false"){
    ao[id] = false;
} else {
    ao[id] = $(this).val();
}

possible duplicate of [How can I convert a string to boolean in JavaScript?](http://stackoverflow.com/questions/263965/how-can-i-convert-a-string-to-boolean-in-javascript) — KevSheedy, Nov 11 '13 at 13:31
Whats wrong with this code? All answers so far, in my opinion, are less readable. Extract into a function (accepting a string, returning a boolean or string) and never worry about it again. — Caramiriel, Jan 08 '19 at 09:58

Tibos · Accepted Answer · 2016-11-29T16:46:09.907

21

Most readable:

var thisval = $(this).val();
ao[id] = thisval === 'true' ? true : 
         thisval === 'false' ? false : 
         thisval;

One-liner based on the conditional operator:

var thisval = $(this).val();
ao[id] = thisval === 'true' ? true : (thisval === 'false' ? false : thisval);

One-liner based on || and && behavior:

var thisval = $(this).val();
ao[id] = thisval === 'true' || (thisval !== 'false') && thisval || false;

Shortest one-liner (combination of the above):

var thisval = $(this).val();
ao[id] = thisval === 'true' || (thisval === 'false' ? false : thisval);

edited Nov 29 '16 at 16:46

answered Nov 11 '13 at 13:31

Tibos

26,984
4
46
61

Third solution is the best to me because, even in a case where we don't know for sure that ```thisval``` is a string , at the end, if not equal to ```true``` or ```false```, we can assume ```thisval``` is probably already a **Boolean** and if it's not it wont pass further test anyway. – svassr Feb 07 '14 at 20:39
@svassr All 3 solutions behave the same (give exactly the same output for any input). The only difference is in what JS operators are used. I believe that with the proper optimizations they might even result in the same bytecode. – Tibos Feb 07 '14 at 21:50
Right indeed. I understood that, I was just confused because I was reading 3 posts on the topic at the same time :p – svassr Feb 10 '14 at 15:06
@Gaspa79 The code is equivalent to that of the OP - values other than 'true' and 'false' are returned as they are. The conditional operator does not enforce anything about the type of the result. Run the code in the console to convince yourself :D. – Tibos Nov 29 '16 at 16:48
@Tibos I stand corrected. The ? operator doesn't work the same way it does in strongly typed languages it seems. I did well in giving you +1 before. Thanks. – Gaspa79 Nov 29 '16 at 20:06

Khanh TO · Answer 2 · 2013-11-11T13:49:29.400

9

Try JSON.parse().

"true" and "false" are actually json representations of true, false. This is how ajax parses json object as a string from server side. If on server side, we return true, false => the browser will receive it as a string "true" or "false" (json representation)

if ( $(this).val() == "true" ||  $(this).val() == "false") {
    ao[id] = JSON.parse($(this).val());
}else {
    ao[id] = $(this).val();
}

DEMO

edited Nov 11 '13 at 13:49

answered Nov 11 '13 at 13:42

Khanh TO

47,821
12
98
113

Using JSON just for parsing something you know can only be either `"true"` or `"false"`sounds like quite an overkill... also, the `.val()` method wil be called three times every time the code above is called. – Haroldo_OK Nov 29 '16 at 16:56
1

@Haroldo_OK: I don't think it's overkill, that's also something we use when we parse response from server. – Khanh TO Dec 01 '16 at 13:27
this also works for numbers, eg. JSON.parse('123') // returns 123 – Alex Feb 11 '22 at 21:23

score 3 · Answer 3 · answered Nov 11 '13 at 13:32

3

String.prototype.bool = function() {
    return (/^true$/i).test(this);
};


if ( $(this).val() == "true" ||  $(this).val() == "false") {
    ao[id] = $(this).val().bool();
}else {
    ao[id] = $(this).val();
}

answered Nov 11 '13 at 13:32

Ankit Tyagi

2,341
9
19

score 1 · Answer 4 · answered Nov 11 '13 at 13:30

1

This might be slightly more elegant:

var thisval = $(this).val();

if (thisval === "true" || thisval === "false") {
    thisval = (thisval === "true");
}

answered Nov 11 '13 at 13:30

Strille

5,556
2
23
39

msanjay · Answer 5 · 2021-09-01T11:28:25.977

('' + thisval).toLowerCase() === 'true'

May be useful if extra string and toLower is negligible - may be simple.

Its true if its explicitly set to true, (boolean or case insensitive string) and false otherwise.

 // so complete solution to get the same value if neither true nor false
const thisvalStr = ('' + thisval).toLowerCase();
a[id] = thisvalStr === 'true'? true: (thisvalStr === 'false')? false: thisval;

Can be modified to include additional values like, 1, etc as per need by using || or an array.

const TrueValuesStr = ['1', 'true']

TrueValueStr.contains(('' + thisval).toLowerCase() === 'true')

vladzam · Answer 6 · 2013-11-11T13:50:40.070

0

var thisval = $(this).val();

switch(thisval) { 
case 'true' : 
    ao[id] = true; 
    break; 

case 'false' : 
    ao[id] = false; 
    break; 

default: 
    ao[id] = thisval; 

}

I would create an array to hold the two possible values for when the string is true or false. Then, I would search to see if thisval is found inside that array. If it is not, the indexOf function will return -1. Afterwards, it is just a matter of going over those values and build up the cases that you want.

EDIT: I have updated the code based on the suggestion of Tibos. Thank you!

edited Nov 11 '13 at 13:50

answered Nov 11 '13 at 13:31

vladzam

5,057
6
28
35

1

The OP did ask for a way to improve his code, where is the improvement? – Mark Walters Nov 11 '13 at 13:34
Usually, when there are more than 2 if statements, switch shows better performance. That is the only thing I kept in mind when providing my answer. I thought that by 'improved way' the asker meant performance management :) – vladzam Nov 11 '13 at 13:36
The improvement in performance for this would be minimal and not noticeable. Caching his `$(this).val()` would be useful in any case, but this code just seems more longwinded in my opinion. – Mark Walters Nov 11 '13 at 13:42
Indexof negates all the performance benefit from going to a switch instead of two ifs. Why not `switch(thisval) { case 'true' : ao[id] = true; break; case 'false' : ao[id] = false; break; default: ao[id] = thisval; }`? – Tibos Nov 11 '13 at 13:45
You are right, Tibos! That would be faster indeed! I will update the code accordingly. – vladzam Nov 11 '13 at 13:49

score 0 · Answer 7 · answered Jan 08 '19 at 09:51

In nodejs by using node-boolify we can use isBoolean() & Boolify()

Validation Results

var isBoolean = require('node-boolify').isBoolean;

isBoolean(true); //true
isBoolean('true'); //true
isBoolean('TRUE'); //false
isBoolean(1); //true
isBoolean(2); //false
isBoolean(false); //true
isBoolean('false'); //true
isBoolean('FALSE'); //false
isBoolean(0); //true
isBoolean(null); //false
isBoolean(undefined); //false
isBoolean(); //false
isBoolean(''); //false

Boolean Conversion Results

var Boolify = require('node-boolify').Boolify;

Boolify(true); //true
Boolify('true'); //true
Boolify('TRUE'); //null
Boolify(1); //true
Boolify(2); //null
Boolify(false); //false
Boolify('false'); //false
Boolify('FALSE'); //null
Boolify(0); //false
Boolify(null); //null
Boolify(undefined); //null
Boolify(); //null
Boolify(''); //null

colemars · Answer 8 · 2021-11-09T00:48:33.287

I'd prob prefer

const parseMaybeBool = val =>
  typeof val === 'string'
    ? val.toLowerCase() === 'true' || val.toLowerCase() === 'false'
      ? Boolean(val)
      : val
    : val;

or w/ types

const isBool = (val: unknown): val is boolean =>
  typeof val === 'boolean' ||
  (typeof val === 'string' &&
    (val.toLowerCase() === 'true' || val.toLowerCase() === 'false'));

const parseMaybeBool = (val: unknown) =>
  isBool(val) ? Boolean(val) : val;

Javascript check if string if true or false and convert to Boolean

8 Answers8