我在我们的代码库中遇到了这个代码片段:
function initDoughnutChart(target) {
var labels = $(target).data("labels").toString().split(",");
var dataString = $(target).data("chart-data").toString();
// single-entry values
if (typeof dataString === 'number') {
dataString = dataString.toString();
}
var data = dataString.split(",");
...
我认为 if 语句是多余的 dataString
是否正确?初始化为值 <something>.<something>.toString()
因此它将(总是?)具有 'string'
的类型(除非它的初始化抛出,然后根本不会达到 if)。
最佳答案
我认为你是对的。我可以将 datastring
视为 number
的唯一方法是在代码的其他地方定义自定义 toString
(我看不出任何理由为什么)。
当然,代码中还有其他注意事项,比如 $(target).data("labels").
如果 target
是未定义。
关于javascript - 这是一个 'dead code' 即它只是多余的吗?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/65234867/