search

aliyun / api-gateway-nodejs-sdk

The API Gateway SDK for Node.js
MIT License
106 stars 23 forks source link

我对 Client.buildUrl() 的内这段代码有点疑问 #5

Open wangzishi opened 7 years ago

wangzishi commented 7 years ago 
if (toStringify[key] && ('' + toStringify[key])) {
  list[i] = `${key}=${toStringify[key]}`;
 } else {
  list[i] = `${key}`;
}

if 判断的 condition 为什么要这么写呢？ 什么情况下会是 toStringify[key] 为 true，而 '' + toStringify[key] 为 false 呢？ https://github.com/aliyun/api-gateway-nodejs-sdk/blob/master/lib/client.js#L106

JacksonTian commented 7 years ago

欢迎发 PR，这个地方应该是参考了其他 SDK 不小心引入的坏味道。

yuanchenxi95 commented 6 years ago

当toStringfy[key] = ‘’时

yuanchenxi95 commented 6 years ago

我觉得在check是不是空string的时候，explicit check会比较好