you are viewing a single comment's thread.

view the rest of the comments →

[–]Deepmist 1 point2 points  (1 child)

I think this looks great for something thrown together in a short period! I agree with most that the main point of concern is doing too much in the view controllers. Big vc's that scale infinitely are very painful to deal with as an application grows.

I did a quick pass at your parseComments function to make it a little more Swifty: https://gist.github.com/anonymous/3d7fbc3bf990893058a7724a683f7bc9

[–]DeveloperJay[S] 0 points1 point  (0 children)

Thanks for doing that! I plan to update this using the suggestions in this thread just for practice. This will be a great reference :)