2013-08-30 6 views
1

У меня есть пользовательский UISegmentedControl, и когда я его инициализирую, он иногда падает. Это редкость, примерно в 1% случаев (достаточно редко, чтобы пройти «строгое» тестирование приложений Apple), и хотя у меня есть этот точный код в трех других представлениях, он падает только на одном из них.Почему мое приложение иногда падает, когда я меняю рамку?

Код:

NSArray *providers = [[NSArray alloc] initWithObjects:@"All", @"Soon", @"Attn", @"Late",  @"Done", nil]; //categories for segmented control 
FancySegmentedControl *fancy = [[FancySegmentedControl alloc] initWithItems:providers]; 
fancy.backgroundColor = [UIColor clearColor]; //change bg color 
fancy.frame = CGRectMake(11, 86, 263, 29); //lldb crashes here 
[fancy setBackgroundColor:[UIColor colorWithRed:42/255.0f 
              green:82/255.0f 
              blue:164/255.0f alpha:1] forState:UIControlStateNormal]; 

Итак, мои симптомы:

-crash не бывает в большинстве случаев.

-Краш происходит только на одном из четырех контроллеров представления, даже если код идентичен.

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

-Мой проект использует ARC.

Код для FancySegmentedControl выглядит следующим образом:

@interface FancySegmentedControl : UISegmentedControl 
{ 
    UIColor *bgNotSelected; 
    UIColor *bgSelected; 
    UIColor *barNotSelected; 
    UIColor *barSelected; 
} 

-(void)setBackgroundColor:(UIColor *)color forState:(UIControlState)state; 

-(void)setBarColor:(UIColor *)color forState:(UIControlState)state; 

-(void)setTextAttributes:(NSDictionary *)attrs forState:(UIControlState) state; 

@end 

@implementation FancySegmentedControl 

- (id)initWithItems:(NSArray *)items 
{ 
    self = [super initWithItems:items]; 
    bgSelected = [UIColor blueColor]; 
    bgNotSelected = [UIColor whiteColor]; 
    barNotSelected = [UIColor lightGrayColor]; 
    barSelected = [UIColor orangeColor]; 
    self.selectedSegmentIndex = 0; 
    if (self) { 
     //change text stuff 
     NSDictionary *attributes = [NSDictionary dictionaryWithObjectsAndKeys: 
            [UIFont boldSystemFontOfSize:17], UITextAttributeFont, 
            [UIColor blackColor], UITextAttributeTextColor, 
            nil]; 
     [self setTitleTextAttributes:attributes forState:UIControlStateNormal]; 
     NSDictionary *highlightedAttributes = [NSDictionary dictionaryWithObject:[UIColor whiteColor] forKey:UITextAttributeTextColor]; 
     [self setTitleTextAttributes:highlightedAttributes forState:UIControlStateHighlighted]; 

     //set all dividers to nothing 
     UIView *yourView = [[UIView alloc] initWithFrame:CGRectMake(0, 0, 1, 30)]; 
     UIGraphicsBeginImageContext(yourView.bounds.size); 
     [yourView.layer renderInContext:UIGraphicsGetCurrentContext()]; 
     UIImage *image = UIGraphicsGetImageFromCurrentImageContext(); 
     UIGraphicsEndImageContext(); 
     [self setDividerImage:image 
      forLeftSegmentState:UIControlStateNormal 
      rightSegmentState:UIControlStateNormal 
        barMetrics:UIBarMetricsDefault]; 
     [self setDividerImage:image 
      forLeftSegmentState:UIControlStateSelected 
      rightSegmentState:UIControlStateNormal 
        barMetrics:UIBarMetricsDefault]; 
     [self setDividerImage:image 
      forLeftSegmentState:UIControlStateNormal 
      rightSegmentState:UIControlStateSelected 
        barMetrics:UIBarMetricsDefault]; 


     yourView = [[UIView alloc] initWithFrame:CGRectMake(0, 0, 40, 30)]; 
     yourView.backgroundColor = bgNotSelected; 
     UIView *barView = [[UIView alloc] initWithFrame:CGRectMake(0, yourView.frame.size.height - 3, 40, 3)]; 
     barView.backgroundColor = barNotSelected; 
     [yourView addSubview:barView]; 
     UIGraphicsBeginImageContext(yourView.bounds.size); 
     [yourView.layer renderInContext:UIGraphicsGetCurrentContext()]; 
     image = UIGraphicsGetImageFromCurrentImageContext(); 
     UIGraphicsEndImageContext(); 

     UIImage *normalBackgroundImage = image; 
     [self setBackgroundImage:normalBackgroundImage 
         forState:UIControlStateNormal 
         barMetrics:UIBarMetricsDefault]; 

     yourView.backgroundColor = bgSelected; 
     barView = [[UIView alloc] initWithFrame:CGRectMake(0, yourView.frame.size.height - 3, 40, 3)]; 
     barView.backgroundColor = barSelected; 
     [yourView addSubview:barView]; 
     UIGraphicsBeginImageContext(yourView.bounds.size); 
     [yourView.layer renderInContext:UIGraphicsGetCurrentContext()]; 
     image = UIGraphicsGetImageFromCurrentImageContext(); 
     UIGraphicsEndImageContext(); 
     UIImage *selectedBackgroundImage = image; 
     [self setBackgroundImage:selectedBackgroundImage 
         forState:UIControlStateSelected 
         barMetrics:UIBarMetricsDefault]; 
    } 
    return self; 
} 

-(void)setBackgroundColor:(UIColor *)color forState:(UIControlState)state 
{ 
    UIColor *barColor; 
    if (state == UIControlStateSelected) 
    { 
     bgSelected = color; 
     barColor = barSelected; 
    } 
    else 
    { 
     bgNotSelected = color; 
     barColor = barNotSelected; 
    } 
    UIView *yourView = [[UIView alloc] initWithFrame:CGRectMake(0, 0, 40, 30)]; 
    yourView.backgroundColor = color; 
    UIView *barView = [[UIView alloc] initWithFrame:CGRectMake(0, yourView.frame.size.height - 3, 40, 3)]; 
    barView.backgroundColor = barColor; 
    [yourView addSubview:barView]; 
    UIGraphicsBeginImageContext(yourView.bounds.size); 
    [yourView.layer renderInContext:UIGraphicsGetCurrentContext()]; 
    UIImage *image = UIGraphicsGetImageFromCurrentImageContext(); 
    UIGraphicsEndImageContext(); 

    [self setBackgroundImage:image 
        forState:state 
        barMetrics:UIBarMetricsDefault]; 
} 

-(void)setBarColor:(UIColor *)color forState:(UIControlState)state 
{ 
    UIColor *bgColor; 
    if (state == UIControlStateSelected) 
    { 
     barSelected = color; 
     bgColor = bgSelected; 
    } 
    else 
    { 
     barNotSelected = color; 
     bgColor = bgNotSelected; 
    } 
    UIView *yourView = [[UIView alloc] initWithFrame:CGRectMake(0, 0, 40, 30)]; 
    yourView.backgroundColor = bgColor; 
    UIView *barView = [[UIView alloc] initWithFrame:CGRectMake(0, yourView.frame.size.height - 3, 40, 3)]; 
    barView.backgroundColor = color; 
    [yourView addSubview:barView]; 
    UIGraphicsBeginImageContext(yourView.bounds.size); 
    [yourView.layer renderInContext:UIGraphicsGetCurrentContext()]; 
    UIImage *image = UIGraphicsGetImageFromCurrentImageContext(); 
    UIGraphicsEndImageContext(); 

    [self setBackgroundImage:image 
        forState:state 
        barMetrics:UIBarMetricsDefault]; 
} 

-(void)setTextAttributes:(NSDictionary *)attrs forState:(UIControlState) state 
{ 
    if (state == UIControlStateSelected) 
    { 
     //in case user mistakes the states 
     state = UIControlStateHighlighted; 
    } 
    [self setTitleTextAttributes:attrs forState:state]; 
} 


/* 
// Only override drawRect: if you perform custom drawing. 
// An empty implementation adversely affects performance during animation. 
- (void)drawRect:(CGRect)rect 
{ 
    // Drawing code 
} 
*/ 

@end 

Полное сообщение об ошибке:

* thread #1: tid = 0x1f03, 0x0134609b libobjc.A.dylib`objc_msgSend + 15, stop reason =  EXC_BAD_ACCESS (code=2, address=0xb0000008) 
frame #0: 0x0134609b libobjc.A.dylib`objc_msgSend + 15 
frame #1: 0x0059bcd5 UIKit`-[UISegmentedControl _setBackgroundImage:forState:barMetrics:] + 148 
frame #2: 0x0059bd69 UIKit`-[UISegmentedControl setBackgroundImage:forState:barMetrics:] + 73 
frame #3: 0x0004f9f5 Services`-[FancySegmentedControl setBarColor:forState:](self=0x0ea5d740, _cmd=0x000518fd, color=0x08db4630, state=0x00000004) + 1141 at FancySegmentedControl.m:135 
frame #4: 0x0000538c Services`-[ServicesViewController fixSearchBar](self=0x07f791a0, _cmd=0x0005170b) + 1132 at ServicesViewController.m:174 
frame #5: 0x00003f38 Services`-[ServicesViewController viewDidLoad](self=0x07f791a0, _cmd=0x017fe1dd) + 616 at ServicesViewController.m:49 
frame #6: 0x0056964e UIKit`-[UIViewController view] + 184 
frame #7: 0x00569941 UIKit`-[UIViewController contentScrollView] + 36 
frame #8: 0x0057b47d UIKit`-[UINavigationController _computeAndApplyScrollContentInsetDeltaForViewController:] + 36 
frame #9: 0x0057b66f UIKit`-[UINavigationController _layoutViewController:] + 43 
frame #10: 0x0057b93b UIKit`-[UINavigationController _startTransition:fromViewController:toViewController:] + 303 
frame #11: 0x0057c3df UIKit`-[UINavigationController _startDeferredTransitionIfNeeded] + 288 
frame #12: 0x0057c561 UIKit`-[UINavigationController __viewWillLayoutSubviews] + 33 
frame #13: 0x006984ca UIKit`-[UILayoutContainerView layoutSubviews] + 222 
frame #14: 0x004e2301 UIKit`-[UIView(CALayerDelegate) layoutSublayersOfLayer:] + 145 
frame #15: 0x019b8e72 CoreFoundation`-[NSObject performSelector:withObject:] + 66 
frame #16: 0x003c292d QuartzCore`-[CALayer layoutSublayers] + 266 
frame #17: 0x003cc827 QuartzCore`CA::Layer::layout_if_needed(CA::Transaction*) + 231 
frame #18: 0x00352fa7 QuartzCore`CA::Context::commit_transaction(CA::Transaction*) + 377 
frame #19: 0x00354ea6 QuartzCore`CA::Transaction::commit() + 374 
frame #20: 0x00354580 QuartzCore`CA::Transaction::observer_callback(__CFRunLoopObserver*, unsigned long, void*) + 80 
frame #21: 0x0198b9ce CoreFoundation`__CFRUNLOOP_IS_CALLING_OUT_TO_AN_OBSERVER_CALLBACK_FUNCTION__ + 30 
frame #22: 0x01922670 CoreFoundation`__CFRunLoopDoObservers + 384 
frame #23: 0x018ee4f6 CoreFoundation`__CFRunLoopRun + 1174 
frame #24: 0x018eddb4 CoreFoundation`CFRunLoopRunSpecific + 212 
frame #25: 0x018edccb CoreFoundation`CFRunLoopRunInMode + 123 
frame #26: 0x01b49879 GraphicsServices`GSEventRunModal + 207 
frame #27: 0x01b4993e GraphicsServices`GSEventRun + 114 
frame #28: 0x004a3a9b UIKit`UIApplicationMain + 1175 
frame #29: 0x000036d0 Services`main(argc=1, argv=0xbffff214) + 80 at main.m:16 
frame #30: 0x000025e5 Services`start + 53 
+0

Похоже, у вас есть свой собственный класс для сегментированного управления. Можете ли вы включить его код? – LuisCien

+0

Я не думаю, что код имеет к этому какое-либо отношение, поскольку он не умирает ни на одном другом vc. – Lugubrious

+0

Если вы знаете, код размещен на gitHub: https://github.com/mkeehan/MySegmentedControl – Lugubrious

ответ

2

UIControlState - это перечисление. Вы передаете указатель на значение, которое должно быть UIControlState, но фактическое значение указателя используется напрямую, даже не разыменовывается.

Удалите указатель из параметра UIControlState в подписи, и все будет в порядке.

-(void)setBarColor:(UIColor *)color forState:(UIControlState)state 

Это будет встретить в UISegmentedControl в сигнатуру метода:

-(void)setBackgroundImage:(UIImage *)backgroundImage forState:(UIControlState)state barMetrics:(UIBarMetrics)barMetrics 
+0

Я отредактировал код так, как я считал правильным, ошибка все еще происходит. Правильно ли я отредактировал? – Lugubrious

+0

Это та же ошибка? Или это происходит где-то еще? Это изменение должно решить, по крайней мере, оригинальную проблему. – allprog

+0

Я очистил сборку (которую я нашел очень хорошим фиксатором после исправления ошибок памяти), позвольте мне посмотреть, все ли это происходит. Я просто приму ваш ответ, если он не появится в следующий час или около того. Спасибо! – Lugubrious

0

вы пробовали это работает с зомби включен?

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

Похоже, в вашем коде:

frame #1: 0x0059bcd5 UIKit`-[UISegmentedControl _setBackgroundImage:forState:barMetrics:] + 148 

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

Поставьте точку останова здесь и убедитесь, что PARAMS реальны, и т.д. Или

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

+0

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