CODEWITHSUNDEEP
iphoneobjective-ciosuitableview
Max Woolf
Max Woolf

Reputation: 4058

Adding Accessory to UITableView

I've got a UITableView displayed on screen for a while. In each cell is a song and artist name. In the background, each song and artist name is searched for online (using the Spotify API). It finds the URL to play one song, and then moves on to the next one! :) Sounds simple... but what I want is when each song is found, for the Checkmark accessory to appear in that row.

Currently i've got the following code to do this...

[[table cellForRowAtIndexPath:[NSIndexPath indexPathForRow:currentConnectionNumber inSection:0]] setAccessoryType:UITableViewCellAccessoryCheckmark];
[table setNeedsDisplay];

But all that happens is when all of the songs has been found, THEN the checkmarks appear... Why is this and how can I make the checkmarks appear one at a time?

thanks

Upvotes: 0

Views: 3895

Answers (3)

Ell Neal
Ell Neal

Reputation: 6064

You need to set the checkmark in tableView:cellForRowAtIndexPath:

- (void)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {

    static NSString *reuseIdentifier = @"cell";
    UITableViewCell *cell = nil;

    cell = [tableView dequeueReusableCellForIdentifier:reuseIdentifier];
    if (cell == nil) {
        cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:reuseIdentifier] autorelease];
    }

    NSString *cellTitle = [self cellTitleForRowAtIndexPath:indexPath]; // You need to implement this method
    BOOL hasURL = [self hasURLForRowAtIndexPath:indexPath]; // You need to implement this method

    cell.textLabel.text = cellTitle;

    if (hasURL)
        cell.accessoryType = UITableViewCellAccessoryCheckmark;
    else
        cell.accessoryType = UITableViewCellAccessoryNone;

    return cell;
}

Then reload the cells when your request finishes

- (void)myRequestFinished:(SomeKindOfWebRequest *)webRequest {

    NSIndexPath *indexPathToReload = [self indexPathForWebRequest:webRequest]; // You need to implement this method
    NSArray *indexPaths = [NSArray arrayWithObject:indexPathToReload];

    [self.tableView reloadRowsAtIndexPaths:indexPaths withRowAnimtation:UITableViewRowAnimationNone];
}

Upvotes: 1

Manish Agrawal
Manish Agrawal

Reputation: 11026

Are you reloading your tableView using

[yourtableView reloadData]

when your data modifies.

Upvotes: 0

thatguy
thatguy

Reputation: 312

I think rather than doing this you can set a bool flag for as per your requirement and add your checkmark logic in

cellForRowAtIndexPath:

and if that flag is true add accessary checkmark otherwise don't.

  if (isValid) {

        cell.accessoryType = UITableViewCellAccessoryCheckmark;

    }
    else{

        cell.accessoryType = UITableViewCellAccessoryNone;
    }

Upvotes: 0

Related Questions