Obj-C, 'NSRangeException', причина: '- [NSMutableArray objectAtIndex:]: индекс 2 за пределами [0 .. 1]' - PullRequest
Новая
       9

Obj-C, 'NSRangeException', причина: '- [NSMutableArray objectAtIndex:]: индекс 2 за пределами [0 .. 1]'

0 голосов
/ 13 ноября 2011

Я отправил вопрос на днях, но я не включил достаточно кода, и у меня возникла еще одна проблема. Я не уверен, должен ли я использовать [[UIApplication sharedApplication] windows], но после исправления, который я посоветовал сделать на днях, массив не имеет правильного количества значений.

Может кто-нибудь посоветовать / показать мне, где я иду не так, и если его правильно использовать [[UIApplication sharedApplication] windows]? 

- (void) addDoneToKeyboard {

    doneButton.hidden =  NO;

    //Add a button to the top, above all windows
    NSArray *allWindows = [[UIApplication sharedApplication] windows];
    NSUInteger topWindowIndex = [allWindows count];// fix - 1;
    UIWindow *topWindow = [allWindows objectAtIndex:topWindowIndex]; //SIGABRT

Завершение работы приложения из-за необработанного исключения 'NSRangeException', причина: '- [NSMutableArray objectAtIndex:]: индекс 2 за пределами [0 .. 1]' 

    // check if top window is of keypad or else
    NSString *topViewClassName = [NSString stringWithFormat:@"%@", 
             [topWindow class]];
    while (![topViewClassName isEqualToString:@"UITextEffectsWindow"] ) {
        --topWindowIndex;

        if(topWindowIndex < 1) // fix 0
            break;

        topWindow = [allWindows objectAtIndex:topWindowIndex];
        topViewClassName = [NSString stringWithFormat:@"%@", [topWindow class]];
    }

// 

    if(topWindowIndex < 1) { // fix 0
        topWindowIndex = [allWindows count] - 1;
        topWindow = [allWindows objectAtIndex:topWindowIndex];
    }

    if (doneButton.superview)
        [doneButton removeFromSuperview];

    [topWindow addSubview:doneButton];

    if (!doneButtonShownRecently) {
        [UIView beginAnimations:nil context:nil];
        [UIView setAnimationBeginsFromCurrentState:YES];
        [UIView setAnimationDuration:SLIDE_IN_ANIMATION_DURATION];
        [UIView setAnimationCurve:UIViewAnimationCurveEaseInOut];
        doneButton.frame = CGRectMake(0, 480-53, 
                         doneButton.frame.size.width, 53);
        [UIView commitAnimations];
    } else {
        doneButton.frame = CGRectMake(0, 427, 
                         doneButton.frame.size.width, 53);
    }

    doneButtonShown = YES;
}

Ответы [ 2 ]

1 голос
/ 13 ноября 2011

Этот код у вас здесь тоже неверный: 

// check if top window is of keypad or else
NSString *topViewClassName = [NSString stringWithFormat:@"%@", 
         [topWindow class]];
while (![topViewClassName isEqualToString:@"UITextEffectsWindow"] ) {
    --topWindowIndex;

    if(topWindowIndex < 1) // fix 0
        break;

    topWindow = [allWindows objectAtIndex:topWindowIndex];
    topViewClassName = [NSString stringWithFormat:@"%@", [topWindow class]];
}

То, что вы действительно хотите: 

// check if top window is of keypad or else
BOOL foundTextEffectsWindow = NO;
while (topWindowIndex >= 0) {
    topWindow = [allWindows objectAtIndex:topWindowIndex];
    if ([topWindow isKindOfClass:[UITextEffectsWindow class]]) {
      foundTextEffectsWindow = YES;
      break;
    }

    topWindowIndex--;
}
if (foundTextEffectsWindow) {
   // do stuff with the window
}
1 голос
/ 13 ноября 2011

Ваши комментарии к коду здесь уже говорят о том, что вам нужно сделать: 

NSUInteger topWindowIndex = [allWindows count] - 1;  
UIWindow *topWindow = [allWindows objectAtIndex:topWindowIndex];

Подсчет начинается с 1, а индексы массива начинаются с 0 - два значения индекса массива равны 0 (для вашего первогоокно) и 1 (для вашего второго окна), но вы пытались получить доступ к значению индекса 2.

Изменить: Обратите внимание, что следующий код теперь полностью избыточен в приведенном выше: 

if(topWindowIndex < 1) { // fix 0
topWindowIndex = [allWindows count] - 1;
topWindow = [allWindows objectAtIndex:topWindowIndex];
}
...