0

先日質問を投稿しましたが、十分なコードが含まれておらず、さらに問題が発生しています。を使用する必要があるかどうかはわかりません[[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;
}
4

2 に答える 2

1

ここでのコードコメントは、あなたが何をする必要があるかをすでに教えてくれているようです。 

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

カウントは1から始まり、配列インデックスは0から始まります。2つの配列インデックス値は0（最初のウィンドウの場合）と1（2番目のウィンドウの場合）ですが、インデックス値2にアクセスしようとしていました。

編集：次のコードが上記で完全に冗長になっていることに注意してください。   

if(topWindowIndex < 1) { // fix 0
topWindowIndex = [allWindows count] - 1;
topWindow = [allWindows objectAtIndex:topWindowIndex];
}
于 2011-11-13T10:27:26.873 に答える
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]];
}

あなたが本当に欲しいのは：

// 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
}
于 2011-11-13T11:05:45.237 に答える