YTKKeyValueStore icon indicating copy to clipboard operation
YTKKeyValueStore copied to clipboard

将 String 和 Number 接口都合并到 Object 中

Open xserver opened this issue 10 years ago • 4 comments

1、在表中添加 type 字段记录类型; 2、putObject 判断类型,都抽象为 String 入库; 3、取出时,判断type,转换为相应类型;

xserver avatar Oct 24 '14 08:10 xserver

@xserver 抱歉回复晚了,你的这个pull request我当时看了后很犹豫,所以一直没有想好是否应该merge过来。,我认为你的这种写法是有一些好处的。如果是新应用这么写还是不错的。但主要的问题是,你的这个存储方案改了存储逻辑,那么会造成历史数据无法读取,而我们的线上应用都依赖了这个库,如果我接受这个pull request,我需要另外写相关的数据转换逻辑。否则用户的历史数据都丢了,这是不可接受的。

所以我可能现在无法merge你的请求。非常抱歉。。。。

另外提个小意见,不要的代码可以直接删掉,不用注释掉,注释掉还是挺不干净的。

tangqiaoboy avatar Nov 06 '14 02:11 tangqiaoboy

@tangqiaoboy 的确如此,Good Luck.

刚玩 Git ,貌似有历史对比功能,删掉的确比较好。Cool. thank you.

xserver avatar Nov 06 '14 08:11 xserver

支持合并的建议,现在的接口putObject和putString等不是一个level的,可以做一个release版本保存现有状态,新版用合并后的接口

boydc2014 avatar Dec 11 '14 02:12 boydc2014

@xserver @tangqiaoboy @boydc2014 我在swift版中在保留数据库结构的情况下对string和number接口进行了整合,可以看一下: https://github.com/sgxiang/YTKKeyValueStore_Swift

sgxiang avatar Jan 08 '15 13:01 sgxiang