…
// Sending ‘CYOAViewController *’ to parameter of incompatible type ‘id ‘
// space added between id and bracket because your comment handler is parsing HTML… off buttons are good
delegate:self
…
// Class ‘CYOAViewController’ does not implement the ‘UIActionSheetDelegate’ protocol
otherButtleTitles:[currentPiece choice1], @”Retry”, nil];
…

You’re also using alert views as the primary interface. Quick, but sloppy.

If I ever get started on my own app (the one that uses this kind of prompt/action structure), I’d be happy to give you some credit for getting it started. Throw me an email if you’re interested.

On an unrelated note, Safari’s “Undo Close Tab” feature should have an off button. Or at least a different shortcut. I can’t undo typing if I closed a tab recently -.-“

…
// Sending ‘CYOAViewController *’ to parameter of incompatible type ‘id’
delegate:self
…
// Class ‘CYOAViewController’ does not implement the ‘UIActionSheetDelegate’ protocol
otherButtleTitles:[currentPiece choice1], @”Retry”, nil];
…

You’re also using alert views as the primary interface. Quick, but sloppy.

If I ever get started on my own app (the one that uses this kind of prompt/action structure), I’d be happy to give you some credit for getting it started. Throw me an email if you’re interested.

On an unrelated note, Safari’s “Undo Close Tab” feature should have an off button. Or at least a different shortcut. I can’t undo typing if I closed a tab recently -.-“

In your actionSheet:clickedButtonAtIndex: method, you call [self makeTheStory] for the second time. This is an issue because the previously allocated AdventurePiece’s are still out there in memory, and are inaccessible since they were created as on the stack instances. There are two approaches to fixing this.

1) Make the objects part of the autorelease pool by (I believe) changing the constructor of Adventure piece from:
if(self = [super init])
TO:
if(self = [super autorelease])
If I am wrong about that I’m sorry…

2) Make the following changes to use self managed memory: (I did not compile this so it may need some tweaking!)
In your AdventurePiece.m write the following destructor:
– (void) dealloc
{
if(choice1) // If object != nil
[choice1 release];
if(choice2) // If object != nil
[choice2 release];
[super dealloc];
}

And then you must maintain a reference to the first piece to start the chain reaction of deletion. Add to CYOAViewController.h:
AdventurePiece *startPiece;
(And add it to the synthesize in the .m)

Add to the makeTheStory method at the last line:
startPiece = currentPiece;
This copies the address so that when currentPiece changes the top level (starting) piece is saved.

Then, finally, in the actionSheet:clickedButtonAtIndex: method, call:
[startPiece release];
Before you call [self makeTheStory]; that way the entire story is deleted and remade. The other way is to instead of releasing, you could just say currentPiece = startPiece; and reuse the same story, then don’t call [self makeTheStory], but either way works…