JavaScript 表单焦点，我的代码看起来还好吗？

Question

今天早上，我问了一个关于 JS 语法的非常普通的问题，有人建议我使用他们的代码，所以我继续这样做，因为我对 JS 的了解几乎为零。

利用我的 PHP 知识，我做了一点修改，添加了第二个 ID 和 'else if' 语句(见下文)，只是想知道对 JavaScript 有更多了解的人是否能告诉我我，如果一切看起来都不错？

或者也许您会采取完全不同的做法？

window.onload = formfocus;

function formfocus()
{
    var id_one;
    var id_two;

    id_one = document.getElementById( 'author' );
    id_two = document.getElementById( 's_primary' );

    if ( id_one )
    {
        id_one.focus();
        id_one.value = '';
    }
    else if ( id_two )
    {
        id_two.focus();
        id_two.value = '';
    }
}

编辑:我有点担心我的 window.onload = formfocus(); ...但不太确定是否有其他方法可以完成我想要的。

Answer 1

它绝对可以更简洁，特别是如果您使用 jquery。但假设你想坚持使用普通的旧式 JavaScript，我至少会更改以下内容:

var id_one;
var id_two;

id_one = document.getElementById( 'author' );
id_two = document.getElementById( 's_primary' );

至

var id_one = document.getElementById( 'author' );
var id_two = document.getElementById( 's_primary' );

此外，不使用空格包围函数参数是相当标准的做法，但我认为这是一个不应该在这里争论的风格问题。