Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

引入MapList, 取消JsonFilter #231

Closed
juqkai opened this issue May 30, 2012 · 4 comments
Closed

引入MapList, 取消JsonFilter #231

juqkai opened this issue May 30, 2012 · 4 comments

Comments

@juqkai
Copy link
Member

juqkai commented May 30, 2012

MapList结构, 最开始是在JSON实现中提出的, 然后在实际应用中, 发现MapList在很多方面都非常的灵活, 所以就加了很多以MapList结构做二次调整的功能, 比如最早的MapList->Obj, 以及合并MapList, 访问MapList等等小的功能, 一直以来, 这些功能都是些很小很小的功能点, 也比较散乱, 即属于JSON的一部分, 也没有必要单独提出来.

只是这段时间因为公司里面做API, 要用到很多JSON的转换, 过滤这些东西, 所以前段时间又加了一些东西, 那天与wendal说过后, 他觉得做成JsonFilter比较合适, 然后就那样改了, 可是总觉得有点别扭, 从本质上来说, MapList已经不完全是JSON的一个功能点, 或者它的子集了, JSON更多的应该是关注"JSON"本身的东西, 而结构转换, 过滤等等这些东西已经完全的超越了JSON的范围了, 而且在JSON的处理过程中混杂这些东西, 总感觉那么别扭.

这方面的东西越来越多, 感觉是时间把它单独.

MapList包的宗旨就是实现对MapList结构的便捷操作
功能列表:

  1. MapList转对象
  2. 合并MapList
  3. 以"abc.def[2].gh"的形式访问MapList
  4. JSON转MapList
  5. 对象转MapList
  6. 过滤属性值
  7. MapList结构转换

这些功能完全与JSON分离, 可由用户自己选择使用时机, 而不用再在JSON中参合JsonFilter.

@ghost ghost assigned juqkai May 31, 2012
@wendal
Copy link
Member

wendal commented May 31, 2012

把文档补上,测试通过,就可以合并了

不过,马上就发44版了, 要不把这个功能押后到45版? 讨论一下

@juqkai
Copy link
Member Author

juqkai commented May 31, 2012

44什么时候发呢?
我这个周可以把它弄完, 功能基本上都完了, 现在我在写测试, JsonFilter部分还没做, 然后剩下的就是文档了, 这部分以前也写了很多, 只要调整一下应该就可以了

juqkai added a commit that referenced this issue May 31, 2012
@wendal
Copy link
Member

wendal commented May 31, 2012

打算明天或周末发

juqkai added a commit that referenced this issue May 31, 2012
@juqkai
Copy link
Member Author

juqkai commented May 31, 2012

呼终于搞完了, 求代码审核~~~要是没什么问题就把这个关了嘛,

我到是希望能这次发布就加上, 主要是我们公司的项目, 我用了这些东西...T_T...要是这次发布没有的话, 那我就只能把这些弄到我们代码里面去, 可是这样我怕BOSS说...额...T_T,T_T,T_T,T_T

@juqkai juqkai closed this as completed May 31, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants