LCStarRatingView icon indicating copy to clipboard operation
LCStarRatingView copied to clipboard

有很多吐槽的。

Open AntiMoron opened this issue 8 years ago • 2 comments

做了什么:

  • 修正星星的位置跟手,但是值不对
  • 修正半颗星星模式的时候,星星绘制位置不是正中间
  • 加了一个delegate,因为callback这种东西经常存在需要weakify, strongify的东西,有时候对于外界的一些scalar types的变量,还需要__block,程序员难免会忘,这用起来不保险。加delegate也提供了更多编码习惯的选择。
  • 修正了一些typo
  • 统一了大括号的风格
  • 删除掉不必要的空行
  • 修正了nullable specifier.
  • 去掉了一些硬编码

为啥提交代码: 今天恰好需要加个小功能,网上搜了一波code4app搜索过来了。 然后找了一圈你这个最符合我的审美。 但是还有几个小问题:

  1. 星星数量没法改,最好再抽一下
  2. 做snapshot确实对于性能很好,不过当高度比较高的时候好大一片空白也跟着做snapshot了,性能有浪费

AntiMoron avatar Feb 08 '17 09:02 AntiMoron

还有就是有一点: 代码里也没用到什么特别新的特性。 最好让他能xcode 7编译 + 支持比较低级别的deployment target。 让价值高一些。 请原作者再接再厉完善开源库。

AntiMoron avatar Feb 08 '17 09:02 AntiMoron

新增commit: init会去调用initWithFrame.不需要重新声明init方法了,也不必要实现init方法,只initWithFrame重载下就可以了。

AntiMoron avatar Feb 08 '17 12:02 AntiMoron