Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BRStringPickerView 优化一处逻辑 #61

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

gjcbo
Copy link

@gjcbo gjcbo commented Dec 18, 2018

1.直接修改selectValueArr对应的component位置的值就行了,感觉没有必要创建一个新的容器。
2. didSelectRow 这个方法调用的频率非常高,每次都创建新数组,有点浪费,相对来说也会导致内存增加
3.个人建议: self.selectValueArr[component] = self.dataSourceArr[component][row];//直接修改
4.selectValueArr 是一个可数组,不需要copy.

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