Ошибка приложения iOS - невозможно добавить себя в качестве подпредставления

Я получил этот отчет о сбое, но я не знаю, как его отладить.

Fatal Exception NSInvalidArgumentException
Can't add self as subview
0 ...    CoreFoundation  __exceptionPreprocess + 130
1    libobjc.A.dylib     objc_exception_throw + 38
2    CoreFoundation  -[NSException initWithCoder:]
3    UIKit   -[UIView(Internal) _addSubview:positioned:relativeTo:] + 110
4    UIKit   -[UIView(Hierarchy) addSubview:] + 30
5    UIKit   __53-[_UINavigationParallaxTransition animateTransition:]_block_invoke + 1196
6    UIKit   +[UIView(Animation) performWithoutAnimation:] + 72
7    UIKit   -[_UINavigationParallaxTransition animateTransition:] + 732
8    UIKit   -[UINavigationController _startCustomTransition:] + 2616
9    UIKit   -[UINavigationController _startDeferredTransitionIfNeeded:] + 418
10   UIKit   -[UINavigationController __viewWillLayoutSubviews] + 44
11   UIKit   -[UILayoutContainerView layoutSubviews] + 184
12   UIKit   -[UIView(CALayerDelegate) layoutSublayersOfLayer:] + 346
13   QuartzCore  -[CALayer layoutSublayers] + 142
14   QuartzCore  CA::Layer::layout_if_needed(CA::Transaction*) + 350
15   QuartzCore  CA::Layer::layout_and_display_if_needed(CA::Transaction*) + 16
16   QuartzCore  CA::Context::commit_transaction(CA::Transaction*) + 228
17   QuartzCore  CA::Transaction::commit() + 314
18   QuartzCore  CA::Transaction::observer_callback(__CFRunLoopObserver*, unsigned long, void*) + 56

Версия iOS 7.0.3. Кто-нибудь испытал эту странную аварию?

ОБНОВИТЬ:

Я не знаю, где в моем коде произошел этот сбой, поэтому я не могу опубликовать код здесь, извините.

Второе ОБНОВЛЕНИЕ

Смотрите ответ ниже.

Я размышляю, основываясь на чем-то похожем, что я недавно отлаживал ... если вы нажимаете (или выдвигаете) контроллер представления с помощью Animated: ДА, он не завершается сразу же, и происходят плохие вещи, если вы делаете еще одно нажатие или высовываете перед анимацией завершается. Вы можете легко проверить, действительно ли это так, временно изменив операции Push и Pop на Animated: NO (чтобы они выполнялись синхронно) и посмотрев, устраняет ли это сбой. Если это действительно ваша проблема, и вы хотите снова включить анимацию, то правильная стратегия - реализовать протокол UINavigationControllerDelegate. Это включает в себя следующий метод, который вызывается после завершения анимации:

navigationController:didShowViewController:animated:

По сути, вы хотите переместить некоторый код по мере необходимости в этот метод, чтобы гарантировать, что никакие другие действия, которые могут вызвать изменение в стеке NavigationController, не будут выполняться, пока анимация не будет завершена и стек не будет готов к дополнительным изменениям.

Мы также начали сталкиваться с этой проблемой, и, скорее всего, наша проблема была вызвана той же проблемой.

В нашем случае нам приходилось извлекать данные из серверной части в некоторых случаях, что означало, что пользователь может что-то нажать, и тогда перед навигацией будет небольшая задержка. Если пользователь быстро переключается, он может получить два навигационных нажатия от одного и того же контроллера представления, что вызвало именно это исключение.

Наше решение - это категория в UINavigationController, которая предотвращает нажатия / всплывающие сообщения, если только верхний vc не является тем же самым в данный момент времени.

.h файл:

@interface UINavigationController (SafePushing)

- (id)navigationLock; ///< Obtain "lock" for pushing onto the navigation controller

- (void)pushViewController:(UIViewController *)viewController animated:(BOOL)animated navigationLock:(id)navigationLock; ///< Uses a horizontal slide transition. Has no effect if the view controller is already in the stack. Has no effect if navigationLock is not the current lock.
- (NSArray *)popToViewController:(UIViewController *)viewController animated:(BOOL)animated navigationLock:(id)navigationLock; ///< Pops view controllers until the one specified is on top. Returns the popped controllers. Has no effect if navigationLock is not the current lock.
- (NSArray *)popToRootViewControllerAnimated:(BOOL)animated navigationLock:(id)navigationLock; ///< Pops until there's only a single view controller left on the stack. Returns the popped controllers. Has no effect if navigationLock is not the current lock.

@end

.m файл:

@implementation UINavigationController (SafePushing)

- (id)navigationLock
{
    return self.topViewController;
}

- (void)pushViewController:(UIViewController *)viewController animated:(BOOL)animated navigationLock:(id)navigationLock
{
    if (!navigationLock || self.topViewController == navigationLock) 
        [self pushViewController:viewController animated:animated];
}

- (NSArray *)popToRootViewControllerAnimated:(BOOL)animated navigationLock:(id)navigationLock
{
    if (!navigationLock || self.topViewController == navigationLock)
        return [self popToRootViewControllerAnimated:animated];
    return @[];
}

- (NSArray *)popToViewController:(UIViewController *)viewController animated:(BOOL)animated navigationLock:(id)navigationLock
{
    if (!navigationLock || self.topViewController == navigationLock)
        return [self popToViewController:viewController animated:animated];
    return @[];
}

@end

Пока что это, похоже, решило проблему для нас. Пример:

id lock = _dataViewController.navigationController.navigationLock;
[[MyApi sharedClient] getUserProfile:_user.id success:^(MyUser *user) {
    ProfileViewController *pvc = [[ProfileViewController alloc] initWithUser:user];
    [_dataViewController.navigationController pushViewController:pvc animated:YES navigationLock:lock];
}];

По сути, правило таково: перед любыми задержками, не связанными с пользователем, захватите блокировку у соответствующего контроллера Nav и включите ее в вызов push / pop.

Слово «блокировка» может быть слегка некачественным, поскольку оно может указывать на наличие какой-либо формы блокировки, требующей разблокировки, но, поскольку нигде нет метода «разблокировки», вероятно, все в порядке.

(Как примечание, «не связанные с пользователем задержки» - это любые задержки, вызываемые кодом, то есть что-либо асинхронное. Пользователи, нажимающие на контроллер Nav, который анимированно толкается, не учитываются, и для них не нужно делать версию navigationLock: version). случаев.)

Этот код решает проблему: https://gist.github.com/nonamelive/9334458

Он использует частный API, но я могу подтвердить, что он безопасен в App Store. (Одно из моих приложений, использующих этот код, было одобрено App Store.)

@interface UINavigationController (DMNavigationController)

- (void)didShowViewController:(UIViewController *)viewController animated:(BOOL)animated;

@end

@interface DMNavigationController ()

@property (nonatomic, assign) BOOL shouldIgnorePushingViewControllers;

@end

@implementation DMNavigationViewController

#pragma mark - Push

- (void)pushViewController:(UIViewController *)viewController animated:(BOOL)animated
{
    if (!self.shouldIgnorePushingViewControllers)
    {
        [super pushViewController:viewController animated:animated];
    }

    self.shouldIgnorePushingViewControllers = YES;
}

#pragma mark - Private API

// This is confirmed to be App Store safe.
// If you feel uncomfortable to use Private API, you could also use the delegate method navigationController:didShowViewController:animated:.
- (void)didShowViewController:(UIViewController *)viewController animated:(BOOL)animated
{
    [super didShowViewController:viewController animated:animated];
    self.shouldIgnorePushingViewControllers = NO;
}

Я опишу более подробную информацию об этой аварии в моем приложении и пометить это как ответ.

Мое приложение имеет UINavigationController с корневым контроллером UITableViewController, который содержит список объектов заметок. Объект заметки имеет свойство содержимого в html. Выбор заметки перейдет в детальный контроллер.

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath
{
    //get note object
    DetailViewController *controller = [[DetailViewController alloc] initWithNote:note];
    [self.navigationController pushViewController:controller animated:YES];
}

Детальный контроллер

Этот контроллер имеет UIWebView, отображать содержимое заметки, переданной от корневого контроллера.

- (void)viewDidLoad
{
    ...
    [_webView loadHTMLString:note.content baseURL:nil];
    ...
}

Этот контроллер является делегатом элемента управления webview. Если заметка содержит ссылки, нажмите на ссылку, чтобы перейти в веб-браузер приложения.

- (BOOL)webView:(UIWebView *)webView shouldStartLoadWithRequest:(NSURLRequest *)request navigationType:(UIWebViewNavigationType)navigationType
{
    WebBrowserViewController *browserController = [[WebBrowserViewController alloc] init];
    browserController.startupURL = request.URL;
    [self.navigationController pushViewController:webViewController animated:YES];
    return NO;
}

Я получал выше отчет о сбое каждый день. Я не знаю, где в моем коде произошел этот сбой. После некоторых исследований с помощью пользователя, я наконец смог исправить эту ошибку. Этот HTML-контент вызовет сбой:

...
<iframe src="http://google.com"></iframe>
...

В методе viewDidLoad подробного контроллера я загрузил этот html в элемент управления webview, сразу после этого вышеупомянутый метод делегата был вызван немедленно с request.URL - источник iframe (google.com). Этот метод делегата вызывает метод pushViewController, находясь в viewDidLoad => crash!

Я исправил этот сбой, проверив навигационный тип:

- (BOOL)webView:(UIWebView *)webView shouldStartLoadWithRequest:(NSURLRequest *)request navigationType:(UIWebViewNavigationType)navigationType
{
    if (navigationType != UIWebViewNavigationTypeOther)
    {
        //go to web browser controller
    }
}

Надеюсь это поможет

У меня была та же проблема, что просто работало для меня, менял Animated: Yes на Animated: No.

Похоже, проблема была в том, что анимация не была завершена вовремя.

Надеюсь, это кому-нибудь поможет.

Чтобы воспроизвести эту ошибку, попробуйте нажать два контроллера вида одновременно. Или толкать и высовывать одновременно. Пример:

Я создал категорию, которая перехватывает эти вызовы и делает их безопасными, следя за тем, чтобы никаких других нажатий не происходило во время выполнения. Просто скопируйте код в свой проект, и из-за метода метания вам будет хорошо.

#import "UINavigationController+Consistent.h"
#import <objc/runtime.h>
/// This char is used to add storage for the isPushingViewController property.
static char const * const ObjectTagKey = "ObjectTag";

@interface UINavigationController ()
@property (readwrite,getter = isViewTransitionInProgress) BOOL viewTransitionInProgress;

@end

@implementation UINavigationController (Consistent)

- (void)setViewTransitionInProgress:(BOOL)property {
    NSNumber *number = [NSNumber numberWithBool:property];
    objc_setAssociatedObject(self, ObjectTagKey, number , OBJC_ASSOCIATION_RETAIN);
}


- (BOOL)isViewTransitionInProgress {
    NSNumber *number = objc_getAssociatedObject(self, ObjectTagKey);

    return [number boolValue];
}


#pragma mark - Intercept Pop, Push, PopToRootVC
/// @name Intercept Pop, Push, PopToRootVC

- (NSArray *)safePopToRootViewControllerAnimated:(BOOL)animated {
    if (self.viewTransitionInProgress) return nil;
    if (animated) {
        self.viewTransitionInProgress = YES;
    }
    //-- This is not a recursion, due to method swizzling the call below calls the original  method.
    return [self safePopToRootViewControllerAnimated:animated];

}


- (NSArray *)safePopToViewController:(UIViewController *)viewController animated:(BOOL)animated {
    if (self.viewTransitionInProgress) return nil;
    if (animated) {
        self.viewTransitionInProgress = YES;
    }
    //-- This is not a recursion, due to method swizzling the call below calls the original  method.
    return [self safePopToViewController:viewController animated:animated];
}


- (UIViewController *)safePopViewControllerAnimated:(BOOL)animated {
    if (self.viewTransitionInProgress) return nil;
    if (animated) {
        self.viewTransitionInProgress = YES;
    }
    //-- This is not a recursion, due to method swizzling the call below calls the original  method.
    return [self safePopViewControllerAnimated:animated];
}



- (void)safePushViewController:(UIViewController *)viewController animated:(BOOL)animated {
    self.delegate = self;
    //-- If we are already pushing a view controller, we dont push another one.
    if (self.isViewTransitionInProgress == NO) {
        //-- This is not a recursion, due to method swizzling the call below calls the original  method.
        [self safePushViewController:viewController animated:animated];
        if (animated) {
            self.viewTransitionInProgress = YES;
        }
    }
}


// This is confirmed to be App Store safe.
// If you feel uncomfortable to use Private API, you could also use the delegate method navigationController:didShowViewController:animated:.
- (void)safeDidShowViewController:(UIViewController *)viewController animated:(BOOL)animated {
    //-- This is not a recursion. Due to method swizzling this is calling the original method.
    [self safeDidShowViewController:viewController animated:animated];
    self.viewTransitionInProgress = NO;
}


// If the user doesnt complete the swipe-to-go-back gesture, we need to intercept it and set the flag to NO again.
- (void)navigationController:(UINavigationController *)navigationController willShowViewController:(UIViewController *)viewController animated:(BOOL)animated {
    id<UIViewControllerTransitionCoordinator> tc = navigationController.topViewController.transitionCoordinator;
    [tc notifyWhenInteractionEndsUsingBlock:^(id<UIViewControllerTransitionCoordinatorContext> context) {
        self.viewTransitionInProgress = NO;
        //--Reenable swipe back gesture.
        self.interactivePopGestureRecognizer.delegate = (id<UIGestureRecognizerDelegate>)viewController;
        [self.interactivePopGestureRecognizer setEnabled:YES];
    }];
    //-- Method swizzling wont work in the case of a delegate so:
    //-- forward this method to the original delegate if there is one different than ourselves.
    if (navigationController.delegate != self) {
        [navigationController.delegate navigationController:navigationController
                                     willShowViewController:viewController
                                                   animated:animated];
    }
}


+ (void)load {
    //-- Exchange the original implementation with our custom one.
    method_exchangeImplementations(class_getInstanceMethod(self, @selector(pushViewController:animated:)), class_getInstanceMethod(self, @selector(safePushViewController:animated:)));
    method_exchangeImplementations(class_getInstanceMethod(self, @selector(didShowViewController:animated:)), class_getInstanceMethod(self, @selector(safeDidShowViewController:animated:)));
    method_exchangeImplementations(class_getInstanceMethod(self, @selector(popViewControllerAnimated:)), class_getInstanceMethod(self, @selector(safePopViewControllerAnimated:)));
    method_exchangeImplementations(class_getInstanceMethod(self, @selector(popToRootViewControllerAnimated:)), class_getInstanceMethod(self, @selector(safePopToRootViewControllerAnimated:)));
    method_exchangeImplementations(class_getInstanceMethod(self, @selector(popToViewController:animated:)), class_getInstanceMethod(self, @selector(safePopToViewController:animated:)));
}

@end

Только что столкнулся с этой проблемой. Позвольте мне показать вам мой код:

override func viewDidLoad() { 
  super.viewDidLoad()

  //First, I create a UIView
  let firstFrame = CGRect(x: 50, y: 70, height: 200, width: 200)
  let firstView = UIView(frame: firstFrame)
  firstView.addBackgroundColor = UIColor.yellow
  view.addSubview(firstView) 

  //Now, I want to add a subview inside firstView
  let secondFrame = CGRect(x: 20, y:50, height: 15, width: 35)
  let secondView = UIView(frame: secondFrame)
  secondView.addBackgroundColor = UIColor.green
  firstView.addSubView(firstView)
 }

Ошибка появляется из-за этой строки:

firstView.addSubView(firstView)

Вы не можете добавить себя в подпредставление. Я изменил строку кода на:

firstView.addSubView(secondView)

Ошибка ушла, и я смог увидеть оба мнения. Просто подумал, что это поможет любому, кто хочет увидеть пример.

Найдите в вашем коде «addSubview».

В одном из мест, где вы вызывали этот метод, вы пытались добавить представление в его собственный массив подвидов, используя этот метод.

Например:

[self.view addSubview:self.view];

Или:

[self.myLabel addSubview:self.myLabel];

Я думаю, что нажатие / выталкивание контроллеров представления с анимацией в любой момент должно быть прекрасно, и SDK должен любезно обработать очередь вызовов для нас.

Следовательно, это не так, и все решения пытаются игнорировать последующие изменения, что можно считать ошибкой, поскольку окончательный стек навигации не соответствует предназначению кода.

Вместо этого я реализовал очередь push-вызовов:

// SafeNavigationController.h

@interface SafeNavigationController : UINavigationController
@end

// SafeNavigationController.m

#define timeToWaitBetweenAnimations 0.5

@interface SafeNavigationController ()

@property (nonatomic, strong) NSMutableArray * controllersQueue;
@property (nonatomic)         BOOL animateLastQueuedController;
@property (nonatomic)         BOOL pushScheduled;
@property (nonatomic, strong) NSDate * lastAnimatedPushDate;

@end

@implementation SafeNavigationController

- (void)awakeFromNib
{
    [super awakeFromNib];

    self.controllersQueue = [NSMutableArray array];
}

- (void)pushViewController:(UIViewController *)viewController
                  animated:(BOOL)animated
{
    [self.controllersQueue addObject:viewController];
    self.animateLastQueuedController = animated;

    if (self.pushScheduled)
        return;

    // Wait for push animation to finish
    NSTimeInterval timeToWait = self.lastAnimatedPushDate ? timeToWaitBetweenAnimations + [self.lastAnimatedPushDate timeIntervalSinceNow] : 0.0;
    dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)((timeToWait > 0.0 ? timeToWait : 0.0) * NSEC_PER_SEC)),
                   dispatch_get_main_queue(), ^
                   {
                       [self pushQueuedControllers];

                       self.lastAnimatedPushDate = self.animateLastQueuedController ? [NSDate date] : nil;
                       self.pushScheduled = NO;
                   });
    self.pushScheduled = YES;
}

- (void)pushQueuedControllers
{
    for (NSInteger index = 0; index < (NSInteger)self.controllersQueue.count - 1; index++)
    {
        [super pushViewController:self.controllersQueue[index]
                         animated:NO];
    }
    [super pushViewController:self.controllersQueue.lastObject
                     animated:self.animateLastQueuedController];

    [self.controllersQueue removeAllObjects];
}

@end

Он не обрабатывает смешанные очереди push и pops, но это хороший старт для исправления большинства наших сбоев.

Суть: https://gist.github.com/rivera-ernesto/0bc628be1e24ff5704ae

Извините за опоздание на вечеринку. У меня недавно была эта проблема, когда моя навигационная панель переходила в поврежденное состояние из-за одновременного нажатия более одного контроллера представления. Это происходит потому, что другой контроллер представления перемещается, пока первый контроллер представления все еще анимируется. Взяв подсказку из неамелевского ответа, я нашел простое решение, которое работает в моем случае. Вам просто нужно создать подкласс UINavigationControllerи переопределить метод pushViewController и проверить, завершена ли предыдущая анимация контроллера представления. Вы можете прослушать завершение анимации, сделав свой класс делегатом UINavigationControllerDelegateи задав для него делегат self.

Я загрузил суть здесь, чтобы все было просто.

Просто убедитесь, что вы установили этот новый класс как NavigationController в вашей раскадровке.

Основываясь на замечательном совете @RobP, я создал подкласс UINavigationController , чтобы избежать подобных проблем. Он обрабатывает нажатия и / или нажатия, и вы можете безопасно выполнить:

[self.navigationController pushViewController:vc1 animated:YES];
[self.navigationController pushViewController:vc2 animated:YES];
[self.navigationController pushViewController:vc3 animated:YES];
[self.navigationController popViewControllerAnimated:YES];

Если 'acceptConflictingCommands' помечает его как истинное (по умолчанию), пользователь увидит анимированное нажатие vc1, vc2, vc3, а затем увидит анимированный щелчок vc3. Если значение acceptConflictingCommands равно false, все запросы push / pop будут отбрасываться до тех пор, пока vc1 не будет полностью передан - следовательно, остальные 3 вызова будут отброшены.

Решение nonamelive является удивительным. Но если вы не хотите использовать частный API, вы можете просто достичь UINavigationControllerDelegateметода. Или вы можете изменить анимацию YESна NO. Вот пример кода, вы можете наследовать его. Надеюсь, это полезно :)

https://github.com/antrix1989/ANNavigationController

Я много раз искал эту проблему, возможно, она выдвигает два или более VC одновременно, что вызывает проблему с анимацией, вы можете сослаться на это: Can't Add Self as Subview 崩溃 解决 办法

просто убедитесь, что есть один VC на переходный процесс одновременно, удачи.

Иногда вы по ошибке пытались добавить представление к собственному представлению.

halfView.addSubview(halfView)

измените это на ваш подвид.

halfView.addSubview(favView)

Я также столкнулся с этой проблемой. Когда я сделал анализ журнала Firebase, я обнаружил, что эта проблема возникает только при холодном запуске приложения. Поэтому я написал демо, которое может воспроизвести этот сбой.

,

Я также обнаружил, что при отображении корневого viewcontroller окна выполнение нескольких нажатий больше не вызовет той же проблемы. (Вы можете комментировать testColdStartUp (rootNav) в AppDelegate.swift и раскомментировать комментарий testColdStartUp () в ViewController.swift)

PS: я проанализировал сцену этого сбоя в моем приложении. Когда пользователь нажимает push-уведомление для холодного запуска приложения, приложение все еще находится на странице Launch и нажимает еще один push для перехода. В это время в приложении может появиться Crash. Мое текущее решение - кэшировать принудительный запуск или «холодный» запуск универсальной ссылки, чтобы открыть страницу перехода приложения, дождаться отображения rootviewcontroller и затем отложить выполнение.

попробуйте навигацию, используя метод задержки, для завершения последней анимации навигации,

[self performSelector:<#(SEL)#> withObject:<#(id)#> afterDelay:<#(NSTimeInterval)#>]

Представление не может быть добавлено в качестве подпредставления само по себе.

Представления поддерживают иерархию «родитель-потомок», поэтому, если вы добавляете представление как подпредставление само по себе, оно будет происходить через исключение.

если класс UIViewController, то чтобы получить его представление, вы используете self.view.

если класс UIView Class, то чтобы получить его представление, вы используете self.

Вы не можете добавить себя как подпредставление, если это будет класс UiViewController. Вы можете добавить себя как подпредставление, если это будет класс UiView.

Если вы хотите добавить Subview к View, вы можете сделать это следующим образом;

UIView *mainview = [[UIView alloc] initWithFrame:CGRectMake(0, 0, [UIScreen mainScreen].bounds.size.width, [UIScreen mainScreen].bounds.size.height)]; //Creats the mainview
    UIView *subview = [[UIView alloc] initWithFrame:CGRectMake(100, 100, 100, 100)]; //Creates the subview, you can use any kind of Views (UIImageView, UIWebView, UIView…)

    [mainview addSubview:subview]; //Adds subview to mainview