Добавление изображений в массив в последовательности

Question

Я хочу добавить изображения в массив последовательно после загрузки. Я добавляю изображения в массив после загрузки один за другим, но они не находятся в последовательности. Может ли кто-нибудь рассказать мне, что это лучший способ сделать это.

var queue: NSOperationQueue = { 
    let _queue = NSOperationQueue() 
    _queue.maxConcurrentOperationCount = 4 
    return _queue 
}() 

var imageArrayNsData : [NSData] = [] 

let session = NSURLSession.sharedSession() 

@IBAction func didClickOnStart(sender: AnyObject) { 

    queue.cancelAllOperations() 

    let completionOperation = NSBlockOperation() { 
     print("all done") 
    } 

    for (index, imageURL) in imageURLs.enumerate() { 
     let operation = ImageNetworkOperation(session: session, urlString: imageURL) { image, response, error in 

      let dtA : NSData = NSData(data: UIImageJPEGRepresentation(image!, 0.75)!) 
      self.imageArrayNsData.append(dtA) 
      print("JPEG download\(index)") 
     } 

     completionOperation.addDependency(operation) 
     queue.addOperation(operation) 
    } 

    NSOperationQueue.mainQueue().addOperation(completionOperation)   
} 

Полученный выход:

JPEG download0
JPEG download2
JPEG download1
JPEG download3
all done

Answer 1

Вы должны изменить свою модель таким образом, чтобы не иметь значения, в каком порядке загружаются изображения. Например, у вас есть массив изображений URL строк:

var imageURLs: [String] 

Итак, ваш NSData должны храниться в словаре (или NSCache) шпонкой по этой строке адреса:

var imageData = [String: NSData]() 

Тогда при загрузке данные, вы можете обновить этот словарь:

self.imageData[imageURL] = dtA 

Затем, когда вам нужно получить позже эти данные, вы можете использовать ImageUrl, например:

let data = imageData[imageURLs[index]] 

Или вы можете определить его как [Int: NSData] и использовать числовой указатель в качестве ключа. Но идея состоит в том, что вы можете использовать словарь, а затем порядок, в котором вы получаете ответы, не имеет значения, но вы по-прежнему пользуетесь преимуществами выполнения параллельных запросов.

---

То, что я хотел бы предложить, было бы что-то вроде:

var imageData = [String: NSData]() 

@IBAction func didClickOnStart(sender: AnyObject) { 

    queue.cancelAllOperations() 

    let completionOperation = NSBlockOperation() { 
     print("all done") 
    } 

    for (index, imageURL) in imageURLs.enumerate() { 
     let operation = DataOperation(session: session, urlString: imageURL) { data, response, error in 
      guard let data = data where error == nil else { return } 
      guard let httpResponse = response as? NSHTTPURLResponse where httpResponse.statusCode == 200 else { return } 

      NSOperationQueue.mainQueue().addOperationWithBlock { 
       self.imageData[imageURL] = data 
      } 
      print("JPEG download\(index)") 
     } 

     completionOperation.addDependency(operation) 
     queue.addOperation(operation) 
    } 

    NSOperationQueue.mainQueue().addOperation(completionOperation)   
} 

и обращаться к нему так:

if let data = imageData[imageURLs[index]], let image = UIImage(data: data) { 
    // use `image` here 
} 

Или

var imageData = [Int: NSData]() 

@IBAction func didClickOnStart(sender: AnyObject) { 

    queue.cancelAllOperations() 

    let completionOperation = NSBlockOperation() { 
     print("all done") 
    } 

    for (index, imageURL) in imageURLs.enumerate() { 
     let operation = DataOperation(session: session, urlString: imageURL) { data, response, error in 
      guard let data = data where error == nil else { return } 
      guard let httpResponse = response as? NSHTTPURLResponse where httpResponse.statusCode == 200 else { return } 

      NSOperationQueue.mainQueue().addOperationWithBlock { 
       self.imageData[index] = data 
      } 
      print("JPEG download\(index)") 
     } 

     completionOperation.addDependency(operation) 
     queue.addOperation(operation) 
    } 

    NSOperationQueue.mainQueue().addOperation(completionOperation)   
} 

И доступ к нему, как так:

if let data = imageData[index], let image = UIImage(data: data) { 
    // use `image` here 
} 

Примечание, ImageNetworkOperation просто вызывает DataOperation получить NSData, а затем преобразовать его в UIImage.Если вы действительно хотите оригинал NSData, я бы предложил обходить ImageNetworkOperation и просто позвонить DataOperation напрямую, как показано выше.

Answer 2

попробовать:

var previousOperation : NSOperation! = nil 

    for (index, imageURL) in imageURLs.enumerate() 
    { 
     let operation = ImageNetworkOperation(session: session, urlString: imageURL) 
     { image, response, error in 

      let dtA : NSData = NSData(data: UIImageJPEGRepresentation(image!, 0.75)!) 
      self.imageArrayNsData.append(dtA) 
      print("JPEG download\(index)") 
     } 

     completionOperation.addDependency(operation) 

     if (previousOperation != nil) 
     { 
      operation.addDependency(previousOperation) 
     } 

     previousOperation = operation 
     queue.addOperation(operation) 
    } 

    NSOperationQueue.mainQueue().addOperation(completionOperation) 

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

Answer 3

Я не уверен, но я думаю, вы не можете контролировать заказ на загрузку .. означает, что все запросы конвейерны на сервер (независимо от того, какой заказ вы создаете в объектах URL). Что вам нужно сделать, вы должны поддерживать изменяемый массив или словарь, который содержит URL-адрес для фактического сопоставления данных, а затем дождаться, пока все URL-адреса будут полностью загружены. Затем итерации в известном порядке.