Skip to content

Conversation

@AntiMoron
Copy link

做了什么:

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

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

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

@AntiMoron
Copy link
Author

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

@AntiMoron
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant