leancloud / LeanCloudChatKit-Android

95 stars 33 forks source link

重构部分代码 & bug fix #5

Closed daweibayu closed 8 years ago

daweibayu commented 8 years ago

@jwfing 帮我看看整体框架上有没有什么需要改动的地方吧,稍后我一起把这些都改掉。

wujun4code commented 8 years ago

暂缓合并,我会列出一个列表描述 ios 与 android 不同的地方。今天下午给出。

jwfing commented 8 years ago

library 里面用户用不到的库,都隐藏起来吧。

jwfing commented 8 years ago

所有 exception.printStackTrace() 都换成日志输出吧。

wujun4code commented 8 years ago

不同之处

IMKitUser 类的名字不一样

android iOS 建议
LCIMUserProfile LCIMUser LCIMUser

LCIMUser的属性名字不一

android iOS 建议
LCIMUserProfile.userName LCIMUser.*name name

iOS LCIMUser 多出接口

/**
 *  检查与 aPerson 是否表示同一对象
 */
- (BOOL)isEqualToUer:(LCIMUser *)user;

- (void)saveToDiskWithKey:(NSString *)key;

+ (id)loadFromDiskWithKey:(NSString *)key;

Android 设计的 LCIMProfileProvider是为了引导开发者接入自己的用户系统,与 iOS 设计不一

具体需要宜龙跟你解释一下,但是宜龙设计的有点复杂

综合两边的问题

作为第一个用户,我要是开发者,我会在乎如何快速的接入我的用户体系,如何把好友关系导入,如何快速的让自己的真实用户走一遍:登录 -> 选择好友 -> 聊起来 (群聊就是选择若干好友,群聊一起来)

目前,ios 比较接近我理想中的设计,现在这个版本 UI 还是需要协调。

@daweibayu @jwfing @ChenYilong

ChenYilong commented 8 years ago

LCIMUser是Demo里的类,Lib里的是LCIMUserModelDelegate,LCIMUserModelDelegate里接口是一致的。

daweibayu commented 8 years ago

其他的都已经改完了,“library 里面用户用不到的库,都隐藏起来吧” 这个还没理解啥意思?是把所有的文件放到一个 package 里边么? @jwfing