谈谈 parseEventsFromAttrs

[popomore]

可能很多人都不知道这个功能，我也是最近看代码才发现的

var Test = Base.extend({
  attrs: {
    onFocus: spy1,
    onChangeXx: spy2,
    beforeShow: spy3
  },
  show: function() {}
});

看看这个功能是如何用的，可以将一个属性注册为事件，如

onFocus 会将函数注册到 'focus' 事件上，onChangeXx 会将函数注册到 'change:xx' 事件上。beforeShow 则是将函数注册到 'before:show' 事件上。

先从分类上说，before/after 是对方法做切面，change 监听属性的变化，on 是最普通的监听。这些都放到 attr 上会很混乱，容易造成误解。

从用户角度说，这些都可以用实例方法去实现，更加优雅，如

var t = new Test();
t.on('focus', function() {}); 
t.before('show', function() {}); 
t.on('change:xx', function() {}); 

所以我觉得这个功能可以去掉。

[lifesinger]

赞同，这个可以去掉

[lianqin7]

。。。。。这个去掉了。。。晕死，又要自己写一堆hack了

[lianqin7]

beforeShow,afterHide,onClose这些用法在api设计里面很常见啊，从api设计及使用者方便的角度来说，是 方便且合理的

何必要纠结是否会造成attr的混乱呢？毕竟 很多使用者并没有 config === attr 的意识，也不会去在乎内部的实现，只要好用就行，config === attr也是很不合理的

[lianqin7]

@popomore @lifesinger

请考虑下实际的使用者场景，比如现在的

https://github.com/aralejs/dialog @afc163

https://github.com/aralejs/validator @shaoshuai0102

[lianqin7]

@popomore onChange确实用得比较少，after/before + on 还是很常见的说。。。

[afc163]

dialog 里的确有 onConfirm 和 afterHide 的使用方法。

对于有实例的调用可以用下列方法替代：

instance.on('confirm', function() {});

这是没有问题的，但是在 confirmbox 里有这样的静态调用。

Confirmbox.confirm('内容', '标题' , function() {}, {
    onConfirm: function() {},
    beforeHide: function() {}
});

这个正是利用 parseEventsFromAttrs 来完成的，如果去掉了，这种没有显式实例的调用方式，则不知道如何实现。

http://aralejs.org/dialog/examples/confirmbox.html

[popomore]

@lianqin7 是只保留一种使用方式，而不是不支持，用 attribute 和 events 的就好了。

[popomore]

@afc163 单例的参数也可以转化给实例的

ConfirmBox.show = function(message, callback, options) {

  new ConfirmBox()
    .on('confirm', options.onConfirm || function() {})
    .before('hide', options.beforeHide || function() {})
}

[lianqin7]

@popomore 但是把初始化的方式去掉了，在使用场景上就变得麻烦很多

比如

// 原来
new Xxx({
    onClose: funtion(){},
    afterShow: function(){}
});

// 现在
new Xxx({}).on('close', function(){}).after('show', function(){});

// 现在如果我想第一种用法保留，就必须去在Xxx的原型里显式指定了 :(
setUp: function(){
    if(this.get('onClose')){
        this.on('close', this.get('onClose'));
    }

    if(this.get('afterShow')){
        this.after('show', this.get('afterShow'));
    }
}

我想使用者大部分觉得第一种简单易记吧，特别是某些组合场景，只有 原型+配置 的情况，就一定要在原型里hack了，很不方便