tc39 / proposal-first-class-protocols

a proposal to bring protocol-based interfaces to ECMAScript users
352 stars 9 forks source link

Initializing protocol fields #35

Open benash opened 5 years ago

benash commented 5 years ago

Hi, thanks for your work on this interesting proposal. Would be possible to initialize a protocol field by accessing this, or would that need to be a method? E.g.:

protocol Pageable {
  pageSize
  items

  getFirstPage() {
    return this[Pageable.items].slice(0, this[Pageable.pageSize])
  }
}

class GroceryList {
  constructor(items) {
    this.items = items
  }
  implements protocol Pageable {
    pageSize = 2
    items = this.items                // <-- Would this be legal?
    get items() { return this.items } // <-- Or this?
    items() { return this.items }     // <-- Or would this be the only way to go?
  }
}
mlanza commented 1 year ago

Having implemented protocols in my own library I'll offer my perspective.

Protocols are contracts (or rather behaviors) applied to types so any object of a type automatically observes the contract. They're not applied to existing objects. You can create reified instances of a protocol, but that's a separate concern. I know it's all a matter of implementation/choice, but I implemented protocols as solely functions/methods (command or query), not properties, following after what Clojure did.

I would think rather your types implement the properties (and their defaults) apart from the protocol and the protocol methods simply make use of those properties. The properties, if absolutely necessary, would rather become part of an interface (see TypeScript's). And, for what it's worth, although similar, protocols and interfaces are not the same thing.