在javascript中，相同的代码解析不同函数中的参数 - 复制或不复制？

Question

有一系列函数，每个函数都以相同的方式解析其参数。现在这个解析块被剪切并粘贴到每个函数的开头。有更好的方法吗？

Post.update = function( e ) {

        // parse args
        var e, el, orig_el, args, render, callBack;
        for( var i=0; i<arguments.length; i++ ) {
                if( arguments[i] instanceof HTMLElement ) orig_el = arguments[i];
                else if( arguments[i] instanceof Render ) render = arguments[i];
                else if( arguments[i] instanceof Event ) {
                        e = arguments[i];
                        orig_el = orig_el || e.target;
                }
                else if( typeof arguments[i] == 'function' )  callBack = arguments[i];
        }

        // Post.update work here
}

Post.verify = function( e ) {

       // parse args
       ...

       // Post.verify work here
}

解析参数而不是单独传递的原因是，有五个以上可能的参数

• 我很容易在排序和遗漏调用具有长参数列表的函数时犯错误

• 更改函数的一个参数意味着更改对 func 的每次调用

• imho 函数与准确传递必要的内容相比，

所以我的目标是抽象函数的解析部分，同时保持函数的清晰度像现在一样打电话。

Answer 1

当然，您应该使用一个函数来防止复制和粘贴所有代码！

如果您想更改代码中的单个字符怎么办？现在您必须再次复制并粘贴所有内容！这真的没什么好处。

将代码粘贴到函数中并每次调用它，这样当您更改代码函数中的某些内容时，更改将反映到使用该函数的每个地方。

您是否担心向函数传递这么多参数？你真的必须担心，因为将这么多参数传递给一个函数真的很糟糕。只需将所有参数放入一个对象中，然后仅将该对象传递给函数，如下所示：

var arg = {
    arg1: 'bla bla bla',
    arg2: 4,
    //.....
};

// call the function passing only one parameter
myFunction( arg );

然后在函数内，您将有一个参数，并且可以像这样访问所有其他参数：

function myFunction( arg ) {
    // arg.arg1
    // arg.arg2 ...
}

Answer 2

您应该意识到函数不应有超过 3 个参数，而不是从函数中解析 5 个参数。

你想要做的是让最后一个参数是一个对象

Post.update = function(ev, obj) {
  // obj.el
  // obj.render
  // obj.ev
  // obj.cb
}

然后只需要一个可以重用的函数

Post.parseObj = function(obj) {
  ...
} 

Post.update = function(ev, obj) {
  obj = Post.parseObj(obj);
}

Answer 3

定义一次函数并重用它们

var update = function (e) { /* etc */ }
Post.update = update;

Answer 4

为什么不使用 Post.parseArgs 函数？

Post.parseArgs = function(args)
{
    // parse args
    var e, el, orig_el, args, render, callBack;
    for( var i=0; i<arguments.length; i++ ) {
            if( arguments[i] instanceof HTMLElement ) orig_el = arguments[i];
            else if( arguments[i] instanceof Render ) render = arguments[i];
            else if( arguments[i] instanceof Event ) {
                    e = arguments[i];
                    orig_el = orig_el || e.target;
            }
            else if( typeof arguments[i] == 'function' )  callBack = arguments[i];
    }
    // return what you want...
}