Каков наиболее эффективный метод установки значений параметров по умолчанию в javascript?

Я обычно использую второй, потому что он добавляет меньше сигнала.

Никогда не слышал о втором упомянутом вами способе, это интересно. Я бы сделал:

function Foo(par1, par2)
{
    par2 = par2 ? par2 : 'default value';
}

Но теперь, когда вы предложили моему вниманию свой второй метод, я думаю, что я бы использовал его только потому, что он меньше печатает.

Если вы используете jquery, вы можете:

function foo(params){
 var params = $.extend({}, {bar: '2'}, params);
 alert(params.bar);
}
foo(); //Alerts 2

первый на самом деле неверен, так как они будут неопределенными, а не нулевыми.

par2 !== null вернет true в этом случае.

поскольку javascript, если сравнивать слабо, сравнивать null, undefined, false с тем же значением, я бы предложил явно проверять неопределенное значение.

if (par2 !== undefined)
        par2 = "my default";

или же

    par2 = par2 !== undefined ? par2 : "my default";

Это позволит вам передавать такие значения, как false или null.

Однако ваш второй подход удобен, но только в том случае, если вы знаете, что никогда не передадите false или null.

Вопросы Похожий и связанные с задавались раньше.

Лучшим и более надежным подходом будет:

1) проверьте количество переданных аргументов (представьте, что вы хотите разрешить передачу неопределенного значения или null, как в случае функции DOM insertBefore), и только после этого попробуйте установить их значения, если они опущены:

function Foo(par1, par2)
{
    if (arguments.length < 2)
        par2 = "my default"
    else
    if (arguments.length < 3)
        par3 = "my default"
}

2) или, если вы хотите запретить передачу undefined, включите его в конструкцию:

function Foo(par1, par2)
{
    if (arguments.length < 2 && par2 === undefined)
        par2 = "my default"
    else
    if (arguments.length < 3 && par3 === undefined)
        par3 = "my default"

}

3) или, если вы хотите запретить передачу null, включите его в конструкцию:

function Foo(par1, par2)
{
    if (arguments.length < 2 && (par2 === undefined || par2 === null))
        par2 = "my default"
    else
    if (arguments.length < 3 && (par3 === undefined || par3 === null))
        par4 = "my default"
}

Кстати: я бы рекомендовал избегать использования перегрузки функций, на самом деле это вряд ли необходимо.

Мой выбор зависит от типа параметра и требуемого значения по умолчанию.

Например, это присвоит «значение по умолчанию», если par2 равен false, 0, пустой строке, null или undefined:

par2 = par2 || 'default value';

Такое поведение может не соответствовать ожиданиям или требованиям.

Лично я считаю, что вторая форма намного чище и удобнее для чтения, конечно, больше, чем проверка аргументов Сергея - хотя это может иметь место - но я предпочитаю передавать расширяемые объекты для аргументов, чем поддерживать сигнатуры аргументов.

например

function Foo(myArgs)
{
    myArgs.par1 = myArgs.par1 || "my default"
    myArgs.par2 = myArgs.par2 || "my default"
}

Единственное, о чем вы должны подумать при использовании раствора

var foo = bar || 123;

какие значения бара оцениваются как ложные. Это может вызвать у вас проблемы в будущем.

Эрик

Если вы передаете много аргументов функции, вы можете сделать что-то вроде:

function myFunc() {
    arguments[0] = arguments[0] || "default value for first argument";
    arguments[3] = arguments[3] || "default value for fourth argument";
    alert(arguments[3]);
}

Что насчет этого:

function myFunc(arg1, arg2, arg3) {
    switch (arguments.length) {
        case 0 : arg1 = "default1";
        case 1 : arg2 = "default2";
        case 2 : arg3 = "default3";
    }
}