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

UNIT1 week 4 HW #31

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

Conversation

bereket6725
Copy link

I was almost done with it this evening, but my computer acted really funny and couldn't the edit until 5 min ago. :/

@megantaylor
Copy link

IT WORKS! Congrats!

Please remember to make more commits, it will help you keep track of progress and unf^&*k your code.

  • (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
    // Return the number of rows in the section.
    return 5; //we are not really adding or removing cells so it seemd pointless :/
    }
    This is not recommended, you would have to update this number everytime you add more items. Also, If someone else is looking at your code, they don't know what's in the table, its just 5 items of something.

if (self.someCategory.options[indexPath.row]== self.someCategory.selection)
this is comparing the pointer value, when you want to compare the content with the equals method or stringequals

this app is a great opportunity to use a delegate pattern

segue is hidden inside storyboard instead of in code

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.

2 participants