Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Taro.getEnv() 与 process.env.TARO_ENV 的区别? #1080

Closed
cncolder opened this issue Nov 11, 2018 · 2 comments
Closed

Taro.getEnv() 与 process.env.TARO_ENV 的区别? #1080

cncolder opened this issue Nov 11, 2018 · 2 comments
Labels
question Further information is requested

Comments

@cncolder
Copy link
Contributor

问题描述

Taro.getEnv() 返回 'WEAPP' | 'WEB' | 'RN' | 'SWAN' | 'ALIPAY'

process.env.TARO_ENV 返回 'weapp' | 'swan' | 'alipay' | 'h5' | 'rn'

Taro.getEnv() 这个 API 设计的本意应该是用来运行时环境判断, 而 process.env.TARO_ENV 是在编译时替换成字符串. 但是细想下来, 两者永远是保持一致的, 编译成什么, 就只能运行在那个环境里.

这样看来, Taro.getEnv() 在各方面都不及 process.env.TARO_ENV, 再加上历史原因导致的大小写不一致以及 'WEB' 和 'h5' 不一致?

复现步骤
代码任意位置 console.log(process.env.TARO_ENV) 都能打印出正确的结果.

期望行为
建议给 Taro.getEnv() 加上 deprecated 标记, 避免使用.

报错信息

系统信息

  • 操作系统: [e.g. Windows 10]
  • Taro 版本 [e.g. v.0.0.64]
  • Node.js 版本 [e.g. v9.0.0]
  • 报错平台 [h5, weapp]

补充信息
[可选]
[根据你的调查研究,出现这个问题的原因可能在哪里?]

@luckyadam
Copy link
Member

有道理,不过 API 还是保留吧,以备不时之需,我会在调用 API 的时候给出提示,尽量使用 process.env.TARO_ENV

@yuche yuche added question Further information is requested resolved labels Nov 12, 2018
@taro-bot
Copy link

taro-bot bot commented Nov 12, 2018

Hello~

您的问题楼上已经提供了解决方案,如果没有更多的问题这个 issue 将在 15 天后被自动关闭。

如果您在这 15 天中更新更多信息自动关闭的流程会自动取消,如有其他问题也可以发起新的 Issue。

Good luck and happy coding~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants