search

ecomfe / ub-ria

RIA base for union business
GNU General Public License v2.0
94 stars 34 forks source link

【ub-ria升级需求汇总】列表中筛选相关 #242

Open DDDBear opened 9 years ago

DDDBear commented 9 years ago

“筛选”功能目前在列表中是强配的，某些方法采用了硬编码方式实现功能，比如强制要求model中包含filterInfo，且未做任何异常处理，导致不符合此要求的项目运行时报错。

目前发现的硬编码有： View的updateFilterPanelStatus方法强制要求model中包含filterInfo数据； View的cancelFilter方法访问model中filterInfo字段，增加了View和Model的耦合度。

建议修改：1. 增加关键字段非空判断 2. 根据字段判断是否需要执行筛选相关操作

有其他方式和建议可以在此讨论。

otakustay commented 9 years ago

我们为何不在基类中给filterInfo一个空的对象，这样子类只要不自己手贱重写成null就能正确运行，也不需要做非空的判断

DDDBear commented 9 years ago

嗯，这样倒是也行，就是不知道会不会真的很倒霉的被某个同名字段覆盖。。。