Правильное использование необязательных параметров в JavaScript

Question

Мы используем пользовательский сборщик данных, который до сих пор должен был возвращать дату и время. Но теперь условия изменились, и иногда мне нужно вернуть только дату, поэтому я решил добавить необязательный параметр. Теперь сценарий таков:

var date_format = "dd/mm/yyyy HH:MM:ss"; 

function viewCalendar(parent, destField, dateOnly) { 
    if (typeof dateOnly !== "undefined") { 
     date_format = "dd/mm/yyyy"; 
    } 
    //more code... 

Но даже несмотря на это, кажется, работает (я сделал только несколько попыток) мне не нравится это очень много, но, возможно, это потому, что я не привык к пути JavaScript , Если бы это было не из-за какой-то пример кода я бы сделать что-то вроде:

var date_format = "dd/mm/yyyy HH:MM:ss"; 
var dateOnly = true; 

function viewCalendar(parent, destField, dateOnly) { 
    if (typeof dateOnly != true) { 
     date_format = "dd/mm/yyyy"; 
    } 

Но несколько примеров, которые я видел об использовании параметров в необязательный JS я не видел, что-то вроде этого. Каков правильный способ сделать этот вид JavaScript?

Answer 1

У вас есть несколько вариантов, вы уже обнаружили typeof. Если данные не собираются быть falsy если не пропускаются, вы можете использовать logical OR||, тройную conditional operatora?b:c или if с logical NOT, чтобы проверить, чтобы установить его или нет.
У вас также есть выбор по сравнению с неопределенного или void 0, оба из которых по-прежнему будет работать, даже если параметр, как ожидается, будет falsy, за исключением прохождения неопределенных себя в качестве аргумента.

function foo(op0, op1, op2, op3, op4, op5, op6) { 
    // if with logical NOT 
    if (!op0) op0 = 'default0'; 
    // logical OR 
    op1 || (op1 = 'default1'); 
    op2 = op2 || 'default2'; 
    // ternary 
    op3 = op3 ? op3 : 'default3'; 
    // compare, below this line with falsy args too (except explicit `undefined`) 
    if (op4 === undefined) op4 = 'default4'; 
    if (op5 === void 0) op5 = 'default5'; 
    // compare combined with logical OR 
    (op6 !== undefined) || (op6 = 'default6'); 

    // log parameters to see what we have now 
    console.log(op0, op1, op2, op3, op4, op5, op6) 
} 
foo(); // default0 default1 default2 default3 default4 default5 default6 

Пожалуйста, обратите внимание, что в старых браузерах, undefined был записываемые в глобальном масштабе, и что во всех браузерах, если вы не в глобальном масштабе, undefined может быть var d или установить в качестве параметра и поэтому имеет значение, которое не является undefined.

Если вы не в порядке, выберите с помощью оператора void над неопределенным.
Если вы понимаете, что это может произойти, и вам не нужно беспокоиться о том, что кто-то затеняет undefined, не стесняйтесь использовать его.
Если вы когда-нибудь видели, что кто-то это сделал, спросите их, могут ли они varArray для вас тоже, или что-то подобное, поэтому они понимают, что они были глупыми.

Answer 2

я пойду с тройным оператором здесь

date_format = dateOnly ? "dd/mm/yyyy" : date_format ; 

Он будет искать truthyness в dateOnly, и либо переопределить формат или сохранит его

Также

if (typeof dateOnly != true) { 
    date_format = "dd/mm/yyyy"; 
} 

кажется неправильно, это может быть

if (dateOnly == true) { 
    date_format = "dd/mm/yyyy"; 
} 

Answer 3

Другой способ заключается в использовании анонимный объект в качестве единственного параметра:

function viewCalendar(opts) { 
    if (opts.dateOnly) { 
     date_format = "dd/mm/yyyy"; 
    } 
} 

viewCalendar({parent:foo, destField:bar, dateOnly:true}) 

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