有多少人知道 match 的参数不能是字符串

[autoxbc]

很多人会这么写字符串匹配，结果似乎也对

'abcde'.match('cd')
// Array ["cd"]

实际这是错的，或者说某些情况是错的，来个例子

// 匹配京东全站
if( location.hostname.match('.jd.com') )
// 反例
'www.xjd.com'.match('.jd.com')
// Array ["xjd.com"]

道理很简单，match 的参数必须是正则，如果不是，会被隐式转换为正则。

因为是隐式转换，写的人可能没注意到，转换的结果可能和预期的不一致。

这里 dot 应该是纯字符串的一部分，被转换成了通配符。

new RegExp('.jd.com')
->   /.jd.com/ 

坑的是匹配结果放宽而不是收窄，所以调试不会出错，但是边界不对了。

---

应对方法是用 indexOf

'www.xjd.com'.indexOf('.jd.com') !== -1
// false

但是写起来有点长，多出 7 个字符(算上空格)，我一般这么写

!!( 'www.xjd.com'.indexOf('.jd.com')+1 )
// false

因为基本都是写在 if 语句里的，所以不用显示转换

if( location.hostname.indexOf('.jd.com')+1 )

当然也可以用 match 加手动转义，但是太丑了

'www.xjd.com'.match('\\.jd\\.com')
// null
'www.xjd.com'.match(/\.jd\.com/)
// null

大家有什么好的方法说一下

[Athrob]

知道是正则，转义挺好的。

[autoxbc]

@Athrob 转义破坏了字符串的可读性，尤其是长 url 字符串转义后看着难受

[ericls]

有什么理由不用 `.includes` 的吗？

`'www.xjd.com'.includes('jd.com')`

[bumz]

隐式转换太坑爹

[Athrob]

@autoxbc #2 有道理

[LioMore]

有点好笑

[rogerchen]

raw string 被楼主吃掉了

[autoxbc]

@rogerchen 请详细说一下，没有看懂

[shuirong1997]

之前在群里见到有人提出这个坑... 当时还真没想到...

[iyangyuan]

这…不看文档，不深究，单纯无脑看方法名，也能感觉到参数不是字符串

[nino]

从来不知道 match 还能传字符串

[autoxbc]

@iyangyuan
@nino
这可不是我编的，有人就是传字符串，教程也这么写
http://www.w3school.com.cn/jsref/jsref_match.asp

[TJT]

原来 match 还能传字符串

[TJT]

@autoxbc 回 12 楼

[TJT]

不小心按了 cmd + enter，
https://www.w3schools.com/jsref/jsref_match.asp
https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/String/match

都没有说可以传字符串的，MDN 的会说明有隐式转换。

[autoxbc]

@TJT 是个野路子用法

[CRVV]

1. match 的参数可以是字符串，字符串会被转成正则表达式然后匹配。用字符串表示正则表达式是很常见的用法，很多语言不像 JavaScript 一样有专门的正则表达式类型。这个帖子的标题就是错的
2. match 在几乎所有语言里都是正则表达式匹配，不是子串搜索，我真不知道你们是怎么能造出来把 match 当 find 的用法
3. w3schools 是著名的垃圾网站，不知道你们为什么不看正经的文档，还是中文的。
https://developer.mozilla.org/zh-CN/docs/Web/JavaScript/Reference/Global_Objects/String/match

[autoxbc]

@CRVV 您太激动了，观点有点偏颇

1. js 里有些方法就是支持多种类型参数的，比如 replace。对于不支持多类型参数的方法，强行使用非标准输入，并期望(或者根本不知道)隐式转换与预想的一致，这就造成混淆，很容易暗藏 bug，并提高维护成本。您是否认为可以随意使用 js 的灵活性，并在工业环境使用一些魔法代码？

2. match 是什么是由 match 的行为决定的，不是某些人认为的。当正确使用，match 就是可以充当 find 的角色，这甚至不能算作使用副作用。起码比 func && func() 这种要规范多了。要注意的是，includes 也是 ES6 才出现的，之前基本都是用 indexOf(str) !== -1。match 作为 find，不仅简单而且符合语义。

3. 我举了 w3schools 的例子，是为说明误用的普遍性，不是否定 w3schools 的价值，也不同意您否定其价值的观点。如果可以随意甩鄙视链，是否可以说 W3C 的文档更正经？ MDN 的逼格就不够了？ ECMA 也有更详实的文档，这些都是多层次教程的一部分，没有谁是垃圾。

[araraloren]

@autoxbc w3school.com.cn 这是个盗版教程网站吧！

[445141126]

string.search 也是