6

私が抱えている問題についてあなたが与えることができる助け/洞察を前もって感謝します. つい最近、iOS アプリが恐ろしいEXC_BAD_ACCESSエラーで失敗し始めました。私は SO と Google について多くの調査を行い、いくつかの素晴らしいアドバイスを受け取りましたが、どの投稿も私の問題を解決するのに役立ちませんでした. ここまでで、EXC_BAD_ACCESS が次の 2 つのいずれかになる可能性があることを理解しました。

1) A pointer is now pointing to memory that was deallocated.
2) A pointer itself is corrupt.

ここここここ、およびここの指示を読んで従いました。「ビルドして実行」すると、100% の確率で exc_bad_access を再現できます。ただし、デバッガーまたは Instruments で NSZombie を有効にすると、問題を再現できません。さらに、NSZombie はコンソールに役立つ情報を一切残しません。

問題を特定するのに最も近かったのは、コードの 1 行ごとに文字通り NSLog を配置して、どこでクラッシュしたかを確認することでした。私はそれが2つの異なるポイントでクラッシュしていると信じています - どちらもcellForRowAtIndexPath. 以下に、クラッシュしていると思われる 2 つの方法と、コンソールのエラー出力を示します。どんな助けや指導も大歓迎です、ありがとう!

問題のある方法:

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath
{
if (indexPath.section == 0) {
    static NSString *CellIdentifier = @"EventDetailCell";

    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];

    if (cell == nil)
    {
        cell = [[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:CellIdentifier];
        cell.textLabel.textColor = [UIColor colorWithRed:150 green:143 blue:135 alpha:1.0];
        cell.textLabel.highlightedTextColor = [UIColor whiteColor];
        cell.textLabel.font = [UIFont fontWithName:@"Helvetica" size:15];

        if (indexPath.row < 4) {
            UIImageView *disclosureArrow = [[UIImageView alloc] initWithFrame:CGRectMake(280, 15, 10, 14)];
            disclosureArrow.image = [UIImage imageNamed:@"tableRowAccessoryArrow"];
            [cell.contentView addSubview:disclosureArrow];
        }
    } else {
        if ((indexPath.section == 0) && (indexPath.row == 4)) {
            for (UIView *v in [cell.contentView subviews]) {
                [v removeFromSuperview];
            }
        }
    }

    switch (indexPath.row) {
        case 0:
            cell.textLabel.text = [NSString stringWithFormat:@"%@", self.startTime];
            cell.backgroundView = [[UIImageView alloc] initWithImage:[UIImage imageNamed:@"eventDetailRowTop"]];
            cell.imageView.image = [UIImage imageNamed:@"clockIcon"];
            break;
        case 1:
            cell.textLabel.text = @"Location";
            cell.backgroundView = [[UIImageView alloc] initWithImage:[UIImage imageNamed:@"eventDetailRowMiddle"]];
            cell.selectedBackgroundView = [[UIImageView alloc] initWithImage:[UIImage imageNamed:@"eventDetailRowMiddleSelected"]];
            cell.imageView.image = [UIImage imageNamed:@"mapIcon"];
            break;
        case 2:
            cell.textLabel.text = [NSString stringWithFormat:@"%u attendees", [self.attendees count]];
            cell.backgroundView = [[UIImageView alloc] initWithImage:[UIImage imageNamed:@"eventDetailRowMiddle"]];
            cell.selectedBackgroundView = [[UIImageView alloc] initWithImage:[UIImage imageNamed:@"eventDetailRowMiddleSelected"]];
            cell.imageView.image = [UIImage imageNamed:@"attendeesIcon"];
            break;
        case 3:
            cell.textLabel.text = [NSString stringWithFormat:@"%u fomos", [self.fomos count]];
            cell.backgroundView = [[UIImageView alloc] initWithImage:[UIImage imageNamed:@"eventDetailRowBottom"]];
            cell.selectedBackgroundView = [[UIImageView alloc] initWithImage:[UIImage imageNamed:@"eventDetailRowMiddleSelected"]];
            cell.imageView.image = [UIImage imageNamed:@"fomoIcon"];
            break;
        case 4:
        {
            cell.selectionStyle = UITableViewCellSelectionStyleNone;

            UIImage *clockIcon = [UIImage imageNamed:@"clockIcon"];
            UIImageView *iconSlot = [[UIImageView alloc] initWithFrame:CGRectMake(3, 18, 18, 18)];
            iconSlot.image = clockIcon;
            [cell.contentView addSubview:iconSlot];

            NSString *currentTime = @"Happening Now";
            UILabel *currentTimeLabel = [[UILabel alloc] initWithFrame:CGRectMake(25, 8, 150, 36)];
            currentTimeLabel.backgroundColor = [UIColor clearColor];
            currentTimeLabel.textColor = [UIColor whiteColor];
            currentTimeLabel.font = [UIFont fontWithName:@"Helvetica" size:14];
            currentTimeLabel.text = currentTime;
            [cell.contentView addSubview:currentTimeLabel];

            UIImage *listView = [UIImage imageNamed:@"listViewSelected"];
            self.listViewButton = [UIButton buttonWithType:UIButtonTypeCustom];
            self.listViewButton.frame = CGRectMake(180, 8, 36, 36);
            self.listViewButton.tag = 1;
            [self.listViewButton addTarget:self action:@selector(togglePhotosView:) forControlEvents:UIControlEventTouchUpInside];
            [self.listViewButton setBackgroundImage:listView forState:UIControlStateNormal];
            [cell.contentView addSubview:self.listViewButton];

            UIImage *gridView = [UIImage imageNamed:@"gridViewIcon"];
            self.gridViewButton = [UIButton buttonWithType:UIButtonTypeCustom];
            self.gridViewButton.frame = CGRectMake(240, 7.2f, 36, 36);
            self.gridViewButton.tag = 2;
            [self.gridViewButton addTarget:self action:@selector(togglePhotosView:) forControlEvents:UIControlEventTouchUpInside];
            [self.gridViewButton setBackgroundImage:gridView forState:UIControlStateNormal];
            [cell.contentView addSubview:self.gridViewButton];
            break;
        }
        default:
            break;
    }
    return cell;
} else {
    if ([self.listViewObjects count] == 0) {
        // Do something when there are no pictures or stories to show
        static NSString *CellIdentifier = @"NoPhotosYetCell";

        UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];

        if (cell == nil)
        {
            cell = [[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:CellIdentifier];
            cell.textLabel.textColor = [UIColor colorWithRed:0.150 green:0.143 blue:0.135 alpha:1.0];
            cell.textLabel.highlightedTextColor = [UIColor whiteColor];
            cell.selectionStyle = UITableViewCellSelectionStyleNone;
            cell.textLabel.text = @"No photos have been added yet.";
        }
        return cell;
    }
    else {
        if (self.isGridView == YES) {
            static NSString *CellIdentifier = @"EventContentGridViewCell";

            EventDetailGridRow *cell = (EventDetailGridRow *)[tableView dequeueReusableCellWithIdentifier:CellIdentifier];

            if (cell == nil)
            {
                cell = [[EventDetailGridRow alloc] initWithFrame:CGRectMake(0, 0, 306, 102)];
                cell.backgroundColor = [UIColor redColor];
            }

            NSMutableArray *mutablePhotos = [NSMutableArray arrayWithArray:self.photos];

            int offset = (indexPath.row * 3);
            int maxRange = 3;

            if (([mutablePhotos count] < (offset + maxRange)))
            {
                maxRange = ([mutablePhotos count] - offset);
            }
            NSArray *firstThree = [mutablePhotos subarrayWithRange:NSMakeRange(offset, maxRange)];

            cell.photos = firstThree;

            return cell;

        } else {
            static NSString *CellIdentifier = @"EventContentListViewCell";

            EventDetailListRowCell *cell = (EventDetailListRowCell *)[tableView dequeueReusableCellWithIdentifier:CellIdentifier];

            if (cell == nil)
            {
                cell = [[EventDetailListRowCell alloc] initWithFrame:CGRectMake(0, 0, 306, 400)];
            }

            NSString *authorName;
            NSString *authorUID;
            BOOL isPhoto = YES;

            if ([[self.listViewObjects objectAtIndex:indexPath.row] valueForKey:@"caption"] != nil)
            {
                // It's a photo object
                NSDictionary *photo = [self.listViewObjects objectAtIndex:indexPath.row];
                NSString *photoID = [photo valueForKey:@"id"];
                NSString *photoName = [photo valueForKey:@"image_file_name"];
                NSURL *photoURL = [NSURL URLWithString:[NSString stringWithFormat:@"http://s3.amazonaws.com/motlee-development-photos/images/%@/compressed/%@", photoID, photoName]];
                authorName = [photo valueForKeyPath:@"owner.name"][0];
                authorUID = [photo valueForKeyPath:@"owner.uid"][0];

                UIImageView *imageContainer = [[UIImageView alloc] initWithFrame:CGRectMake(0, 20, 300, 300)];
                [imageContainer setImageWithURL:photoURL];
                [cell.contentView addSubview:imageContainer];
            }
            else if ([[self.listViewObjects objectAtIndex:indexPath.row] valueForKey:@"body"] != nil)
            {
                isPhoto = NO;
                NSDictionary *story = [self.listViewObjects objectAtIndex:indexPath.row];
                authorName = [story valueForKeyPath:@"owner.name"][0];
                authorUID = [story valueForKeyPath:@"owner.uid"][0];

                UIImageView *imageContainer = [[UIImageView alloc] initWithFrame:CGRectMake(0, 20, cell.frame.size.width, 148)];
                imageContainer.image = [UIImage imageNamed:@"storyContainer"];
                [cell.contentView addSubview:imageContainer];

                UILabel *storyBody = [[UILabel alloc] initWithFrame:CGRectMake(30, 20, (306 - 60), 148)];
                storyBody.numberOfLines = 3;
                storyBody.backgroundColor = [UIColor clearColor];
                storyBody.textAlignment = NSTextAlignmentCenter;
                storyBody.font = [UIFont fontWithName:@"Helvetica" size:17];
                storyBody.textColor = [UIColor whiteColor];
                storyBody.text = [NSString stringWithFormat:@"\"%@\"", [story valueForKey:@"body"]];
                [cell.contentView addSubview:storyBody];

            }
            else {
                // Something screwy going on here, should have been one of the above objects
            }

            NSURL *profilePicURL = [NSURL URLWithString:[NSString stringWithFormat:@"http://graph.facebook.com/%@/picture", authorUID]];
            UIImageView *profilePic;
            if (isPhoto) profilePic = [[UIImageView alloc] initWithFrame:CGRectMake(20, 347, 40, 40)];
            else profilePic = [[UIImageView alloc] initWithFrame:CGRectMake(20, 179, 40, 40)];
            [profilePic setImageWithURL:profilePicURL placeholderImage:[UIImage imageNamed:@"placeholder"]];
            [cell.contentView addSubview:profilePic];

            UILabel *takenByLabel;
            if (isPhoto) {
                takenByLabel = [[UILabel alloc] initWithFrame:CGRectMake(70, 342, 170, 30)];
            }
            else {
                takenByLabel = [[UILabel alloc] initWithFrame:CGRectMake(70, 174, 170, 30)];
            }

            takenByLabel.textColor = [UIColor lightGrayColor];
            takenByLabel.font = [UIFont fontWithName:@"Helvetica" size:12];
            takenByLabel.backgroundColor = [UIColor clearColor];
            if (isPhoto) takenByLabel.text = @"taken by";
            else takenByLabel.text = @"written by";
            [cell.contentView addSubview:takenByLabel];

            UILabel *photoTakerLabel;
            if (isPhoto) {
                photoTakerLabel = [[UILabel alloc] initWithFrame:CGRectMake(70, 360, 170, 30)];
            }
            else {
                photoTakerLabel = [[UILabel alloc] initWithFrame:CGRectMake(70, 192, 170, 30)];
            }
            photoTakerLabel.textColor = [UIColor whiteColor];
            photoTakerLabel.font = [UIFont fontWithName:@"Helvetica" size:14];
            photoTakerLabel.backgroundColor = [UIColor clearColor];
            photoTakerLabel.text = authorName;
            [cell.contentView addSubview:photoTakerLabel];
            return cell;
        }
    }
}
}

ログ出力:

libobjc.A.dylib`objc_msgSend:
0x1f5908c:  movl   8(%esp), %ecx
0x1f59090:  movl   4(%esp), %eax
0x1f59094:  testl  %eax, %eax
0x1f59096:  je     0x1f590e8                 ; objc_msgSend + 92
0x1f59098:  movl   (%eax), %edx
0x1f5909a:  pushl  %edi
0x1f5909b:  movl   8(%edx), %edi   <-- EXC_BAD_ACCESS (code=2, address=0xb0000008)
4

1 に答える 1

2

いくつかの場所で初期化中に cellIdentifier が欠落しているようです。

static NSString *CellIdentifier = @"EventContentGridViewCell";

EventDetailGridRow *cell = (EventDetailGridRow *)[tableView dequeueReusableCellWithIdentifier:CellIdentifier];
if (cell == nil)
{
   cell = [[EventDetailGridRow alloc] initWithFrame:CGRectMake(0, 0, 306, 102)];//where is CellIdentifier?? 

それは次のようなものだったはずです。

cell = [[EventDetailGridRow alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:CellIdentifier];
cell.frame = CGRectMake(0, 0, 306, 102);

"EventContentGridViewCell"のオブジェクトの作成中にcellIdentifier が設定されているのを確認できませんEventDetailGridRow。上記のコードの他のすべてのカスタム セル クラスにも同じことが当てはまります。ただし、これだけが上記のコードの問題であるかどうかはわかりません。

上記の変更で問題が解決しない場合は、このHow to add a breakpoint to objc_exception_throw? を確認してください。Xcode/iPhone の未処理の例外に関するスタック トレースまたは詳細情報。問題が発生している場所を把握するのに役立ちます。

于 2012-11-08T23:21:24.083 に答える