Includes() function alternative React

Question

I created a function that checks if the link matches the path. Everything works when I'm running the local server, but on the build process it fails with the error "Cannot read property 'includes' of undefined'. While i'm digging into the error, what's the best way to reformat this function without using includes()?

 matchLink = (link, path) => {

        return path.includes(link)
    };

If it works in development but not production, chances are it’s a build problem not a code problem — MTCoster
– MTCoster, Commented Feb 15, 2019 at 0:26
The error is about path, not includes. path is undefined, thus does not have an includes method. — Shawn K
– Shawn K, Commented Feb 15, 2019 at 0:27
I understand :) That's why I'm asking what's the other best way to reformat this function that's efficient — Max T
– Max T, Commented Feb 15, 2019 at 0:28
when I console.log path it shows the path, the typeof path also confirms that it's a string — Max T
– Max T, Commented Feb 15, 2019 at 0:29

powerc9000 · Accepted Answer · 2019-02-15 00:49:15Z

2

You should be checking if path is undefined and return false if it is.

If you don't want to use includes, indexOf can work as well.

matchLink = (link, path) => {
  return !!path && path.indexOf(link) > -1;
};

edited Feb 15, 2019 at 0:49

answered Feb 15, 2019 at 0:32

powerc9000

2,11220 silver badges32 bronze badges

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

Comments

Code Maniac · Accepted Answer · 2019-02-15 01:39:01Z

1

There are some more alternatives you can use

const matchLink = (link, path) => {
  let reg = new RegExp(link)
  console.log('RegEx Method : ',  reg.test(path))
  console.log('Search Method : ', path.search(link)>-1)
};


matchLink("http://www.example.com","http://www.example.com/123")
matchLink("http://www.example123.com","http://www.example.com/123")

answered Feb 15, 2019 at 1:39

Code Maniac

37.9k5 gold badges44 silver badges65 bronze badges

Collectives™ on Stack Overflow

Includes() function alternative React

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related