无法运行Observer Pattern示例

时间:2015-04-25 17:14:29

标签: javascript debugging design-patterns observer-pattern observers

我正在尝试重现书Learning JavaScript Design Patterns中给出的观察者模式示例,但我在extend函数中出现此错误:

enter image description here

我正在使用本书下面的代码:

  

首先,让我们模拟一个主题可能具有的依赖观察者列表:

我已在fiddle中设置了代码,我想了解为什么我会收到错误。

2 个答案:

答案 0 :(得分:1)

您问题中报告错误的问题是extend()的第一个参数不是有效对象。

当我运行你的jsFiddle时,它会报告Observer is not defined

如果Observer()模式应与Subject()模式平行,那么您缺少这段代码:

function Observer() {
  this.observers = new ObserverList();
}

或许,您可能只需要改变:

extend( new Observer(), check );

到:

extend( new ObserverList(), check );

这样它就会使用您为ObserverList()显示的代码。

事实上,当我按下jsFiddle中的按钮时,出现的错误是Uncaught ReferenceError: Observer is not defined,这进一步证实了上述情况。

而且,当我应用这个更改时,代码似乎在这里运行:http://jsfiddle.net/jfriend00/8xmu1mcg/,虽然我不确切知道它应该做什么,但它添加了一个复选框,并且没有错误。

答案 1 :(得分:1)

唯一的问题是你忽略了定义Observer。添加函数定义后,小提琴起作用:

http://jsfiddle.net/cm5a62jb/1/



// The Observer
function Observer(){
  this.update = function(){
    // ...
  };
}

function ObserverList(){
  this.observerList = [];
}
 
ObserverList.prototype.add = function( obj ){
  return this.observerList.push( obj );
};
 
ObserverList.prototype.count = function(){
  return this.observerList.length;
};
 
ObserverList.prototype.get = function( index ){
  if( index > -1 && index < this.observerList.length ){
    return this.observerList[ index ];
  }
};
 
ObserverList.prototype.indexOf = function( obj, startIndex ){
  var i = startIndex;
 
  while( i < this.observerList.length ){
    if( this.observerList[i] === obj ){
      return i;
    }
    i++;
  }
 
  return -1;
};
 
ObserverList.prototype.removeAt = function( index ){
  this.observerList.splice( index, 1 );
};

function Subject(){
  this.observers = new ObserverList();
}
 
Subject.prototype.addObserver = function( observer ){
  this.observers.add( observer );
};
 
Subject.prototype.removeObserver = function( observer ){
  this.observers.removeAt( this.observers.indexOf( observer, 0 ) );
};
 
Subject.prototype.notify = function( context ){
  var observerCount = this.observers.count();
  for(var i=0; i < observerCount; i++){
    this.observers.get(i).update( context );
  }
};

// Extend an object with an extension
function extend( extension, obj ){
  for ( var key in extension ){
    obj[key] = extension[key];
  }
}
 
// References to our DOM elements
 
var controlCheckbox = document.getElementById( "mainCheckbox" ),
  addBtn = document.getElementById( "addNewObserver" ),
  container = document.getElementById( "observersContainer" );
 
 
// Concrete Subject
 
// Extend the controlling checkbox with the Subject class
extend( new Subject(), controlCheckbox );
 
// Clicking the checkbox will trigger notifications to its observers
controlCheckbox.onclick = function(){
  controlCheckbox.notify( controlCheckbox.checked );
};
 
addBtn.onclick = addNewObserver;
 
// Concrete Observer
 
function addNewObserver(){
 
  // Create a new checkbox to be added
  var check = document.createElement( "input" );
  check.type = "checkbox";
 
  // Extend the checkbox with the Observer class
  extend( new Observer(), check );
 
  // Override with custom update behaviour
  check.update = function( value ){
    this.checked = value;
  };
 
  // Add the new observer to our list of observers
  // for our main subject
  controlCheckbox.addObserver( check );
 
  // Append the item to the container
  container.appendChild( check );
}
&#13;
<button id="addNewObserver">Add New Observer checkbox</button>
<input id="mainCheckbox" type="checkbox"/>
<div id="observersContainer"></div>
&#13;
&#13;
&#13;