it-roy-ru.com

UICollectionView reloadData не работает должным образом в iOS 7

Я обновлял свои приложения для работы на iOS 7, который в большинстве случаев проходит гладко. В более чем одном приложении я заметил, что метод reloadDataUICollectionViewController действует не совсем так, как раньше. 

Я загружу UICollectionViewController, заполню UICollectionView некоторыми данными, как обычно. Это прекрасно работает в первый раз. Однако если я запрашиваю новые данные (заполняю UICollectionViewDataSource), а затем вызываю reloadData, он будет запрашивать источник данных для numberOfItemsInSection и numberOfSectionsInCollectionView, но, похоже, он не вызывает cellForItemAtIndexPath надлежащее количество раз. 

Если я изменю код, чтобы перезагрузить только один раздел, он будет работать правильно. Это не проблема для меня, чтобы изменить их, но я не думаю, что я должен это делать. reloadData должен перезагрузить все видимые ячейки в соответствии с документацией. 

Кто-нибудь еще видел это?

93
VaporwareWolf

Заставить это в главном потоке:

dispatch_async(dispatch_get_main_queue(), ^ {
    [self.collectionView reloadData];
});
71
Shaunti Fondrisi

В моем случае количество ячеек/разделов в источнике данных никогда не менялось, и я просто хотел перезагрузить видимый контент на экране ..

Мне удалось обойти это, позвонив: 

[self.collectionView reloadItemsAtIndexPaths:[self.collectionView indexPathsForVisibleItems]];

затем: 

[self.collectionView reloadData];
63
liamnichols

У меня была точно такая же проблема, однако мне удалось выяснить, что происходит неправильно . В моем случае я вызывал reloadData из collectionView: cellForItemAtIndexPath: , который выглядит не правильно. 

Диспетчерский вызов reloadData в основную очередь исправил проблему раз и навсегда.

  dispatch_async(dispatch_get_main_queue(), ^{
    [self.collectionView reloadData];
  });
26
Anton Matosov

Перезагрузка некоторых предметов не работает для меня. В моем случае, и только потому, что у collectionView, который я использую, есть только один раздел, я просто перезагружаю этот конкретный раздел. На этот раз содержимое корректно перезагружено . Странно, что это происходит только на iOS 7 (7.0.3)

[self.collectionView reloadSections:[NSIndexSet indexSetWithIndex:0]];
19
miguelsanchez

Свифт 4 - 3

// GCD    
DispatchQueue.main.async(execute: collectionView.reloadData)

// Operation
OperationQueue.main.addOperation(collectionView.reloadData)

Swift 2

// Operation
NSOperationQueue.mainQueue().addOperationWithBlock(collectionView.reloadData)
12
dimpiax

У меня была та же проблема с reloadData на iOS 7. После долгой сессии отладки я нашел проблему. 

На iOS7 reloadData в UICollectionView не отменяет предыдущие обновления, которые еще не были завершены (Обновления, которые вызывались внутри executeBatchUpdates: block). 

Лучшее решение для устранения этой ошибки - остановка всех обновлений, которые в данный момент обрабатываются, и вызов reloadData. Я не нашел способ отменить или остановить блок executeBatchUpdates. Поэтому, чтобы исправить ошибку, я сохранил флаг, который указывает, есть ли блок executeBatchUpdates, который в данный момент обрабатывается. Если нет блока обновления, который в данный момент обрабатывается, я могу немедленно вызвать reloadData, и все работает как положено. Если есть блок обновления, который в данный момент обрабатывается, я вызову reloadData для полного блока executeBatchUpdates.

11
user2459624

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

Также просто добавив что-то так просто, как 

UIView *aView = [UIView new];
[collectionView addSubView:aView];

приведет к вызову методов

Также я поиграл с размером кадра - и вуаля, методы вызывались. 

Есть много ошибок с iOS7 UICollectionView.

4
Avner Barr

Решение, данное Shaunti Fondrisi, почти идеально. Но такой кусок кода или кодов, как, ставят в очередь выполнение UICollectionView's reloadData() в NSOperationQueue's mainQueue, действительно помещают время выполнения в начало следующего цикла событий в цикле выполнения, что может привести к обновлению UICollectionView одним щелчком.

Чтобы решить эту проблему. Мы должны поместить время выполнения того же куска кода в конец текущего цикла событий, но не в начало следующего. И мы можем достичь этого, используя CFRunLoopObserver.

CFRunLoopObserver наблюдает за всеми операциями ожидания источника ввода и операциями входа и выхода цикла выполнения. 

public struct CFRunLoopActivity : OptionSetType {
    public init(rawValue: CFOptionFlags)

    public static var Entry: CFRunLoopActivity { get }
    public static var BeforeTimers: CFRunLoopActivity { get }
    public static var BeforeSources: CFRunLoopActivity { get }
    public static var BeforeWaiting: CFRunLoopActivity { get }
    public static var AfterWaiting: CFRunLoopActivity { get }
    public static var Exit: CFRunLoopActivity { get }
    public static var AllActivities: CFRunLoopActivity { get }
}

Среди этих действий .AfterWaiting можно наблюдать, когда заканчивается текущий цикл события, и .BeforeWaiting можно наблюдать, когда только что начался следующий цикл события. 

Поскольку на NSRunLoop существует только один экземпляр NSThread, а NSRunLoop точно управляет NSThread, мы можем считать, что доступ осуществляется с одного экземпляра NSRunLoop, который никогда не пересекается с потоками.

Основываясь на упомянутых выше моментах, теперь мы можем написать код: диспетчер задач на основе NSRunLoop:

import Foundation
import ObjectiveC

public struct Weak<T: AnyObject>: Hashable {
    private weak var _value: T?
    public weak var value: T? { return _value }
    public init(_ aValue: T) { _value = aValue }

    public var hashValue: Int {
        guard let value = self.value else { return 0 }
        return ObjectIdentifier(value).hashValue
    }
}

public func ==<T: AnyObject where T: Equatable>(lhs: Weak<T>, rhs: Weak<T>)
    -> Bool
{
    return lhs.value == rhs.value
}

public func ==<T: AnyObject>(lhs: Weak<T>, rhs: Weak<T>) -> Bool {
    return lhs.value === rhs.value
}

public func ===<T: AnyObject>(lhs: Weak<T>, rhs: Weak<T>) -> Bool {
    return lhs.value === rhs.value
}

private var dispatchObserverKey =
"com.WeZZard.Nest.NSRunLoop.TaskDispatcher.DispatchObserver"

private var taskQueueKey =
"com.WeZZard.Nest.NSRunLoop.TaskDispatcher.TaskQueue"

private var taskAmendQueueKey =
"com.WeZZard.Nest.NSRunLoop.TaskDispatcher.TaskAmendQueue"

private typealias DeallocFunctionPointer =
    @convention(c) (Unmanaged<NSRunLoop>, Selector) -> Void

private var original_dealloc_imp: IMP?

private let swizzled_dealloc_imp: DeallocFunctionPointer = {
    (aSelf: Unmanaged<NSRunLoop>,
    aSelector: Selector)
    -> Void in

    let unretainedSelf = aSelf.takeUnretainedValue()

    if unretainedSelf.isDispatchObserverLoaded {
        let observer = unretainedSelf.dispatchObserver
        CFRunLoopObserverInvalidate(observer)
    }

    if let original_dealloc_imp = original_dealloc_imp {
        let originalDealloc = unsafeBitCast(original_dealloc_imp,
            DeallocFunctionPointer.self)
        originalDealloc(aSelf, aSelector)
    } else {
        fatalError("The original implementation of dealloc for NSRunLoop cannot be found!")
    }
}

public enum NSRunLoopTaskInvokeTiming: Int {
    case NextLoopBegan
    case CurrentLoopEnded
    case Idle
}

extension NSRunLoop {

    public func perform(closure: ()->Void) -> Task {
        objc_sync_enter(self)
        loadDispatchObserverIfNeeded()
        let task = Task(self, closure)
        taskQueue.append(task)
        objc_sync_exit(self)
        return task
    }

    public override class func initialize() {
        super.initialize()

        struct Static {
            static var token: dispatch_once_t = 0
        }
        // make sure this isn't a subclass
        if self !== NSRunLoop.self {
            return
        }

        dispatch_once(&Static.token) {
            let selectorDealloc: Selector = "dealloc"
            original_dealloc_imp =
                class_getMethodImplementation(self, selectorDealloc)

            let swizzled_dealloc = unsafeBitCast(swizzled_dealloc_imp, IMP.self)

            class_replaceMethod(self, selectorDealloc, swizzled_dealloc, "@:")
        }
    }

    public final class Task {
        private let weakRunLoop: Weak<NSRunLoop>

        private var _invokeTiming: NSRunLoopTaskInvokeTiming
        private var invokeTiming: NSRunLoopTaskInvokeTiming {
            var theInvokeTiming: NSRunLoopTaskInvokeTiming = .NextLoopBegan
            guard let amendQueue = weakRunLoop.value?.taskAmendQueue else {
                fatalError("Accessing a dealloced run loop")
            }
            dispatch_sync(amendQueue) { () -> Void in
                theInvokeTiming = self._invokeTiming
            }
            return theInvokeTiming
        }

        private var _modes: NSRunLoopMode
        private var modes: NSRunLoopMode {
            var theModes: NSRunLoopMode = []
            guard let amendQueue = weakRunLoop.value?.taskAmendQueue else {
                fatalError("Accessing a dealloced run loop")
            }
            dispatch_sync(amendQueue) { () -> Void in
                theModes = self._modes
            }
            return theModes
        }

        private let closure: () -> Void

        private init(_ runLoop: NSRunLoop, _ aClosure: () -> Void) {
            weakRunLoop = Weak<NSRunLoop>(runLoop)
            _invokeTiming = .NextLoopBegan
            _modes = .defaultMode
            closure = aClosure
        }

        public func forModes(modes: NSRunLoopMode) -> Task {
            if let amendQueue = weakRunLoop.value?.taskAmendQueue {
                dispatch_async(amendQueue) { [weak self] () -> Void in
                    self?._modes = modes
                }
            }
            return self
        }

        public func when(invokeTiming: NSRunLoopTaskInvokeTiming) -> Task {
            if let amendQueue = weakRunLoop.value?.taskAmendQueue {
                dispatch_async(amendQueue) { [weak self] () -> Void in
                    self?._invokeTiming = invokeTiming
                }
            }
            return self
        }
    }

    private var isDispatchObserverLoaded: Bool {
        return objc_getAssociatedObject(self, &dispatchObserverKey) !== nil
    }

    private func loadDispatchObserverIfNeeded() {
        if !isDispatchObserverLoaded {
            let invokeTimings: [NSRunLoopTaskInvokeTiming] =
            [.CurrentLoopEnded, .NextLoopBegan, .Idle]

            let activities =
            CFRunLoopActivity(invokeTimings.map{ CFRunLoopActivity($0) })

            let observer = CFRunLoopObserverCreateWithHandler(
                kCFAllocatorDefault,
                activities.rawValue,
                true, 0,
                handleRunLoopActivityWithObserver)

            CFRunLoopAddObserver(getCFRunLoop(),
                observer,
                kCFRunLoopCommonModes)

            let wrappedObserver = NSAssociated<CFRunLoopObserver>(observer)

            objc_setAssociatedObject(self,
                &dispatchObserverKey,
                wrappedObserver,
                .OBJC_ASSOCIATION_RETAIN_NONATOMIC)
        }
    }

    private var dispatchObserver: CFRunLoopObserver {
        loadDispatchObserverIfNeeded()
        return (objc_getAssociatedObject(self, &dispatchObserverKey)
            as! NSAssociated<CFRunLoopObserver>)
            .value
    }

    private var taskQueue: [Task] {
        get {
            if let taskQueue = objc_getAssociatedObject(self,
                &taskQueueKey)
                as? [Task]
            {
                return taskQueue
            } else {
                let initialValue = [Task]()

                objc_setAssociatedObject(self,
                    &taskQueueKey,
                    initialValue,
                    .OBJC_ASSOCIATION_RETAIN_NONATOMIC)

                return initialValue
            }
        }
        set {
            objc_setAssociatedObject(self,
                &taskQueueKey,
                newValue,
                .OBJC_ASSOCIATION_RETAIN_NONATOMIC)

        }
    }

    private var taskAmendQueue: dispatch_queue_t {
        if let taskQueue = objc_getAssociatedObject(self,
            &taskAmendQueueKey)
            as? dispatch_queue_t
        {
            return taskQueue
        } else {
            let initialValue =
            dispatch_queue_create(
                "com.WeZZard.Nest.NSRunLoop.TaskDispatcher.TaskAmendQueue",
                DISPATCH_QUEUE_SERIAL)

            objc_setAssociatedObject(self,
                &taskAmendQueueKey,
                initialValue,
                .OBJC_ASSOCIATION_RETAIN_NONATOMIC)

            return initialValue
        }
    }

    private func handleRunLoopActivityWithObserver(observer: CFRunLoopObserver!,
        activity: CFRunLoopActivity)
        -> Void
    {
        var removedIndices = [Int]()

        let runLoopMode: NSRunLoopMode = currentRunLoopMode

        for (index, eachTask) in taskQueue.enumerate() {
            let expectedRunLoopModes = eachTask.modes
            let expectedRunLoopActivitiy =
            CFRunLoopActivity(eachTask.invokeTiming)

            let runLoopModesMatches = expectedRunLoopModes.contains(runLoopMode)
                || expectedRunLoopModes.contains(.commonModes)

            let runLoopActivityMatches =
            activity.contains(expectedRunLoopActivitiy)

            if runLoopModesMatches && runLoopActivityMatches {
                eachTask.closure()
                removedIndices.append(index)
            }
        }

        taskQueue.removeIndicesInPlace(removedIndices)
    }
}

extension CFRunLoopActivity {
    private init(_ invokeTiming: NSRunLoopTaskInvokeTiming) {
        switch invokeTiming {
        case .NextLoopBegan:        self = .AfterWaiting
        case .CurrentLoopEnded:     self = .BeforeWaiting
        case .Idle:                 self = .Exit
        }
    }
}

С помощью кода, приведенного выше, теперь мы можем отправлять выполнение UICollectionView's reloadData() в конец текущего цикла событий с помощью следующего фрагмента кода:

NSRunLoop.currentRunLoop().perform({ () -> Void in
     collectionView.reloadData()
    }).when(.CurrentLoopEnded)

Фактически, такой диспетчер задач на основе NSRunLoop уже был в одной из моих личных используемых сред: Nest. А вот его репозиторий на GitHub: https://github.com/WeZZard/Nest

3
WeZZard

Вы можете использовать этот метод

[collectionView reloadItemsAtIndexPaths:arayOfAllIndexPaths];

Вы можете добавить все indexPath объекты вашего UICollectionView в массив arrayOfAllIndexPaths, повторяя цикл для всех разделов и строк с использованием метода ниже

[aray addObject:[NSIndexPath indexPathForItem:j inSection:i]];

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

3
iDevAmit

Спасибо, прежде всего, за эту тему, очень полезно. У меня была похожая проблема с перезапуском данных, за исключением того, что симптомом было то, что определенные клетки больше не могли быть выбраны постоянно, в то время как другие могли. Нет вызова метода indexPathsForSelectedItems или его эквивалента. Отладка указала на перезагрузку данных. Я попробовал оба варианта выше; и в итоге принял параметр ReloadItemsAtIndexPaths, так как другие параметры не работали в моем случае или заставляли представление коллекции мигать в течение миллисекунды или около того. Код ниже работает хорошо:

NSMutableArray *indexPaths = [[NSMutableArray alloc] init]; 
NSIndexPath *indexPath;
for (int i = 0; i < [self.assets count]; i++) {
         indexPath = [NSIndexPath indexPathForItem:i inSection:0];
         [indexPaths addObject:indexPath];
}
[collectionView reloadItemsAtIndexPaths:indexPaths];`
1
stephane

попробуйте этот код.

 NSArray * visibleIdx = [self.collectionView indexPathsForVisibleItems];

    if (visibleIdx.count) {
        [self.collectionView reloadItemsAtIndexPaths:visibleIdx];
    }
0
Liki qu
 dispatch_async(dispatch_get_main_queue(), ^{

            [collectionView reloadData];
            [collectionView layoutIfNeeded];
            [collectionView reloadData];


        });

это сработало для меня.

0
Prajakta

Вы устанавливаете UICollectionView.contentInset? удалите левый и правый edgeInset, после удаления все в порядке, ошибка все еще существует в iOS8.3.

0
Jiang Qi

Убедитесь, что каждый из методов UICollectionView Delegate выполняет то, что вы ожидаете. Например, если 

collectionView:layout:sizeForItemAtIndexPath:

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

0
Oded Regev

Вот как это работает для меня в Swift 4

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {

let cell = campaignsCollection.dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath) as! Cell

cell.updateCell()

    // TO UPDATE CELLVIEWS ACCORDINGLY WHEN DATA CHANGES
    DispatchQueue.main.async {
        self.campaignsCollection.reloadData()
    }

    return cell
}
0
Wissa

Это случилось и со мной в iOS 8.1 SDK, но я понял это правильно, когда заметил, что даже после обновления datasource метод numberOfItemsInSection: не возвращал новое количество элементов. Я обновил счет и получил его работать.

0
Vinay Jain