I'm not sure if instantiating a view controller from a UIView subclass is good or bad ... but it has a bad smell for me.  I usually like sticking UIViewController allocation back in the UIViewController managing the UIView with a delegate method or something of the sort.