CODEWITHSUNDEEP
iphoneiosobjective-cimageview
Aaradhya
Aaradhya

Reputation: 559

Detect UIImageView click in UITableViewCell

I have to make a call to specific number when imageview is clicked in tableviewcell.The number to which call is made is displayed in a label beside the imageview.But I couldn't get which cell is clicked.Always its referring to the last cell.

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

    if(cell == nil)
    {
        cell =[[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault  reuseIdentifier:CellIdentifier];
        lblphone = [[UILabel alloc] initWithFrame:CGRectZero];
        lblphone.tag = 116;
        lblphone.backgroundColor = [UIColor clearColor];
        [lblphone setFont:[UIFont fontWithName:@"Helvetica" size:12]];
        [lblphone setLineBreakMode:UILineBreakModeWordWrap];
        [lblphone setUserInteractionEnabled:YES];

        UITapGestureRecognizer *tapGestureRecognizer = [[UITapGestureRecognizer alloc] initWithTarget:self action:@selector(labelButton:)];
        tapGestureRecognizer.cancelsTouchesInView=NO;
        [tapGestureRecognizer setNumberOfTapsRequired:1];
        [lblphone addGestureRecognizer:tapGestureRecognizer];
        [tapGestureRecognizer release];
        [cell addSubview:lblphone];

        myImageView = [[UIImageView alloc] initWithFrame:CGRectZero];
        myImageView.tag = 120;
        myImageView.image=[UIImage imageNamed:@"CallImg.png"];

        UITapGestureRecognizer *tapped = [[UITapGestureRecognizer alloc] initWithTarget:self action:@selector(imageSelectedInTable:)];
        [tapped setNumberOfTapsRequired:1];
        [myImageView addGestureRecognizer:tapped];
        [tapped release];
        [cell addSubview:myImageView];
    }

    [myImageView setFrame:CGRectMake(10, 50,30,30)];
    myImageView= (UIImageView*)[cell viewWithTag:120];
    myImageView.image=[UIImage imageNamed:@"CallImg.png"];
    myImageView.userInteractionEnabled=YES;

    CGSize constraint5 = CGSizeMake(320, 2000.0f);
    CGSize size5=[phone sizeWithFont:[UIFont fontWithName:@"Helvetica" size:14] constrainedToSize:constraint5 lineBreakMode:UILineBreakModeWordWrap];
    lblphone =(UILabel *)[cell viewWithTag:116];
    [lblphone setFrame:CGRectMake(45,name.frame.size.height+name.frame.origin.y,320, size5.height)];
    lblphone.textAlignment=UITextAlignmentLeft;
    lblphone.backgroundColor=[UIColor clearColor];
    lblphone.text=[NSString stringWithFormat:@"%@ ",phone ];
    [lblphone sizeToFit];
}

And in the tapgesture of imageclick I am writing this :

-(IBAction)imageSelectedInTable:(UITapGestureRecognizer*)gesture
{
    UIImageView *imgView = (UIImageView *) [gesture view];
    UITableViewCell *cell = (UITableViewCell*)[[imgView superview]superview];
    NSIndexPath *tappedIndexPath = [self.tableView indexPathForCell:cell];
    NSLog(@"Image Tap %@", tappedIndexPath);
    UILabel *phoneno = (UILabel *)[cell viewWithTag:116];

    NSString *phoneNumber = [@"telprompt://" stringByAppendingString:phoneno.text];
    [[UIApplication sharedApplication] openURL:[NSURL URLWithString:phoneNumber]];
}

But it is always referncing to the last cell clicked irrespective of which cell is clicked.Couldn't understand where im going wrong ?

Upvotes: 2

Views: 2129

Answers (3)

iPatel
iPatel

Reputation: 47059

for get/know Which cell is clicked, Write Following code

UITableViewCell *cell = (UITableViewCell *)[[imgView superview]superview];

and if you alSo want to get indexPath of tapped cell then write

NSIndexPath *indexPath = [tableView indexPathForCell:cell];

EDITED :-> Another way is

Give tag of UIImageView in cellForRowAtIndexPath Method Such like, 

imageView.tag = indexPath.row;

And Use Following Code

int row = ((UIImageView*)imgView).tag; //Or// int row = ((UIGestureRecognizer *)sender).view.tag;

NSIndexPath* indexpath = [NSIndexPath indexPathForRow:row inSection:0]; 
UITableViewCell* cell = [table cellForRowAtIndexPath:indexPath];

Here You can get Cell of Tapped UIImageView.

Upvotes: 1

svena
svena

Reputation: 2779

I see you try to access your cell,

UITableViewCell *cell = (UITableViewCell*)[[imgView superview]superview];

but you add your image view to

[cell addSubview:myImageView];

You should be adding your image view to cell.contentView.

Upvotes: 3

danh
danh

Reputation: 62676

It looks like the image view is a subview of the cell (one step down the tree), and the code that's probing for the parent is taking two steps up ...superview superview. (That latter approach is right for nib defined cells which add subviews to the cell's contentView).

Here's a safer way to climb the view hierarchy that will work no matter how the cell was built.

- (UITableViewCell *)tableViewCellContaining:(UIView *)aView {

    UIView *answer = aView;
    while (answer && ![answer isKindOfClass:[UITableViewCell self]])
        answer = answer.superview;
    return answer;
}

I think your code will probably work if you replace this line:

UITableViewCell *cell = (UITableViewCell*)[[imgView superview]superview];

with this line

UITableViewCell *cell = [self tableViewCellContaining:imgView]

Upvotes: 3

Related Questions