What's Wrong?

[loddy1234]

The idea of my app is the user is asked a question via a label, they then type the answer into a text box and presses a button. if they get the answer wrong three times, an "i give up" button becomes active and they can click it to show the answer

here is my .h file

#import <UIKit/UIKit.h>
#import "questionOne.h"

@interface questionOne : UIViewController {
	
	IBOutlet UITextField *answer;
	IBOutlet UIButton *giveUp;
	IBOutlet UIViewController *secondQuestion;

}

-(IBAction)nextQuestion: (id) sender;
-(IBAction)hideKeyboard: (id) sender;
-(IBAction)showAnswer: (id) sender;
@property (nonatomic, retain) IBOutlet UITextField *answer;
@property (nonatomic, retain) IBOutlet UIButton *giveUp;
@end

and here is my .m file

#import "questionOne.h"


@implementation questionOne;
@synthesize answer;
@synthesize giveUp;

NSInteger attempts = 0;


-(IBAction)showAnswer:(id) sender{
	
		answer.text = @"yes";
}



-(IBAction)nextQuestion:(id) sender{
	
	if (answer.text = @"yes"){
		[self presentModalViewController:secondQuestion animated:YES];
	
	}
	
	else {
		attempts ++;
	}


	if (attempts > 2) {
		
		giveUp.enabled = YES;
	}

}


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

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


- (void)dealloc {
    
	[giveUp release];
	[answer release];
	[super dealloc];
}


@end

showAnswer is linked to a button
nextQuestion is linked to a button
secondQuestion is linked to a view controller

When the nextQuestion button is pressed, the text box is set to yes and the next view is shown. Why does it do this???
Any help greatly appreciated.
Thanks in advance

[SoulRed12]

The only possibility that I see is that you've got your nextQuestion button connected to BOTH showAnswer AND nextQuestion. Did you, by any chance, first connect the button accidentally to showAnswer and then try to correct it by connecting the button to nextQuestion? If so, you might have thought the second connection erases the first one, but instead, it just told the button to call BOTH methods.

You're going to want to go to the connections tab of the nextQuestion button, and under "events", find where it says something like "Multiple", click the little "x" button there, and reconnect it to only the nextQuestion method.

Logically (assuming IB was set up correctly), your code works, so if this doesn't do it, it has to be something with IB (or otherwise something outside the scope of this code). In that case try deleting both buttons and recreating and reconnecting them from scratch.

[loddy1234]

Originally Posted by SoulRed12

The only possibility that I see is that you've got your nextQuestion button connected to BOTH showAnswer AND nextQuestion. Did you, by any chance, first connect the button accidentally to showAnswer and then try to correct it by connecting the button to nextQuestion? If so, you might have thought the second connection erases the first one, but instead, it just told the button to call BOTH methods.

You're going to want to go to the connections tab of the nextQuestion button, and under "events", find where it says something like "Multiple", click the little "x" button there, and reconnect it to only the nextQuestion method.

Logically (assuming IB was set up correctly), your code works, so if this doesn't do it, it has to be something with IB (or otherwise something outside the scope of this code). In that case try deleting both buttons and recreating and reconnecting them from scratch.

The enter button was only connected to nextQuestion but i deleted both buttons and re-linked them but it still doesn't work

[kapps11]

No, the problem is in your logic statements.

you used:

if (answer.text = @"yes")

first off, == is the logic statement operand, not =. But saying =, you are actually setting the text to @"yes" and that is why that happens. However, to compare two nsstrings, you need to use the isEqualToString method. So your new if statemtent would look like:

if ([answer.text isEqualTo:@"yes") {
//do stuff
}

[SoulRed12]

Originally Posted by kapps11

No, the problem is in your logic statements.

you used:

if (answer.text = @"yes")

first off, == is the logic statement operand, not =. But saying =, you are actually setting the text to @"yes" and that is why that happens. However, to compare two nsstrings, you need to use the isEqualToString method. So your new if statemtent would look like:

if ([answer.text isEqualTo:@"yes") {
//do stuff
}

Ah! I can't believe I missed that! Sorry OP >_<

[kapps11]

oops sorry! actually its isEqualToString:

my bad hah