Почему isPlainObject проверка redux именно такая?

Question

[Виктор Дубров]

Всем привет
В Redux есть функция проверки isPlainObject , которая проверяет является ли объект переданный в dispatch() - простым
Ее реализация

function isPlainObject(obj) {
  if (typeof obj !== 'object' || obj === null) return false;
  var proto = obj;

  while (Object.getPrototypeOf(proto) !== null) {
    proto = Object.getPrototypeOf(proto);
  }

  return Object.getPrototypeOf(obj) === proto;
}

Вопрос: Почему бы вместо цикла просто не проверить так ?

function isPlainObject(obj) {
    if (typeof obj !== 'object' || obj === null) return false;

    return Object.prototype === Object.getPrototypeOf(obj);
}

Answer 1

[Vladimir Lewandowski]

Прототипом obj может быть Нечто, у которого прототип — Нечто2, у которого прототип уже Object. Цикл идёт до первого прототипа в цепочке.

UPD:

A comparison for equality with Object.prototype only works for objects that actually inherit from Object.prototype. This is not always the case for plain objects though, which might come from another realm - like an iframe - and inherit from the other realm's Object.prototype. To detect these, the code basically first searches for an Object.prototype-like (i.e. inheriting from null) object in the prototype chain of the argument, and then checks whether the argument directly inherits from that.

— источник.

Comments

[Виктор Дубров]

Object.prototype === Object.getPrototypeOf(obj); также решает вопрос

[Vladimir Lewandowski]

Виктор Дубров, хм, вы правы, похоже. Возможно, есть неочевидный кейс.

[Vladimir Lewandowski]

Таки неочевидный кейс есть (на самом деле очевидный). Обновил ответ.

[Vladimir Lewandowski]

Вот ещё дополнение: https://github.com/reduxjs/react-redux/commit/c9d2....

[Виктор Дубров]

Vladimir Lewandowski, Спасибо, вы очень помогли !