又优化了下 UA 逻辑 现在两行就 ok 了

makeding · makeding · commit e20e3c19844d · 2020-09-11T10:24:52.000+08:00
Binaryify/NeteaseCloudMusicApi#922
diff --git a/util/request.js b/util/request.js
@@ -35,16 +35,8 @@ function chooseUserAgent(ua = false) {
       // Linux 就算了
     ]
   }
-  let realUserAgentList = []
-  if (!ua)
-    realUserAgentList = (userAgentList.mobile).concat(userAgentList.pc) // concat 把两个 array 合并
-  else if (ua == 'mobile')
-    realUserAgentList = userAgentList.mobile
-  else if (ua == 'pc')
-    realUserAgentList = userAgentList.pc
-  else
-    return ua // 最后返回自定义的 ua
-  return realUserAgentList[Math.floor(Math.random() * realUserAgentList.length)]
+  let realUserAgentList = userAgentList[ua] || (userAgentList.mobile).concat(userAgentList.pc)
+  return (['mobile', 'pc', false].indexOf(ua) > -1) ? realUserAgentList[Math.floor(Math.random() * realUserAgentList.length)] : ua
 }
 const createRequest = (method, url, data, options) => {
   return new Promise((resolve, reject) => {

Original file line number	Diff line number	Diff line change
`@@ -35,16 +35,8 @@ function chooseUserAgent(ua = false) {`
`35`	`35`	`// Linux 就算了`
`36`	`36`	`]`
`37`	`37`	`}`
`38`		`- let realUserAgentList = []`
`39`		`- if (!ua)`
`40`		`- realUserAgentList = (userAgentList.mobile).concat(userAgentList.pc) // concat 把两个 array 合并`
`41`		`- else if (ua == 'mobile')`
`42`		`- realUserAgentList = userAgentList.mobile`
`43`		`- else if (ua == 'pc')`
`44`		`- realUserAgentList = userAgentList.pc`
`45`		`- else`
`46`		`- return ua // 最后返回自定义的 ua`
`47`		`- return realUserAgentList[Math.floor(Math.random() * realUserAgentList.length)]`
	`38`	`+ let realUserAgentList = userAgentList[ua] \|\| (userAgentList.mobile).concat(userAgentList.pc)`
	`39`	`+ return (['mobile', 'pc', false].indexOf(ua) > -1) ? realUserAgentList[Math.floor(Math.random() * realUserAgentList.length)] : ua`
`48`	`40`	`}`
`49`	`41`	`const createRequest = (method, url, data, options) => {`
`50`	`42`	`return new Promise((resolve, reject) => {`