iPhone: Using a Singleton with Tabview Controller and Navigation Controller

I have developed a small iPhone application by using singleton that I use to navigate through the views. Here is a sample method from my singleton class.

+ (void) loadMenuController:(NSMutableArray *)menuItems{
     MenuViewController *menuViewControler = [[MenuViewController alloc] initWithNibName:@"MenuViewController" bundle:nil];
     [menuViewControler setMenuItems:menuItems];
     RootViewController *root = (
     P2MAppDelegate *appDelegate = (P2MAppDelegate*) [[UIApplication sharedApplication] delegate];
     UINavigationController *navController = [appDelegate navigationController];
     [navController pushViewController:menuViewControler animated:YES]; 
     [menuViewControler release];
}

Now my requirement has changed to require a tab view controller . I could change my application delegate to a tabview controller but I still need to navigate inside each tab. I am unable get a clue how to navigate from my singleton class.

Please guide me. Please let me know if my query is not c开发者_高级运维lear.

Thanks in advance.

Regards, Malleswar

---

You shouldn't be using a singleton to manage the interface and even if you did, you wouldn't put the UI logic in a class method. You need to rethink your design from scratch.

The normal pattern is to hold the navigation controller or the tabbar controller as an attribute of the application delegate. The app delegate itself should not be a subclass of any controller but just a NSObject subclass that implements the application delegate protocol.

Look at the Apple supplied template projects in Xcode to see the quick and dirty way to structure apps built around navigation and/or tabs.

Singletons should only be used when you have to ensure that one and only one instance of class is alive at one time. You don't need to make your own singleton to manage the UI. The application delegate is attached to the application object which is itself a singleton. This means the app delegate provides all the restriction on class for the UI you might need. You don't need another singleton in addition to that.

Overuse of singletons is dangerous and can cause your design to get trapped in a dead end resulting in a massive rewrite. Think carefully before employing them.