Committing the homework#6
Conversation
|
Oh, important to note - the correct project is Unit1WeekFourHomework, not anything inside of the file marked "Homework" in the repo. I fucked something up with the thing I had in there initially, but it's fixed. |
Remove extra files & fix project/target naming
There was a problem hiding this comment.
I haven't gotten to your controller files yet, but your data model probably shouldn't have to know anything about index paths. I'll add additional comments if I see this come up later on.
|
Nice work. The only major suggestion I can think of is to make sure your data model doesn't know too much about the rest of your app (i.e. you shouldn't store index paths in your data model because they're specific to table/collection views). It's not a huge deal for a very small app like this one, but when you start working on anything even slightly more complicated, it becomes much more important to maintain that separation. |
2 participants
Committing my homework! Love you, Mike! Almost deleted the entire homework and had to start from scratch, but thankfully I recovered.