Случаен срив с персонализиран преход на контролера за изглед

Имам UIViewController, който представя/отхвърля друг контролер за изглед с персонализирана анимация, използвайки модела на conversioningDelegate, въведен в iOS 7.

Забелязах в моите регистрационни файлове на Crashlytics, че този раздел от кода от време на време се срива в производството и не успях да разбера защо. Никога не се срива при моето тестване и изглежда засяга само около 10% от нашите потребители. Засегнати са iOS 7 и iOS 8, както и iPad, iPhone и iPod.

Ето проследяването на стека от Crashlytics:

Crashed: com.apple.main-thread
EXC_BAD_ACCESS KERN_INVALID_ADDRESS at 0x00014000

Thread : Crashed: com.apple.main-thread
0  libobjc.A.dylib                0x303c3f76 objc_msgSend + 21
1  UIKit                          0x25ed0ac5 -[UIViewController _customAnimatorForDismissedController:] + 64
2  UIKit                          0x25ed0927 -[UIViewController _dismissViewControllerWithTransition:from:completion:] + 538
3  UIKit                          0x25e7e4b3 -[UIViewController dismissViewControllerWithTransition:completion:] + 822
4  UIKit                          0x25e7e4b3 -[UIViewController dismissViewControllerWithTransition:completion:] + 822
5  UIKit                          0x25e7e127 -[UIViewController dismissViewControllerAnimated:completion:] + 222
6  Skin Creator                   0x000f38e7 -[SEUSBodySideChooserViewController imageEditViewController:didFinishWithImage:] (SEUSBodySideChooserViewController.m:251)
7  Skin Creator                   0x000ae2e9 -[SEUSImageEditViewController sendFinishedDelegateMethod] (SEUSImageEditViewController.m:409)
8  Skin Creator                   0x000d585f -[SEUSCloseMenuController circleMenu:didSelectItemAtIndex:] (SEUSCloseMenuController.m:69)
9  Skin Creator                   0x000ceb75 -[SEUSCircleMenu menuButtonPressed:] (SEUSCircleMenu.m:203)
10 UIKit                          0x25dec427 -[UIApplication sendAction:to:from:forEvent:] + 70
11 UIKit                          0x25dec3c9 -[UIControl sendAction:to:forEvent:] + 44
12 UIKit                          0x25dd6fcd -[UIControl _sendActionsForEvents:withEvent:] + 584
13 UIKit                          0x25debdf9 -[UIControl touchesEnded:withEvent:] + 584
14 UIKit                          0x25db0b47 _UIGestureRecognizerUpdate + 10158
15 UIKit                          0x25de5afd -[UIWindow _sendGesturesForEvent:] + 784
16 UIKit                          0x25de53cd -[UIWindow sendEvent:] + 520
17 UIKit                          0x25dbbb5d -[UIApplication sendEvent:] + 196
18 UIKit                          0x2602f4e3 _UIApplicationHandleEventFromQueueEvent + 13874
19 UIKit                          0x25dba59f _UIApplicationHandleEventQueue + 1294
20 CoreFoundation                 0x228dd5e7 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 14
21 CoreFoundation                 0x228dc9fb __CFRunLoopDoSources0 + 222
22 CoreFoundation                 0x228db079 __CFRunLoopRun + 768
23 CoreFoundation                 0x22828981 CFRunLoopRunSpecific + 476
24 CoreFoundation                 0x22828793 CFRunLoopRunInMode + 106
25 GraphicsServices               0x29ba9051 GSEventRunModal + 136
26 UIKit                          0x25e1a981 UIApplicationMain + 1440
27 Skin Creator                   0x000ead6f main (main.m:16)

Само за да можете да разберете това малко по-добре [SEUSBodySideChooserViewController imageEditViewController:didFinishWithImage:] е обратно извикване на делегат, при което отхвърлям контролера за изглед.

Код, който използвам за представяне на View Controller

_toPixelEditTransitioningDelegate = [[SEUSFadeInTransitioningDelegate alloc] init];
_imageEditViewController.transitioningDelegate = _toPixelEditTransitioningDelegate;
[self presentViewController:_imageEditViewController animated:YES completion:nil];

Код, който използвам за отхвърляне на View Controller

[viewController dismissViewControllerAnimated:YES completion:nil];

Моят преходен делегат

- (id<UIViewControllerAnimatedTransitioning>)animationControllerForPresentedController:(UIViewController *)presented presentingController:(UIViewController *)presenting sourceController:(UIViewController *)source
{
    SEUSFadeInTransitioning *transitioning = [[SEUSFadeInTransitioning alloc] init];
    return transitioning;
}

- (id<UIViewControllerAnimatedTransitioning>)animationControllerForDismissedController:(UIViewController *)dismissed
{
    SEUSFadeInTransitioning *transitioning = [[SEUSFadeInTransitioning alloc] init];
    transitioning.reverse = YES;
    return transitioning;
}

Преходен обект, който имплементира протокола UIViewControllerAnimatedTransitioning

- (instancetype)init
{
    if (self = [super init])
    {
        _reverse = NO;
        _duration = 0.25f;
    }
    return self;
}

- (void)animateToPixelEdit:(id<UIViewControllerContextTransitioning>)transitionContext
{
    UIViewController* overviewVC = [transitionContext viewControllerForKey:UITransitionContextFromViewControllerKey];
    UIViewController* pixelVC = [transitionContext viewControllerForKey:UITransitionContextToViewControllerKey];
    [[transitionContext containerView] addSubview:overviewVC.view];
    [[transitionContext containerView] addSubview:pixelVC.view];

    pixelVC.view.alpha = 0;

    [UIView animateWithDuration:[self transitionDuration:transitionContext] animations:^
    {
        pixelVC.view.alpha = 1;
    }
    completion:^(BOOL finished)
    {
        [transitionContext completeTransition:![transitionContext transitionWasCancelled]];
    }];
}

- (void)animateToOverview:(id<UIViewControllerContextTransitioning>)transitionContext
{
    UIViewController* pixelVC = [transitionContext viewControllerForKey:UITransitionContextFromViewControllerKey];
    UIViewController* overviewVC = [transitionContext viewControllerForKey:UITransitionContextToViewControllerKey];
    [[transitionContext containerView] addSubview:pixelVC.view];
    [[transitionContext containerView] addSubview:overviewVC.view];

    overviewVC.view.alpha = 0;

    [UIView animateWithDuration:[self transitionDuration:transitionContext] animations:^
    {
        overviewVC.view.alpha = 1;
    }
    completion:^(BOOL finished)
    {
        [transitionContext completeTransition:![transitionContext transitionWasCancelled]];
    }];
}

- (NSTimeInterval)transitionDuration:(id<UIViewControllerContextTransitioning>)transitionContext
{
    return _duration;
}

- (void)animateTransition:(id<UIViewControllerContextTransitioning>)transitionContext
{
    if (self.reverse == NO)
    {
        [self animateToPixelEdit:transitionContext];
    }
    else
    {
        [self animateToOverview:transitionContext];
    }
}

Някой да забележи нещо, което може да се обърка? Наистина е трудно да се отстрани неизправност, тъй като не успях да възпроизведа срива.


person Joel Bell    schedule 13.11.2014    source източник
comment
Възможно ли е imageEditVC да извиква своя делегатен метод didFinish два пъти, ако потребител докосне няколко пъти?   -  person Acey    schedule 13.11.2014
comment
О, човече. Ако е толкова просто, не съм сигурен дали трябва да се чувствам глупав или развълнуван. Искам да го тествам сега!   -  person Joel Bell    schedule 13.11.2014
comment
Нямам успех с репликирането на това досега, ще кодирам проверка за него, просто ще го поставя в кутия и ще го избутам. Междувременно някой вижда ли нещо друго, което потенциално може да го причинява?   -  person Joel Bell    schedule 13.11.2014
comment
Добре още едно предположение... toPixelEditTransitioningDelegate силно ли е запазено някъде?   -  person Acey    schedule 13.11.2014
comment
Мисля, че разбрах, благодарение на вашите коментари. Изглежда, че бързото докосване може да представи imageEditVC два пъти, а след това, когато потребителят се опита да го затвори, след като е бил представен върху себе си, това ще причини този срив. Благодаря за вашата помощ!   -  person Joel Bell    schedule 13.11.2014
comment
Чувствайте се свободни да отговорите, ако искате, ще приема.   -  person Joel Bell    schedule 13.11.2014


Отговори (2)


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

person Acey    schedule 13.11.2014
comment
Вярно е. В моя случай превключването между разделите на сегментния контролер и анимирането на контролера за изглед при докосвания причиняваха срива. Лесно решение би било да деактивирате взаимодействието в контролата на потребителския интерфейс или просто UIApplication.shared.beginIgnoringInteractionEvents() и да активирате отново, когато анимациите са готови UIApplication.shared.endIgnoringInteractionEvents() - person Raj D; 11.01.2021

Разреших грешка, подобна на тази, в крайна сметка беше препратката transitioningDelegate (_toPixelEditTransitioningDelegate в примера с въпроса), която беше освободена.

Ако видите EXEC_BAD_ACCESS, можете да активирате NSZombies, което ще ви помогне да проследите обекта с лоша препратка.

Надяваме се това да се окаже полезно за някой друг!

person Oliver Atkinson    schedule 25.07.2016