views:

191

answers:

2

I want to use this for an object factory: Given a string, create a Class, and if this Class supports a protocol (with a Create() method) then alloc the class and call Create.

+2  A: 
Class klass = NSClassFromString(classname);
if ([klass instancesRespondToSelector:@selector(create)]) {
  [[klass alloc] create];
}

May I, however, point out just how many awful Objective-C rules you're breaking by doing the above? For example, you should never be calling methods on an allocated-but-not-initialized instance. The Xcode Static Analyzer will give you all sorts of warnings about memory leaks.

A better option would be this:

[[[klass alloc] init] create];

But you seem to imply that you don't want to call init.

You could consider a class method: [klass create], which would return a non-owned instance of klass. Then you'd just check [klass respondsToSelector:@selector(create)] before calling it.

BJ Homer
Thanks. You're right, the Create method should be a class method.I think this is usually how object factories are designed.
Jacko
+3  A: 
NSString *className; //assume this exists
Class class = NSClassFromString(className);
if ([class conformsToProtocol:@protocol(SomeProtocol)]) {
    id instance = [[class alloc] init];
    [instance create];
}
Chuck
Cool. Thanks for the advice.
Jacko