How to match url in javascript using lastIndexOf?

Question

I'm new to javascript but am trying to conditionally display a button/link on a page if it matches a certain url with the code below:

LinkNode = document.getElementById("csv");
var url = document.documentURI
  if(document.documentURI.substr(url.lastIndexOf("/"+1)) == "/colleagues"){
    LinkNode.style.visibility= "visible";
  }
  else{
    LinkNode.style.visibility= "hidden";
}

Can anyone advise what I am doing wrong or how to match my url ending in "/colleagues" because this isn't matching? Is it possibly easier to use a regex?

How might I test to see what document.documentURI.substr(url.lastIndexOf("/"+1)) is actually producing?

Either remove the +1 or put the plus 1 outside the lastIndexOf brackets and change compare string. — Jordan Lowe
– Jordan Lowe, Commented Sep 7, 2016 at 8:08

Ahmed Jehanzaib · Accepted Answer · 2016-09-07 08:13:05Z

2

@MattE simply update your code

document.documentURI.substr(url.lastIndexOf("/")) == "/colleagues"

answered Sep 7, 2016 at 8:13

Ahmed Jehanzaib

1841 silver badge10 bronze badges

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

1 Comment

Ahmed Jehanzaib Over a year ago

@MattE Welcome :-)

Mayday · Accepted Answer · 2016-09-07 08:08:38Z

1

If you don't need to be very accurate, you can just use it this way:

if (document.documentURI.indexOf("/colleagues") !== -1) {
  LinkNode.style.visibility= "visible";
}
else {
  LinkNode.style.visibility= "hidden";
}

NOTE

This will just find in your whole URI if "/colleagues" exist. This means, it won't find it in the domain, since it has the "/". But it might be that you have "www.domain.com/colleagues/sample" or "www.domain.com/sample/colleagues" and it will enter the if in both cases

answered Sep 7, 2016 at 8:08

Mayday

5,2165 gold badges27 silver badges65 bronze badges

2 Comments

MattE Over a year ago

Thanks, this also works but in my case I will have 'colleagues' in other parts of the string on other pages and not want to display the link... However this is really useful to know :-)

Mayday Over a year ago

No problem. I found this post in stackoverflow, in order to answer your question "Is it possibly easier to use a regex?" stackoverflow.com/questions/273789/…

a.szechlicki · Accepted Answer · 2016-09-07 08:10:08Z

1

you can use a regex match:

var urlIndex = url.search("\/colleagues$");
if (urlIndex !== -1) {
  LinkNode.style.visibility= "visible";
} else {
  LinkNode.style.visibility= "hidden";
}

string.search method returns index of first match, or -1 if there is no match.

as for testing, you can use debugging tools that are available in almost every desktop browser, and set a breakpoint at a line you want to inspect other way is to add console.log('some text', someValue) and see output in browser console

answered Sep 7, 2016 at 8:10

a.szechlicki

661 silver badge4 bronze badges

1 Comment

MattE Over a year ago

this also works thanks, appreciate the advice on troubleshooting :-)

Collectives™ on Stack Overflow

How to match url in javascript using lastIndexOf?

3 Answers 3

1 Comment

2 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related