Putting an "or" in javascript

Wardy7 · 02-21-2013, 02:30 PM

Hi,
I have some code that I want to include an "or" option into the code but not having much luck with it because I'm not too javascript savvy

Code:

if(anchorSrc.match("mysite.co.uk")&&anchorVal.match("Great Site"))

Basically the above code makes sure that a link to mysite.co.uk is in place and that it uses the link text "Great Site".
What I want to do is add that the "anchorVal.match" can be "Great Site" OR "Super Site" if that makes sense but I'm not having much luck with it.

Any help would be appreciated, thanks!

Philip M · 02-21-2013, 03:04 PM

"and" in Javascript is &&
"or" is ||

You must use brackets to group the conditions:-

Code:

if ( (anchorSrc.match("mysite.co.uk")) && ((anchorVal.match("Great Site")) || (anchorVal.match("Super Site"))) ) {

Customer in UK travel agent shop: - "What month is it now in Australia?"

Obsidian · 02-21-2013, 03:11 PM

"||" is be the statement for "or" in JavaScript as earlier stated by Phillip M.

The link below is a good reference to these Logical Operators provided from W3Schools.com -- I personally use this site as a coding reference.

http://www.w3schools.com/js/js_comparisons.asp

Wardy7 · 02-21-2013, 03:21 PM

Quote:

Originally Posted by Philip M

"and" in Javascript is &&
"or" is ||

You must use brackets to group the conditions:-

Code:

if ( (anchorSrc.match("mysite.co.uk")) && ((anchorVal.match("Great Site")) || (anchorVal.match("Super Site"))) ) {

Customer in UK travel agent shop: - "What month is it now in Australia?"

Philip you are an absolute star man that is brilliant and does the trick!

Thanks also for the explanation of the || for an "or" too, it would explain why my code didn't like it when I tried adding in an "or" into it.

One other slight deviation to this question to save me starting a new thread, but is there any simple way to make the "anchorVal.match" parts not to be case sensetive like they currently are or am I best just adding lots of "or" statements in to cover a few variations that users might do if they change the case?

Philip M · 02-21-2013, 03:34 PM

Code:

if ((anchorSrc.toLowerCase().match("mysite.co.uk")) && ((anchorVal.toLowerCase().match("great site")) || (anchorVal.toLowerCase().match("super site"))) ) {

Philip M · 02-21-2013, 03:41 PM

Quote:

Originally Posted by Obsidian

"||" is be the statement for "or" in JavaScript as earlier stated by Phillip M.

The link below is a good reference to these Logical Operators provided from W3Schools.com -- I personally use this site as a coding reference.

http://www.w3schools.com/js/js_comparisons.asp

That particular page is fine, but sadly a good deal of w3schools.com which was great 10-12 years ago is very much out of date, and is not a very good exemplar of modern Javascript.
Alerts, prompts, document.write(), and quite a few other things covered in the tutorials are all obsolete!

The best and most up-to-date tutorials I know are at http://www.javascriptexample.net/

Old Pedant · 02-21-2013, 11:42 PM

Another way to do it:

Code:

if (    anchorSrc.match(/mysite.co.uk/i) 
     && ( anchorVal.match(/great site/i) || anchorVal.match(/super site/i) )
) {

The /i there means "ignore case". Technically, the argument to match SHOULD be a regular expression. When you use a string instead, as Philip did, it is converted to a regular expression for you, but then you can't specify /i or /g.

Another way, even better:

Code:

if (    anchorSrc.match(/mysite.co.uk/i) 
     && anchorVal.match(/(great site|super site)/i) 
) {

A regular expression can be used to match any number of things if they are separated by | and grouped in ( ).

Still another way (completely equivalent for these purposes):

Code:

if ( (/mysite.co.uk/i).test(anchorSrc) && (/(great site|super site)/i).test(anchorSrc) ) {