search

aralejs / validator

Form Validator Widget
aralejs.org/validator/
41 stars 28 forks source link

源码看到这里，禁不住吐糟！ #33

Closed johnnychq closed 11 years ago

johnnychq commented 11 years ago

Core类中的setup方法有如下语句。

 this.on('formValidate', function () {
    var that = this;
    $.each(this.items, function (i, item) {
      that.query(item.element).get('hideMessage').call(that, null, item.element);
    });
  });

其中 that.query(item.element) 不就是 item 本身吗？

难道有特殊的原因，让你选择了绕圈？？

johnnychq commented 11 years ago

看源码本来是件很美妙的事情，validator的注释不全那咱就把代码写好；但现实的代码都如上所示的话，会让人崩溃，暗生肝火啊？

lizzie commented 11 years ago

淡定淡定, 正在重构~~

lizzie commented 11 years ago

关键点加了注释. 你说的 setup 方法中的问题已经改掉, https://github.com/aralejs/validator/blob/master/src/core.js#L201 .

issue 先关了~