Improved DCI class

Author: mbrowne

samples/dci/DCI.js

@@ -3,12 +3,17 @@ var Context = (function () {
     }
     Context.extend = //for plain JS version
     function (callback) {
+        //return type isn't really void; void is used because otherwise TS gives this error when running `new MyContext(...`:
+        //"Call signatures used in a 'new' expression must have a 'void' return type."
         return function () {
             var args = [];
             for (var _i = 0; _i < (arguments.length - 0); _i++) {
                 args[_i] = arguments[_i + 0];
             }
             var context = new callback();
+            if (!context.bindRoles)
+                throw new Error('bindRoles() method not found');
+
             context.bindRoles.apply(callback, arguments);
             return context;
         };

samples/dci/DCI.ts

@@ -1,9 +1,14 @@
 export class Context
 {
 	//for plain JS version
-	static extend(callback): Function {
-		return function(...args : any[]) {
+	static extend(callback) {
+		//return type isn't really void; void is used because otherwise TS gives this error when running `new MyContext(...`:
+		//"Call signatures used in a 'new' expression must have a 'void' return type."
+		
+		return function(...args : any[]): void {
 			var context = new callback();
+			if (!context.bindRoles) throw new Error('bindRoles() method not found');
+			
 			context.bindRoles.apply(callback, arguments);
 			return context;
 		}

samples/dci/js/TransferMoney/index.js

@@ -4,7 +4,6 @@ var TransferMoney = require('./TransferMoney');
 var src = new Account(20);
 var dst = new Account(10);
 
-//var ctx = TransferMoney(src, dst, 10);
 var ctx = new TransferMoney(src, dst, 10);
 
 ctx.execute();

samples/dci/js/TransferMoney/index.ts

@@ -4,16 +4,12 @@ import TransferMoney = require('./TransferMoney');
 var src = new Account(20);
 var dst = new Account(10);
 
-
-//var ctx = new TransferMoney(src, dst, 10);
-
-
-var ctx = TransferMoney(src, dst, 10);
+var ctx = new TransferMoney(src, dst, 10);
 
 ctx.execute();
 
 //ctx.bindRoles(dst, src, 50);
 //ctx.execute();
 
 console.log(src.getBalance());
-console.log(dst.getBalance());
+console.log(dst.getBalance());