javascript - 这是一个 'dead code' 即它只是多余的吗？

Question

我在我们的代码库中遇到了这个代码片段:

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)。

Answer 1

我认为你是对的。我可以将 datastring 视为 number 的唯一方法是在代码的其他地方定义自定义 toString (我看不出任何理由为什么)。

当然，代码中还有其他注意事项，比如 $(target).data("labels"). 如果 target 是未定义。