Typescript: 不要为简单的字符串提供“转换为模板字符串”重构

创建于 2020-02-13  ·  4评论  ·  资料来源: microsoft/TypeScript


TypeScript版本: 3.8.1-rc


搜索词:

  • 重构
  • 重构
  • 代码动作

回购
对于诸如

import ImageEditor from 'tui-image-editor';

将光标放在字符串上

错误:
提供convert to template string重构。 这没有帮助。

我建议只在字符串连接表达式上显示convert to template string

Bug Refactorings Fixed

最有用的评论

就像一个相反的观点,我实际上发现此行为很有用(在模板字符串合法的情况下)。 使用TS / JS多年后,我的肌肉记忆现在默认为${...}而不是将模板值转换为字符串的+ ,并且能够快速将相关字符串转换为模板非常好字符串(如果还没有的话)。

也就是说@DanielRosenwasser ,另一种可能的方法是提出建议,如果所讨论的字符串包含${ 。 只是一个想法。 :slightly_smiling_face:

所有4条评论

显然,这很愚蠢,因为在某些位置模板字符串无效-我认为对简单字符串禁用此功能可能是可以的,但是我们应该重新考虑是否要对所有字符串都启用它并禁用它对于无理的职位

  • 进出口说明符
  • 属性名称
  • 也许还有其他东西?

我宁愿只在将光标放在字符串连接表达式上时才能看到此重构

另外,我们可能想将其修复为3.8(无论是针对最终版本还是在恢复版本中),因为每个字符串都显示灯泡很烦人

就像一个相反的观点,我实际上发现此行为很有用(在模板字符串合法的情况下)。 使用TS / JS多年后,我的肌肉记忆现在默认为${...}而不是将模板值转换为字符串的+ ,并且能够快速将相关字符串转换为模板非常好字符串(如果还没有的话)。

也就是说@DanielRosenwasser ,另一种可能的方法是提出建议,如果所讨论的字符串包含${ 。 只是一个想法。 :slightly_smiling_face:

嘿there

快速了解@treybrisbane的观点。

对于上下文,我是提供JS和TS重构的VS Code扩展的作者。 我们曾经提供“转换为模板文字”。 由于它现在由编辑器通过TypeScript处理,因此我们将其删除。

但是,从以下位置切换非常有效(且方便):

const a = "I have some apples";

const a = `I have some apples`;

开始在其中插入变量。

我了解这是作为错误修复的副作用而删除的。 我认为我们真正要避免的是建议对诸如导入之类的无效内容进行重构。 我确实在扩展程序中实现了这种检查。

您可以查看https://github.com/nicoespeon/abracadabra/issues/94以获得更多有关此内容的信息。

如果您需要更多详细信息,我很乐意为您提供帮助😄
谢谢!

此页面是否有帮助?
0 / 5 - 0 等级

相关问题

fwanicka picture fwanicka  ·  3评论

Antony-Jones picture Antony-Jones  ·  3评论

weswigham picture weswigham  ·  3评论

bgrieder picture bgrieder  ·  3评论

CyrusNajmabadi picture CyrusNajmabadi  ·  3评论