If statement causes a 'SIGABRT' signal to be received

[Sentis]

Hi Everyone,

In my program, I have a table. When the user selects a certain cell, I want it to take them to one of two different views, depending on which they pressed.

I am using an IF/ELSE statement to achieve this, however, I think there may be a problem with my code, because the program freezes up both on my Iphone, and in the simulator, and Xcode displays a message saying:

GDB: Program received signal: "SIGABRT"

So here is the IF statement, please have a look over it, and tell me what I am doing wrong, or give me some tips:

if (selectedItem =  glossary) {
		DisclosureViewController *dsController = [[DisclosureViewController alloc] initWithNibName:@"DisclosureView" bundle:[NSBundle mainBundle]];
 		[self.navigationController pushViewController:dsController animated:YES];
		[dsController release];
		dsController = nil;
		
	}
	else {
		DetailViewController *dvController = [[DetailViewController alloc] initWithNibName:@"DetailView" bundle:[NSBundle mainBundle]];
		dvController.selectedItem = selectedItem;
		[self.navigationController pushViewController:dvController animated:YES];
		[dvController release];
		dvController = nil;
	}

Thanks in advance,

Aaron

[bravetarget]

did you mean to do == instead of =?

[Sentis]

I added that in, and it still displays the SIGABRT error when I select the cell for the different view.

Thanks

[Sentis]

This might help.
These are the couple lines of code relating to the problem.

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath {
		NSString *selectedItem =[listOfItems objectAtIndex:indexPath.row];
		NSString *glossary = @"Water";

thanks!

[bravetarget]

where do you allocate listofitems array?

[Sentis]

In the ViewdidLoad method:

- (void)viewDidLoad {
    [super viewDidLoad];
	
	
	listOfItems = [[NSMutableArray alloc] init];
	
	
	[listOfItems addObject:@"Periodic Table"];

[ChrisL]

It's getting late, so I could be missing something, but I don't see any obvious issues with anything you've posted. You'll probably get better help if you post the entire class (or, if that's too big, at the very least the complete code to the didSelectRowAtIndexPath: method). It's very hard to debug when there's only a few brief excerpts of code scattered across multiple posts.

[Sentis]

Here is the full code:

#import "RootViewController.h"
#import "TableViewAppDelegate.h"
#import "DetailViewController.h"
#import "DisclosureViewController.h"

@implementation RootViewController

- (void)viewDidLoad {
    [super viewDidLoad];
	
	
	listOfItems = [[NSMutableArray alloc] init];
	
	
	[listOfItems addObject:@"Periodic Table"];
	[listOfItems addObject:@"Acids & Bases"];
	[listOfItems addObject:@"Metals"];
	[listOfItems addObject:@"Water"];

self.navigationItem.title = @"Blah";

- (void)didReceiveMemoryWarning {
    [super didReceiveMemoryWarning]; }


- (NSInteger)numberOfSectionsInTableView:(UITableView *)tableView {
    return 1;
}



- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
    return[listOfItems count];
}



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


    static NSString *CellIdentifier = @"Cell";
    
    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
    if (cell == nil) {
        cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault
									   reuseIdentifier:CellIdentifier] autorelease];
		//cell.accessoryType = UITableViewCellAccessoryDetailDisclosureButton;
	}	

    
  	
	NSString *cellValue =[listOfItems objectAtIndex:indexPath.row];
	cell.textLabel.text = cellValue;


    return cell;
	}


- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath {
		NSString *selectedItem =[listOfItems objectAtIndex:indexPath.row];
		NSString *glossary = @"Water";
	
	if (selectedItem ==  glossary) {
		DisclosureViewController *dsController = [[DisclosureViewController alloc] initWithNibName:@"DisclosureView" bundle:[NSBundle mainBundle]];
 		[self.navigationController pushViewController:dsController animated:YES];
		[dsController release];
		dsController = nil;
		
	}
	else {
		DetailViewController *dvController = [[DetailViewController alloc] initWithNibName:@"DetailView" bundle:[NSBundle mainBundle]];
		dvController.selectedItem = selectedItem;
		[self.navigationController pushViewController:dvController animated:YES];
		[dvController release];
		dvController = nil;
	}

	
}
	


- (void)dealloc {
	
	[listOfItems release];
	[selectedItem release];
	[glossary release];
    [super dealloc];
}


@end

There are also a couple of warnings about local declarations of 'selectedItem' and 'glossary' hiding the instance variable ould that have anything to do with it?

thanks again!

[Sentis]

I found the problem.

The If statement is fine! the problem occurs sometime during the loading of the 'Disclosure View'. But i have it worked out.

Thanks again for all of your help!

[pereorra]

I think the problem is in

if (selectedItem ==  glossary)

You can't compare strings with ==

Try

if ([selectedItem isEqualToString: glossary])