我怎样才能简化这两行 JavaScript 以使其不重复？

Question

我通过编写以下两行代码解决了一个问题：

MyString = div.getAttribute('data-type') || div.getAttribute('data-id');
MyString = MyString.replace('RemoveThis', '');

虽然它工作得很好，但似乎我经常重复MyString。我试着把它缩短如下：

MyString = div.getAttribute('data-type') || div.getAttribute('data-id').replace('RemoveThis', '');

但它没有起作用。

有什么提示可以避免重复MyString吗？

上述解决方案会通过代码审查吗？

Answer 1

这种重复实际上可能是可取的，因为它使代码更易于阅读和维护。

但按照你的方式，div.getAttribute('data-type') || div.getAttribute('data-id')周围缺少一些括号

MyString = (div.getAttribute('data-type') || div.getAttribute('data-id')).replace('RemoveThis', '');

更好的做法是格式也稍微好一点。。。

MyString = (div.getAttribute('data-type') || div.getAttribute('data-id'))
.replace('RemoveThis', '');

但我还是更喜欢你的原版！