javascript - 组合两个 JavaScript 函数

Question

组合以下功能有什么好处吗？

window.onload = function() {
  if (localStorage.getItem("txt")) {
    textarea.value = localStorage.getItem("txt");
    changeDocTitle(localStorage.getItem("filename"));
    isModified = true;
  } else {
    changeDocTitle(untitled);
  }
};

function newNote() {
  if (!isModified || dontSave()) {
    textarea.value = "";
    changeDocTitle(untitled);
  }
  textarea.focus();
}

合并后将如下所示:

window.onload = function() {
  if (localStorage.getItem("txt")) {
    newNote(localStorage.getItem("txt"), localStorage.getItem("filename"));
  } else {
    newNote();
  }
};

function newNote(txt, filename) {
  if (!isModified || dontSave()) {
    textarea.value = txt || "";
    changeDocTitle(filename || untitled);
    if (textarea.value) {
      isModified = true;
    }
  }
  textarea.focus();
}

• 我也会使用键盘快捷键调用 newNote() 函数..

两者之间有什么区别，有什么理由更喜欢其中一个？

Answer 1

第二个。

如果newNote的作用域仅限于onload函数，那么就没有理由弄脏全局作用域。

<小时/>

编辑

说实话，这并不重要。如果绑定(bind)键盘事件，则该函数将保留在整个文档范围内。所以两者都好。

此外，SO中的问题更以问题为导向。试试https://codereview.stackexchange.com/ ，相反。