-
-
Save remojansen/16c661a7afd68e22ac6e to your computer and use it in GitHub Desktop.
function logClass(target: any) { | |
// save a reference to the original constructor | |
var original = target; | |
// a utility function to generate instances of a class | |
function construct(constructor, args) { | |
var c : any = function () { | |
return constructor.apply(this, args); | |
} | |
c.prototype = constructor.prototype; | |
return new c(); | |
} | |
// the new constructor behaviour | |
var f : any = function (...args) { | |
console.log("New: " + original.name); | |
return construct(original, args); | |
} | |
// copy prototype so intanceof operator still works | |
f.prototype = original.prototype; | |
// return new constructor (will override original) | |
return f; | |
} | |
@logClass | |
class Person { | |
public name: string; | |
public surname: string; | |
constructor(name : string, surname : string) { | |
this.name = name; | |
this.surname = surname; | |
} | |
} | |
var p = new Person("remo", "jansen"); |
@logClassWithArgs({ when : { name : "remo"} }) | |
class Person { | |
public name: string; | |
// ... | |
} | |
function logClassWithArgs(filter: Object) { | |
return (target: Object) => { | |
// implement class decorator here, the class decorator | |
// will have access to the decorator arguments (filter) | |
// because they are stored in a closure | |
} | |
} |
function log(...args : any[]) { | |
switch(args.length) { | |
case 1: | |
return logClass.apply(this, args); | |
case 2: | |
return logProperty.apply(this, args); | |
case 3: | |
if(typeof args[2] === "number") { | |
return logParameter.apply(this, args); | |
} | |
return logMethod.apply(this, args); | |
default: | |
throw new Error(); | |
} | |
} |
function logMethod(target, key, descriptor) { | |
// save a reference to the original method this way we keep the values currently in the | |
// descriptor and don't overwrite what another decorator might have done to the descriptor. | |
if(descriptor === undefined) { | |
descriptor = Object.getOwnPropertyDescriptor(target, key); | |
} | |
var originalMethod = descriptor.value; | |
//editing the descriptor/value parameter | |
descriptor.value = function () { | |
var args = []; | |
for (var _i = 0; _i < arguments.length; _i++) { | |
args[_i - 0] = arguments[_i]; | |
} | |
var a = args.map(function (a) { return JSON.stringify(a); }).join(); | |
// note usage of originalMethod here | |
var result = originalMethod.apply(this, args); | |
var r = JSON.stringify(result); | |
console.log("Call: " + key + "(" + a + ") => " + r); | |
return result; | |
}; | |
// return edited descriptor as opposed to overwriting the descriptor | |
return descriptor; | |
} | |
class Person { | |
public name: string; | |
public surname: string; | |
constructor(name : string, surname : string) { | |
this.name = name; | |
this.surname = surname; | |
} | |
@logMethod | |
public saySomething(something : string, somethingElse : string) : string { | |
return this.name + " " + this.surname + " says: " + something + " " + somethingElse; | |
} | |
} | |
var p = new Person("remo", "jansen"); | |
p.saySomething("I love playing", "halo"); |
function logParameter(target: any, key : string, index : number) { | |
var metadataKey = `__log_${key}_parameters`; | |
if (Array.isArray(target[metadataKey])) { | |
target[metadataKey].push(index); | |
} | |
else { | |
target[metadataKey] = [index]; | |
} | |
} | |
function logMethod(target, key, descriptor) { | |
if(descriptor === undefined) { | |
descriptor = Object.getOwnPropertyDescriptor(target, key); | |
} | |
var originalMethod = descriptor.value; | |
//editing the descriptor/value parameter | |
descriptor.value = function (...args: any[]) { | |
var metadataKey = `__log_${key}_parameters`; | |
var indices = target[metadataKey]; | |
if (Array.isArray(indices)) { | |
for (var i = 0; i < args.length; i++) { | |
if (indices.indexOf(i) !== -1) { | |
var arg = args[i]; | |
var argStr = JSON.stringify(arg) || arg.toString(); | |
console.log(`${key} arg[${i}]: ${argStr}`); | |
} | |
} | |
var result = originalMethod.apply(this, args); | |
return result; | |
} | |
else { | |
var a = args.map(a => (JSON.stringify(a) || a.toString())).join(); | |
var result = originalMethod.apply(this, args); | |
var r = JSON.stringify(result); | |
console.log(`Call: ${key}(${a}) => ${r}`); | |
return result; | |
} | |
} | |
// return edited descriptor as opposed to overwriting the descriptor | |
return descriptor; | |
} | |
class Person { | |
public name: string; | |
public surname: string; | |
constructor(name : string, surname : string) { | |
this.name = name; | |
this.surname = surname; | |
} | |
@logMethod | |
public saySomething(@logParameter something : string, somethingElse : string) : string { | |
return this.name + " " + this.surname + " says: " + something + " " + somethingElse; | |
} | |
} | |
var p = new Person("remo", "jansen"); | |
p.saySomething("I love playing", "halo"); |
function logProperty(target: any, key: string) { | |
// property value | |
var _val = this[key]; | |
// property getter | |
var getter = function () { | |
console.log(`Get: ${key} => ${_val}`); | |
return _val; | |
}; | |
// property setter | |
var setter = function (newVal) { | |
console.log(`Set: ${key} => ${newVal}`); | |
_val = newVal; | |
}; | |
// Delete property. | |
if (delete this[key]) { | |
// Create new property with getter and setter | |
Object.defineProperty(target, key, { | |
get: getter, | |
set: setter, | |
enumerable: true, | |
configurable: true | |
}); | |
} | |
} | |
class Person { | |
@logProperty | |
public name: string; | |
public surname: string; | |
constructor(name : string, surname : string) { | |
this.name = name; | |
this.surname = surname; | |
} | |
} | |
var p = new Person("remo", "Jansen"); | |
p.name = "Remo"; | |
var n = p.name; |
function logParamTypes(target : any, key : string) { | |
var types = Reflect.getMetadata("design:paramtypes", target, key); | |
var s = types.map(a => a.name).join(); | |
console.log(`${key} param types: ${s}`); | |
} | |
class Foo {} | |
interface IFoo {} | |
class Demo{ | |
@logParameters | |
doSomething( | |
param1 : string, | |
param2 : number, | |
param3 : Foo, | |
param4 : { test : string }, | |
param5 : IFoo, | |
param6 : Function, | |
param7 : (a : number) => void, | |
) : number { | |
return 1 | |
} | |
} | |
// doSomething param types: String, Number, Foo, Object, Object, Function, Function |
Hi guys sorry but the decorators signatures are a bit different since I wrote this. You can fins the new signatures here:
declare type ClassDecorator = <TFunction extends Function>(target: TFunction) => TFunction | void;
declare type PropertyDecorator = (target: Object, propertyKey: string | symbol) => void;
declare type MethodDecorator = <T>(target: Object, propertyKey: string | symbol, descriptor: TypedPropertyDescriptor<T>) => TypedPropertyDescriptor<T> | void;
declare type ParameterDecorator = (target: Object, propertyKey: string | symbol, parameterIndex: number) => void;
I will try to update this in the future but right now I don't have time :( Feel free to fork and send a PR if you do!
About the paramter decorator
, I am confused:
var metadataKey = `__log_${key}_parameters`;
if (Array.isArray(target[metadataKey])) {
target[metadataKey].push(index);
}
else {
target[metadataKey] = [index];
}
what do you mean by target[metadataKey]
, it is always undefined
.
it seems that Typescript does not support async decorators yet.
If anyone is looking for decorators with async calls or transactions, please take a look here:
https://github.com/ciroreed/kaop/blob/easy-decorators/test/showcase.js
Class decorators are broken when targeting ES2015. Upon decorated class' initialization it rises error
Error: (SystemJS) Class constructor Person cannot be invoked without 'new'
TypeError: Class constructor Person cannot be invoked without 'new'
My tsconfig.json
file:
{
"compilerOptions": {
"target": "es2015",
"module": "commonjs",
"moduleResolution": "node",
"emitDecoratorMetadata": true,
"experimentalDecorators": true,
"lib": [ "es2015", "dom" ],
"baseUrl": ".",
"paths": {
"app:*": ["app/*"]
}
},
"include": [
"app/**/*.ts"
],
"exclude": [
"node_modules"
]
}
It seems to be a problem with native classes.
Any thoughts?
Correct me if I'm wrong but it looks like @logProperty
decorator defines property on prototype making things a bit wrong: as I can see running code from typescript playground this
is a window
object and then once you make several objects of type Person
, changing the name for one of them will resolve in changing the name for all the others (since this property lives in prototype)?
In class decorator, indeed instanceof
works but a simple console.log
of the resulting object displays:
c
name: "remo"
surname: "jansen"
__proto__: Object
How to get back the name of the original class Person
instead of c
?
@dulowski-marek I'm having the same problem. Anyone knows of a resolution to this while still targeting ES5? Thanks.
I quite often get the error Converting circular structure to JSON on this code - where it calls JSON.stringify() if the object passed has cyclical data structures (most DOM elements have this)
Not a simple problem to solve either - I wonder if we should be using stringify()
https://stackoverflow.com/questions/11616630/json-stringify-avoid-typeerror-converting-circular-structure-to-json
Hi @kristianmandrup, @dulowski-marek, I am having the same problem. Does any of you found any solution?
I got the property decorator to work
export function logProperty() {
return (target: any, key: string) => {
// property value
let _val = this[key];
// property getter
function getter() {
console.log(`Get: ${key} => ${_val}`);
return _val;
}
// property setter
function setter(newVal) {
console.log(`Set: ${key} => ${newVal}`);
_val = newVal;
}
// Delete property.
if (delete this[key]) {
// Create new property with getter and setter
Object.defineProperty(target, key, {
get: getter,
set: setter,
enumerable: true,
configurable: true,
});
}
};
}
export class MyClass {
@logProperty()
public name: string = 'ThatGuy';
}
Be of great help! Thanks for sharing!
I got the property decorator to work
export function logProperty() { return (target: any, key: string) => { // property value let _val = this[key]; // property getter function getter() { console.log(`Get: ${key} => ${_val}`); return _val; } // property setter function setter(newVal) { console.log(`Set: ${key} => ${newVal}`); _val = newVal; } // Delete property. if (delete this[key]) { // Create new property with getter and setter Object.defineProperty(target, key, { get: getter, set: setter, enumerable: true, configurable: true, }); } }; } export class MyClass { @logProperty() public name: string = 'ThatGuy'; }
This doesn't work because all instances then share the same property value - not good :). You can easily verify using the TypeScript playground.
The fix is to define a backing field as an additional property:
function logProperty(target: any, key: string) {
delete target[key];
const backingField = "_" + key;
Object.defineProperty(target, backingField, {
writable: true,
enumerable: true,
configurable: true
});
// property getter
const getter = function (this: any) {
const currVal = this[backingField];
console.log(`Get: ${key} => ${currVal}`);
return currVal;
};
// property setter
const setter = function (this: any, newVal: any) {
console.log(`Set: ${key} => ${newVal}`);
this[backingField] = newVal;
};
// Create new property with getter and setter
Object.defineProperty(target, key, {
get: getter,
set: setter,
enumerable: true,
configurable: true
});
}
class Person {
@logProperty
public name: string;
public surname: string;
constructor(name : string, surname : string) {
this.name = name;
this.surname = surname;
}
}
var p1 = new Person("remo", "Jansen");
var p2 = new Person("elon", "Musk");
console.log(p1.name);
console.log(p2.name);
p1.name = "Remo";
p2.name = "Elon";
console.log(p1.name);
console.log(p2.name);
Unfortunately the backing field still appears in for...in
even if enumerable: false
, so I left it true anyways. And of course the backing field can be set directly... I don't know how to prevent this.
@afr1983 Don't call Object.defineProperty for backingField. This adds the property to the prototype and you are not using it. When you do this[backingField], you are accessing an instance property.
The backingField is expected to show up in for..in, this is the behavior if you manually create a property.
Hi guys sorry but the decorators signatures are a bit different since I wrote this. You can fins the new signatures here:
declare type ClassDecorator = <TFunction extends Function>(target: TFunction) => TFunction | void; declare type PropertyDecorator = (target: Object, propertyKey: string | symbol) => void; declare type MethodDecorator = <T>(target: Object, propertyKey: string | symbol, descriptor: TypedPropertyDescriptor<T>) => TypedPropertyDescriptor<T> | void; declare type ParameterDecorator = (target: Object, propertyKey: string | symbol, parameterIndex: number) => void;I will try to update this in the future but right now I don't have time :( Feel free to fork and send a PR if you do!
@remojansen what does the <T>
at the start of the type alias for MethodDecorator mean? How is declare type MethodDecorator = <T>...
different from declare type MethodDecorator<T> = ...
? Is there a way to specialize it so that MethodDecorator works only for functions of a given type?
https://gist.github.com/remojansen/16c661a7afd68e22ac6e#file-method_decorator-ts-L5
Can you explain why descriptor could be undefined here?
Useful examples! Can you @remojansen please provide an example of decorator of async class method that executes only on Promise.resolve()
? That will be really useful for sending analytics, for example.
@remojansen I get
unexpected ...
when I'm trying to do it. see: https://gist.github.com/amcdnl/c3d43b4d52acedbdf6f1