Segue is not working for tableview cell

Question

I have the following code having capability such that, once I tap on the table cell it will go to another view controller where it has its description. I am trying to do it programmatically.

This is my code that controls the segue part, which is not getting called:

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath {
    UIStoryboard *mainStoryboard = [UIStoryboard storyboardWithName:@"Main" bundle: nil];
    DetailViewController *detailVC = (DetailViewController*)[mainStoryboard instantiateViewControllerWithIdentifier: @"detailViewController"];
    detailVC.navigationItem.title = self.ourNumberOfQuotes[indexPath];
    detailVC.indexRow =  [NSString stringWithFormat:@"%@",indexPath];
    [self.navigationController pushViewController:detailVC animated:YES];

}

Why is this not getting called?

This is my .h file code

@interface QuoteTableViewController : UITableViewController <UIGestureRecognizerDelegate,UITableViewDataSource,UITableViewDelegate> {
    int i;
}
@property (strong, nonatomic) IBOutlet UIView *mainMenu;
@property (strong, nonatomic) IBOutlet UIButton *homeworkButton;
@property (strong, nonatomic) IBOutlet UIButton *uploadButton;
@property (strong, nonatomic) NSArray *arrayOfOurQuotes;
@property (strong, nonatomic) NSMutableArray *ourNumberOfQuotes;

Look for nil values in your code. Since in Obj-C, calls to nil are handled. Do you have a `self.navigationController`? Is detailVC actually being created as well or not? — Shamas S, Aug 24 '15 at 13:05
Are you using a navigation controller? It's probably a view controller and that's why it doesn't work. Try `[seld presentViewController:]` — Pablo A., Aug 24 '15 at 13:07
There is a navigationController before my tableViewController. But i do i know if detailVC is being created? — Dinesh Sekar, Aug 24 '15 at 13:09
Add breakpoint at line where you're pushing view controller and after stop, type lldb command "po detailVC" — MichalMoskala, Aug 24 '15 at 13:21
As @iosDev82 says, do some debugging and check every line in that function runs and that the values are not nil for your storyboard and VC and navigationController. — Rory McKinnel, Aug 24 '15 at 13:22
@DineshSekar, change the line to [self presentViewController:] — Wyetro, Aug 24 '15 at 13:27
@RoryMcKinnel how would i check that those thing you stated are not nil? — Dinesh Sekar, Aug 24 '15 at 13:30
@WMios isn't it like the same line of code i have? Or are you trying to say something else? — Dinesh Sekar, Aug 24 '15 at 13:31
Change `[self.navigationController pushViewController:detailVC animated:YES];` to `[self pushViewController:detailVC animated:YES];` — Wyetro, Aug 24 '15 at 13:31
Also it's supposed to be [self presentViewController:detailVC animated:YES]; — Wyetro, Aug 24 '15 at 13:38
Well it's supposed to be presentViewController not pushViewController — Wyetro, Aug 24 '15 at 13:38
@DineshSekar Click on the left of the code line and you should see a break point added. When you run the code will stop on that line. Step over each line using the step over icon. Looks like a horizontal bar with a right angled arrow on top. Hover your mouse over the variable of interest and it will show you its value. You can also type `print ` in the console and it will print the value out. Or add lots of NSLog statements. Really you should know this and have done this before posting. — Rory McKinnel, Aug 24 '15 at 13:39
But the problem is that the void is not running. I can't tap on any of the cells. So even is i add breakpoints it is to of no use — Dinesh Sekar, Aug 24 '15 at 13:41
You should have posted the `- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath` method — Wyetro, Aug 24 '15 at 13:44
yes it is. Only the `didSelectRowAtIndexPath` is not running — Dinesh Sekar, Aug 24 '15 at 13:51
possible duplicate of [-didSelectRowAtIndexPath: not being called](http://stackoverflow.com/questions/255927/didselectrowatindexpath-not-being-called) — Wyetro, Aug 24 '15 at 13:53
If your code doesn't enter the didDelectRowAtIndexPath method, then you probably haven't set your tableView's delegate. — Natasha, Aug 24 '15 at 13:57
You have to implement UITableViewDelegate, that's like TableView 101... — Wyetro, Aug 24 '15 at 14:03
And you need [self.tableView setDelegate:self]; [self.tableView setDataSource:self]; — Wyetro, Aug 24 '15 at 14:04
I would check out the previous questions that ask this same thing and find what your missing. — Wyetro, Aug 24 '15 at 14:08
Yeah I'm not downloading your project. I'd much rather you include the relevant code for your question. — Wyetro, Aug 24 '15 at 14:18
thanks for your help and sorry for the inconvenience. I just realised my mistake. I had a tap gesture on the view itself. — Dinesh Sekar, Aug 24 '15 at 14:39
Ok, I've included an answer here with the information from the link I had posted above which answers your question. — Wyetro, Aug 24 '15 at 14:55

score 0 · Answer 1 · answered Aug 24 '15 at 13:11

Are you switching to a different storyboard that your current viewController is already using? If it's the same, please change the code within your method (see below) - This line here UIStoryboard... isn't needed.

    UIStoryboard *mainStoryboard = [UIStoryboard storyboardWithName:@"Main" bundle: nil];

isn't needed. You can call self.storyboard

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

   DetailViewController *detailVC = [self.storyboard instantiateViewControllerWithIdentifier:@"detailViewController"];
   detailVC.navigationItem.title = self.ourNumberOfQuotes[indexPath];
   detailVC.indexRow =  [NSString stringWithFormat:@"%@",indexPath];

   [self.navigationController pushViewController:detailVC animated:YES];
    }

Providing the detailVC has been set with the storyboard identifier @"detailViewController" of course.

Natasha · Answer 2 · 2015-08-24T14:49:11.277

0

Okay here is the problem, when I commented out your gesture codes from your viewDidLoad method, the delegate method, didSelectRowAtIndexPath gets called.

Here is what I figured, either the swipe gesture works and cell selection doesn't or if you remove the gesture part, the selection work and didSelectRowAtIndexPath gets called. As I never worked with gesture, I am not sure how to make both actions work together.

Hope this helps a little.

edited Aug 24 '15 at 14:49

answered Aug 24 '15 at 13:16

Natasha

6,651
3
36
58

the problem is that i cannot even go to the DetailViewCotroller. So what is the purpose of putting a breakpoint at the `-viewDidLoad` method when it doesn't even load – Dinesh Sekar Aug 24 '15 at 13:19
I gave you step by step debugging points. You would try to put breakpoint in DetailViewController if your code would have passed the first 2 steps. – Natasha Aug 24 '15 at 13:56
This is all correct here, you maybe forgot to include the reference in your .h – lukeswitz Aug 24 '15 at 14:03
I edited my response based on the updated question and looking into the sample code. – Natasha Aug 24 '15 at 14:41

score 0 · Accepted Answer · edited May 23 '17 at 12:29

From -didSelectRowAtIndexPath: not being called:

I have encountered two things in this situations.

You may have forgot to implement UITableViewDelegate protocol, or there's no delegation outlet between your class and your table view.

You might have a UIView inside your row that is a first responder and takes your clicks away. Say a UIButton or something similar.

You should check to make sure you have implemented the UITableViewDelegate protocol and you need to make sure that you are clicking the cell and not a view or gesture recognizer that is in front of it.

Segue is not working for tableview cell

3 Answers3