-
Notifications
You must be signed in to change notification settings - Fork 25
How would decorators work with fields and get/set? #58
Comments
You might ask over at the decorators proposal, since it looks like your question is more about decorators. (Remember, decorators aren't yet any more part of the language than public fields are, and are no more settled in their behavior!) |
That said, re:
You might be able to make that happen with a decorator, but without one that comes down to #42. Personally I feel very strongly that public fields should be for defining fields, and if you want to trigger setters and other effects the constructor is the correct place to do it. |
(Sorry, I didn't mean to imply this issue ought to be closed! Someone else might have more information. Just, people working on the decorators proposal might too :) ) |
No problem. You'd expect the decorators proposal to consider how it would work with this proposal? |
I know that its champions are considering how it works with this proposal, yes. Both are concerned about working with the other, but most of the machinery you're concerned about would be handled by the decorators proposal. |
This might be outside the current concerns of this proposal, but I'm wondering how decorators will work alongside fields and get/set.
CanJS has a library can-define we use to add rich behavior to a constructor function and its prototype like this:
value
is the initial value of the property. Users expected values to go through the setter, example issue: canjs/can-define#87. I'm trying to figure out how I could migrate this sort of rich behavior toclass
with decorators, fields, and get/set.Ideally, something like:
Would
@type
get called like:And be able to provide a get / set definition? Would it get called during the class definition or the construction of the instance?
I doubt it will work this way, but I'd really like
@prop
to be able to be called with the initial field value:So I could rewrite the getter/setters to use it if
set
hasn't been called as follows:Thanks for any clarity you can provide!
The text was updated successfully, but these errors were encountered: