Here is an impressive Review List from Fadi Ossama that all iOS/Mac developers could use.
I disagree with a couple of these, but in general, this is list is something all Swift developers should strive for!
https://medium.com/swiftcairo/ios-code-review-checklist-482f17f5c7c6
His GitHub repository: https://github.com/FadiOssama/Swift-Code-Review-Checklist
Personal Note: I gave a talk the other day on one of Fadi’s rules: “Clarity is more important than brevity”. I agree 100%! Swift was designed to be an easy to read language, it was not designed to be a Terse one. Don’t be afraid to be verbose. Use many lines to display multiple clause Guard statements, split up multi-parameter calls into separate lines and use vertical white space to group lines of Swift. This is all so other members of your team can easily read and understand your code.
No comments:
Post a Comment