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です。誰かがこの奇妙なクラッシュを経験していますか？

更新：

コードのどこでこのクラッシュが発生したのかわからないので、コードをここに投稿することはできません。

2回目の更新

以下の答えを見てください。

私は最近デバッグしたのと同じようなものに基づいて推測しています... Animated：YESでビューコントローラーをプッシュ（またはポップ）すると、すぐには完了しません。アニメーションの前にもう一度プッシュまたはポップすると、悪いことが起こります。完了します。これが本当に当てはまるかどうかを簡単にテストするには、プッシュ操作とポップ操作を一時的にAnimated：NOに変更して（同期的に完了するように）、クラッシュが解消されるかどうかを確認します。これが本当に問題であり、アニメーションをオンに戻したい場合、正しい戦略はUINavigationControllerDelegateプロトコルを実装することです。これには、アニメーションの完了後に呼び出される次のメソッドが含まれます。

navigationController:didShowViewController:animated:

基本的に、必要に応じていくつかのコードをこのメソッドに移動し、アニメーションが終了してスタックがさらに変更できるようになるまで、NavigationControllerスタックを変更する可能性のある他のアクションが発生しないようにします。

この問題も発生し始めており、同じ問題が原因である可能性が非常に高いです。

私たちのケースでは、場合によってはバックエンドからデータをプルする必要がありました。これは、ユーザーが何かをタップする可能性があり、ナビゲーションプッシュが発生するまでに少し遅延が生じることを意味していました。ユーザーがすばやくタップしていると、同じビューコントローラーから2つのナビゲーションプッシュが発生し、この例外が発生する可能性があります。

私たちのソリューションは、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];
}];

基本的に、ルールは次のとおりです。ユーザーに関連しない遅延が発生する前に、関連するナビゲーションコントローラーからロックを取得し、それをpush / popへの呼び出しに含めます。

「ロック」という言葉は、ロック解除を必要とする何らかの形式のロックが発生していることをほのめかす可能性があるため、少し不適切な表現かもしれませんが、「ロック解除」メソッドはどこにもないので、おそらく大丈夫です。

（補足として、「ユーザーに関連しない遅延」とは、コードが引き起こしている遅延、つまり非同期のことです。アニメーションでプッシュされるナビゲーションコントローラーをタップしてもカウントされず、navigationLock：バージョンを実行する必要はありません。ケース）

このコードは問題を解決します：https : //gist.github.com/nonamelive/9334458

プライベートAPIを使用していますが、App Storeで安全であることを確認できます。（このコードを使用する私のアプリの1つが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です。noteオブジェクトには、htmlのcontentプロパティがあります。ノートを選択すると、詳細コントローラに移動します。

- (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コントロールのデリゲートです。ノートにリンクが含まれている場合は、リンクをタップしてアプリ内Webブラウザーに移動します。

- (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ですぐに呼び出されました。URLはiframeのソース（google.com）です。このデリゲートメソッドは、viewDidLoadでpushViewControllerメソッドを呼び出します=>クラッシュ！

このクラッシュを解決するには、navigationTypeを確認します。

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

お役に立てれば

同じ問題がありましたが、単純に機能したのはAnimated：YesをAnimated：Noに変更することでした。

この問題はアニメーションが時間内に完了しなかったことが原因であるようです。

これが誰かを助けることを願っています。

このバグを再現するには、2つのView Controllerを同時にプッシュしてみてください。または同時に押してポップします。例：

これらの呼び出しをインターセプトし、進行中に他のプッシュが発生しないようにすることで、それらを安全にするカテゴリを作成しました。コードをプロジェクトにコピーするだけで、メソッドのスウィズリングにより、問題ありません。

#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」を検索します。

このメソッドを呼び出した場所の1つで、このメソッドを使用して独自のサブビュー配列にビューを追加しようとしました。

例えば：

[self.view addSubview:self.view];

または：

[self.myLabel addSubview:self.myLabel];

任意の時点でアニメーションを使用してビューコントローラーをプッシュ/ポップすることは完全に問題なく、SDKは呼び出しのキューを丁寧に処理する必要があると思います。

したがって、そうではなく、すべてのソリューションが後続のプッシュを無視しようとします。これは、最終的なナビゲーションスタックがコードが意図したものではないため、バグと見なされる可能性があります。

代わりにプッシュコールキューを実装しました。

// 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

プッシュとポップの混合キューを処理しませんが、ほとんどのクラッシュを修正するのに適しています。

要旨：https : //gist.github.com/rivera-ernesto/0bc628be1e24ff5704ae

パーティーに遅れてすみません。最近、この問題が発生しました。同時に複数のビューコントローラーを押すと、ナビゲーションバーが破損した状態になります。これは、最初のビューコントローラーがまだアニメーションしているときに他のビューコントローラーがプッシュされるために発生します。nonameliveの答えからヒントを得て、自分のケースで機能する簡単なソリューションを思いつきました。UINavigationControllerpushViewControllerメソッドをサブクラス化してオーバーライドし、前のビューコントローラーアニメーションがまだ終了しているかどうかを確認するだけです。クラスをデリゲートに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'フラグがtrueの場合（デフォルト）、ユーザーはvc1、vc2、vc3のアニメーションプッシュを表示し、次にvc3のアニメーションポップを表示します。「acceptConflictingCommands」がfalseの場合、vc1が完全にプッシュされるまで、すべてのプッシュ/ポップリクエストが破棄されます。したがって、他の3つの呼び出しは破棄されます。

nonameliveのソリューションは素晴らしいです。ただし、プライベートAPIを使用しない場合は、メソッドをUINavigationControllerDelegate実行するか、アニメーションYESをに変更できますNO。これがコードのサンプルです。継承できます。お役に立てば幸いです:)

https://github.com/antrix1989/ANNavigationController

私はこの問題を何度も検索しましたが、おそらく2つ以上のVCを同時にプッシュするため、プッシュアニメーションの問題が発生します。これを参照してください：自分をサブビューとして追加できません

移行の進行中のVCが1つあることを確認してください。

誤って独自のビューにビューを追加しようとしたことがあります。

halfView.addSubview(halfView)

これをサブビューに変更します。

halfView.addSubview(favView)

私もこの問題に遭遇しました。Firebaseのログ分析を行ったところ、この問題はアプリがコールドスタートしたときにのみ発生することがわかりました。そこで、このクラッシュを再現できるデモを作成しました。

。

また、ウィンドウのルートビューコントローラーが表示されている場合、複数のプッシュを実行しても同じ問題が再び発生することはありません。（AppDelegate.swiftでtestColdStartUp（rootNav）をコメントし、ViewController.swiftでtestColdStartUp（）コメントをコメント解除できます）

PS：このクラッシュのシーンをアプリで分析しました。ユーザーがプッシュ通知をクリックしてアプリをコールドスタートすると、アプリはまだ起動ページにあり、別のプッシュをクリックしてジャンプします。現時点では、アプリがクラッシュを表示することがあります。私の現在の解決策は、プッシュまたはユニバーサルリンクのコールドスタートをキャッシュしてアプリのジャンプページを開き、rootviewcontrollerが表示されるのを待ってから、実行を遅らせることです。

最後のナビゲーションアニメーションを完了するために、遅延メソッドを使用してナビゲーションを試してください。

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

ビュー自体のビューをサブビューとして追加することはできません。

ビューは親子階層を維持するため、ビュー自体をサブビューとして追加すると、例外が発生します。

クラスがUIViewControllerの場合、そのビューを取得するにはself.viewを使用します。

クラスがUIViewクラスの場合、そのビューを取得するにはselfを使用します。

UiViewControllerクラスになる場合は、セルフをサブビューとして追加できません。UiViewクラスになる場合は、セルフをサブビューとして追加できます。

サブビューをビューに追加したい場合は、次のようにできます。

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