2

ビューにコレクションビューを使っています。
コレクションビューに後からセクションを追加できるようにしています。
そして、その追加されたセクションに付いているボタンをタップして、それぞれカメラロールから写真を選択し、表示できるようにしたいと考えています。

今のところセクションを追加しカメラロールから写真を選択して表示できるのですが、写真を選択するとすべてのセクションの選択された写真が表示されてしまいます。
おそらくreloadDataでコレクションビューのデータをすべて更新してしまってるからだと思うのですが...

//画像選択後の処理
- (void)imagePickerController:(UIImagePickerController *)picker didFinishPickingMediaWithInfo:(NSDictionary *)info
{
    NSLog(@"画像選択後の処理");
    UIImage *originalImage = (UIImage *)[info objectForKey:UIImagePickerControllerOriginalImage];
    UIImage *editImage = (UIImage *)[info objectForKey:UIImagePickerControllerEditedImage];

    UIImage *saveImage;

    if (editImage) {
        saveImage = editImage;
    } else {
        saveImage = originalImage;
    }

    //ここで選択された画像を表示する
    [self addSelectedPicture:self item:saveImage];

    [picker dismissViewControllerAnimated:YES completion:nil];

    //ここでreloadしないと画像選択後に戻ったときに表示されない
    [self.collectionView reloadData];
    //[self.collectionView reloadSections:[NSIndexSet indexSetWithIndex:0] ];
}


- (void)addSelectedPicture:(SubjectViewController *)controller item:(UIImage *)item
{
    [_images addObject:item];

    [self.collectionView reloadData];
    //[self.collectionView reloadSections:[NSIndexSet indexSetWithIndex:0]];
}




- (void)addItemViewCOntrollerDidFinish2:(AddDateViewController *)controller item:(NSString *)item
{
    if (!_objects4) {
        _objects4 = [[NSMutableArray alloc] init];
    }

    [_objects4 insertObject:item atIndex:0];

    [_sectionDates insertObject:controller.makeDate.date atIndex:0];

    //ここでreloadDataにするとセクションが追加されるたびに全体が更新されてしまう
    [_collectionView reloadData];
    //以下をやると最初はセクションがないからエラーになる
    //[self.collectionView reloadSections:[NSIndexSet indexSetWithIndex:0]];

    [self dismissViewControllerAnimated:YES completion:NULL]; 
}




//コレクションビューのセルに表示する画像
- (UICollectionViewCell *)collectionView:(UICollectionView *)collectionView cellForItemAtIndexPath:(NSIndexPath *)indexPath
{
    UICollectionViewCell *cell;
    cell = [collectionView dequeueReusableCellWithReuseIdentifier:@"Cell"forIndexPath:indexPath];

    UIImageView *imageView = (UIImageView *)[cell viewWithTag:1];

    imageView.image =_images[indexPath.row];

    return cell;
}

セクションを追加し、その追加されたセクションにそれぞれ個々の画像を表示するにはどのようにすればいいでしょうか。
どなたかご教授いただければと思います。
よろしくお願い致します。

1 件の回答 1

1

選択した画像を一次元配列で保持しているとお見受けしましたが、
セルではなくセクションを追加するのであれば二次元配列等で保持した方が良いのではないでしょうか?

一次元配列

// セクションが変わるとrowも0に戻るので同じ位置が取得される
_images[indexPath.row];

二次元配列

// セクションごとに画像を保持する
_images[indexPath.section][indexPath.row];  

----- 追記 -----

[_images addObject:item];

こちらの修正はされていますか?
配列の中に配列を格納するので、

[_images addObject:@[item]];
 or
_images[0] = @[item];

// 冗長に書くとこんな感じ
NSMutableArray* array = [NSMutableArray new];
[array addObject:item];
[_images addObject:array];

のような記述になります。
上記の例から添え字部分を変更する必要がありますが、
質問に記載されている実装だとaddSelectedPicture内でindexPathが取得できないので、
そのあたりは検討が必要になると思います。

5
  • セクションごとの配列を作ってその中に画像ごとの配列を作ればいいということでしょうか。
    – h_got
    Commented 2015年3月23日 10:01
  • @h_got 配列ではなくても良いと思いますが、セクションごとに別の集まりにしておいた方が扱いやすいのではないかと思います。
    – ita_3y
    Commented 2015年3月23日 12:42
  • セクションを別のまとまりですか。まだプログラミングに詳しくなく申し訳ありません。ちなみに教えていただいた二次元配列での_images[indexPath.section][indexPath.row];をやったところunrecognized selector sent to instanceのエラーが出ました。
    – h_got
    Commented 2015年3月24日 9:03
  • 追記の回答ありがとうございます。コレクションビューに表示するための配列は画像の配列(images)と同様(sectionDatesとして)作ってあります。そこで教えてくださったようにsectionDates内にimagesを格納してみたのですが、選択した画像のみをsectionDates配列のimage配列から取り出せずにいます。
    – h_got
    Commented 2015年3月25日 7:35
  • @h_got sectionDates配列のimage配列 という構成なのであれば、_images[indexPath.se‌​ction][indexPath.row];ではなく_sectionDates[indexPath.se‌​ction][indexPath.row];ではないでしょうか。
    – ita_3y
    Commented 2015年3月25日 9:19

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。