react源码写法疑惑

Question

// React v16.2 
// ReactBaseClasses.js

/**
 * Base class helpers for the updating state of a component.
 */
function PureComponent(props, context, updater) {
  // Duplicated from Component.
  this.props = props;
  this.context = context;
  this.refs = emptyObject;
  // We initialize the default updater but the real one gets injected by the
  // renderer.
  this.updater = updater || ReactNoopUpdateQueue;
}

function ComponentDummy() {}
ComponentDummy.prototype = Component.prototype;
var pureComponentPrototype = (PureComponent.prototype = new ComponentDummy());
pureComponentPrototype.constructor = PureComponent;
// Avoid an extra prototype jump for these methods.
Object.assign(pureComponentPrototype, Component.prototype);
pureComponentPrototype.isPureReactComponent = true;

问题在于为什么要造一个ComponentDummy
我的理解是如果为了继承prototype的话下面没必要再用assign再设置一遍方法，他既然用了继承了，这边注释说为了避免方法额外的原型查找才用的assign，既然这样的话写成下面这样不就好了吗

var pureComponentPrototype = PureComponent.prototype;
Object.assign(pureComponentPrototype, Component.prototype);
pureComponentPrototype.isPureReactComponent = true;

Answer 1

本来要找两次现在把方法提了一级只找一次

Answer 2

PR里面有一个跟你一样的想法。链接在这里。

首先你的想法是完全正确的。为什么不这么做的。我们看Dan的猜测。Dan说可能是因为IE8不支持Object.assign吧。为了兼容老用户。不想一下子升上来抛弃掉他们。

但是他也不是很确定。于是他@sebmarkbage。sebmarkbage算是补充。就算有了Object.assign。其他人用了polyfills。但是也无法保证polyfills是否是标准的。可能有些人会用一些非标准的实现就会出问题。所以还是不改这里比较稳。