Как сформировать условие с логическим оператором?

Question

[svilkov87]

Всем привет!

Немного изменил вопрос. Нужно получить условие не с И, а с ИЛИ.

Есть объект:

var obj = {
    one  :  true,
    two  :  true
  };

Как сформировать такое условие:
Если есть в объекте one ИЛИ one не равен false, то что-то вывести в консоль?

Comments

[AskJs]

if (obj.one && obj.one !== false)
console.log("test")

Answer 1

[AskJs]

if (obj.one || obj.one !== false)
console.log("test")

&& - и
|| - или

Comments

[sgrogov]

В случае

var obj = {
  one: undefined, // или 0, или null и т.д.
};

условие сработает не так, как хотелось бы

[AskJs]

sgrogov, sgrogov, сделайте еще условие)

[sgrogov]

Ага, изи :) правда одним тут не обойтись

if ((obj.one || obj.one === null || obj.one === undefined || obj.one === 0 || obj.one === NaN || obj.one === "") && obj.one !== false) {
  console.log('test');
}

[AskJs]

sgrogov, ваш ответ не лучше, а еще сложнее...если парень не знает азов, а вьІ ему пихаете
hasOwnProperty.call/lodash

[svilkov87]

AskJs Спасибо.

[svilkov87]

Я сначала отметил ваш ответ решением, а потом понял, что условие должно быть не с И, а с ИЛИ.
Прошу прощения.

[AskJs]

svilkov87, поправил ответ тогда.

[sgrogov]

Давайте по-порядку.

Почему hasOwnProperty

Первая часть задачи звучит как "Если есть в объекте one". То есть все следующие варианты подходят под это условие:

[
	{ one: true },
	{ one: 1 },
	{ one: 0 },
	{ one: 'foo' },
	{ one: '' },
	{ one: [] },
	{ one: {} },
	{ one: NaN },
	{ one: null },
	{ one: undefined },
]

но не подходит вариант, где в объекте нет ключа one, например
{ two: true }

Если проверять как предлагаете вы через if (obj.one) {}, то результаты будут следующими:

[
	{ one: true }, 		// true
	{ one: 1 }, 		// true
	{ one: 0 },			// false
	{ one: 'foo' },		// true
	{ one: '' },		// false
	{ one: [] },		// true
	{ one: {} },		// true
	{ one: NaN },		// false
	{ one: null },		// false
	{ one: undefined },	// false
	{ two: true },		// false
]

То есть if (obj.one) {} для того, чтобы проверить "Если есть в объекте one" не подходит.

Поэтому для проверки наличия ключа в объекте используется именно Object.hasOwnProperty. Можно конечно использовать его так

var obj = {
	one: true,
	...objData,
};
if (obj.hasOwnProperty('one')) {}

но тогда всё сломается для объектов вроде

{
	one: true,
	hasOwnProperty: 'foo',
}

Поэтому достаточно оптимальным и надёжным способом является именно Object.hasOwnProperty.call(object, property).

Если парень не знает азов, а я ему пихаю hasOwnProperty.call/lodash

Да не, не пихаю если честно :)
Во-первых hasOwnProperty и call и есть те самые "азы", с которыми необходимо разобраться прежде чем использовать сравнения с приведением типа (if (obj.one) {} именно такой и является).
Во-вторых lodash я предложил потому, что конструкция Object.hasOwnProperty.call действительно выглядит пугающе. Да и в реальных проектах очень часто используются всякие lodash, underscore и прочие.

Условие должно быть не с И, а с ИЛИ

В таком случае достаточно одной проверки if (obj.one) {}. Потому что любое значение obj удовлетворяющее условию if (obj.one) {} не удовлетворяет условию if (obj.one !== false) {}.

Резюмируя

1. if (obj.one) {} может подойти в ограниченном количестве случаев, но нужно быть предельно осторожным при использовании таких проверок. Как только вы присвоите obj.one любое falsy значение (0, '', NaN, null, undefined) проверка сработает не так, как задумывалось.

2. Постарайтесь избегать хранения разных типов данных в одних и тех же переменных/свойствах объекта. Это также позволит сделать код более предсказуемым, а проверки простыми. Да JavaScript это позволяет, но появление таких инструментов как TypeScript и Flow ясно говорит о том, что в больших проектах с большими командами динамическая типизация приносит больше проблем, чем пользы.

3. Разберитесь в основах языка. В частности, как именно работает неточное сравнение и к каким проблемам может может приводить неосторожное использование.

Answer 2

[sgrogov]

if (Object.hasOwnProperty.call(obj, 'one') && obj.one !== false) {
  console.log('foo');
}

или используйте lodash (или аналоги)

if (_.has(obj, 'one') && obj.one !== false) {
  console.log('foo');
}

Comments

[AskJs]

Ваше решение не работает с undefined
axaxax

[AskJs]

one : undefined
И при obj.one !== false и при obj.one !== true
console - works)

[sgrogov]

const check = value => Object.hasOwnProperty.call(value, 'one') && value.one !== false;

check({}); // false, one нет в объекте - всё верно
check({ one: undefined }); // true, one есть в объекте и он не равен false - всё верно
check({ one: true }); // true, one есть в объекте и он не равен false - всё верно
check({ one: false }); // false, one есть в объекте, но он равен false - всё верно

Всё ж работает, что не так?