search

Sneezry / Dualx

Chrome OS QQ Client
200 stars 51 forks source link

几点小改动 #31

Closed xndcn closed 11 years ago

xndcn commented 11 years ago
  1. main.js里的friendsInfo.categories排序，直接用Array.sort函数完成
  2. main.js里边for(i=0; i<categories.length-1; i++)似乎不应该是categories.length-1吧？
  3. HTML5QQ.friendsInfo改用dict，索引用uin，这样需要在friendsInfo里找uin对应的好友时就不用遍历整个friendsInfo了。

最后一个改动较大，我担心没改全，希望有人帮忙测试或者review 还有几个地方也应该用dict实现的，慢慢改

xndcn commented 11 years ago

求测试求review... 后续几个改动都要基于此...

Sneezry commented 11 years ago

@xndcn 你的工作真的很了不起，因为改别人的代码和自己写代码的工作量完全不是一个数量级的，而你却能为了优化改这么多！但也正是改了这么多，所以需要更长时间进行测试，所以请不要着急，你的工作一定不会白费的！同时也欢迎更多的人download @xndcn 的code进行测试，如果测试结束没有问题就会merge的 :-)

Sneezry commented 11 years ago

Hi，@xndcn 我刚刚测试了下，好友和好友分组有些混乱，即好友归组不正确，归组那里的数组逻辑比较复杂，等我再看看是哪里出了问题。

xndcn commented 11 years ago

@sneezry 过奖过奖，还是自己写代码的比较厉害... 你的这个项目是我用过的最好的第3方qq，所以很希望能一起把它完善起来。 惭愧啊，原来是main.js的598行HTML5QQ.friendsInfo.categories[categories[i].index]弄错了，应该是HTML5QQ.friendsInfo.categories[i]才对，我重新commit了，你再试下