Обрабатывать ложные аргументы

Question

Обрабатывать ложные аргументы

1

Есть ли какой-либо чистый способ присвоить значения по умолчанию функции, если какой-либо из ее аргументов является ложным.

Я знаю, что вы можете сделать что-то вроде

function addSomething(element, someArr = []) {
  someArr.push(element);
}

Но это не обрабатывает, если второй аргумент равен null, и тогда вы получите "не может прочитать свойство" push "нулевого значения.

Я надеялся, что ниже будет работать

function addSomething(element, someArr || []) {
  someArr.push(element);
}

....

Так что я оставил выполнение проверки внутри метода?

function addSomething(element, someArr) {
  someArr = someArr || [];
  someArr.push(element);
  // OR
  someArr && someArr.push(element);
}

Или есть шаблон, который я пропустил

user3711421 09 июнь 2017, в 10:49

Источник

Теги:

javascript

2 ответа

Ещё вопросы

Horia Coman · Answer 1 · 2017-06-09T05-04-00.000Z

2

Ваш код - лучшее, на что вы можете надеяться. null отличается от массива, поэтому, если вы явно передаете его в свою функцию, он должен:

Сбой громко, потому что в программе есть ошибка. Вы хотели получить массив и получили нуль.
Обработайте его изящно, потому что вы также хотите разрешить null.

Я думаю, вы хотите второй случай. С помощью аннотаций TypeScript эта функция может выглядеть так:

function addSomething(element, someArr: any[]|null)

И вам придется явно признать этот факт.

Horia Coman 09 июнь 2017, в 05:04

0

похоже, что это не работает. пожалуйста, добавьте рабочий пример. если вы используете внутри параметров someArr = someArr || [] тогда вы получите ошибку Use before declaration
Nina Scholz 09 июнь 2017, в 08:04
0

Кроме того, учитывая ответ PeterMander, это не выглядит как «лучшее, на что вы можете надеяться»
Freeman Lambda 09 июнь 2017, в 08:05
1

Вопрос ничего не говорит о TypeScript; ваш TypeScript в ответе просто затеняет суть.
T.J. Crowder 09 июнь 2017, в 08:05
0

OP интересовала языковая функция, которая позволяла бы им пропускать весь свой код. Их нет, поэтому они должны писать вручную.
Horia Coman 09 июнь 2017, в 08:09
0

@NinaScholz это работает в консоли браузера. Конечно, это не хороший код, и линтер и т. Д. Должны пометить его, но он будет работать
Horia Coman 09 июнь 2017, в 08:10
0

откуда вы берете any[] ?
Nina Scholz 09 июнь 2017, в 08:11
0

@NinaScholz Я не знал, что такое someArr , поэтому я someArr самый широкий из возможных типов только ради быстрого написания.
Horia Coman 09 июнь 2017, в 08:13
0

@ NinaScholz, я думаю, что это TypeScript, а не ES (по крайней мере, это то, что написано в ответе, ничего не знаю о TS)
Kaiido 09 июнь 2017, в 08:17
0

Я пометил его с помощью Javascript, так что машинопись не подходит для меня :(, но интересно узнать, как это делает другой язык
user3711421 09 июнь 2017, в 09:47
0

Typescript - это просто слой аннотаций типов поверх JS, в противном случае это тот же язык. В любом случае я просто пытался указать, как ваш вклад может быть двух разных типов, и вы должны рассматривать его как таковой.
Horia Coman 09 июнь 2017, в 09:50

Показать ещё 8 комментариев

PeterMader · Answer 2 · 2017-06-09T05-28-00.000Z

Даже ваш последний подход не на 100% безопасен:

function addSomething(element, someArr) {
  someArr && someArr.push(element);
}
addSomething('test', 'foobar');

Вы должны проверить, является ли параметр массивом:

function addSomething(element, someArr) {
  Array.isArray(someArr) && someArr.push(element);
}
addSomething('test', 'foobar'); // fails silently

Это правда, хорошая мысль. Я был обеспокоен только нулем, потому что это то, что я получаю, если вызов API не удался. Таким образом, аргумент arg всегда будет нулевым. Но если кто-то попытается вставить строку, конечно, она потерпит неудачу. Но Array.isArray (someArr) лучше, так как я хочу, чтобы функция была устойчивой. Отмечу это как исключенный ответ в ближайшее время, хочу увидеть и другие мнения ... Но в основном я должен сделать> someArr = Array.isArray(someArr) || []; (я использую методы массива несколько раз в этой функции)