8

尝试实现一个应用程序，该应用程序在连接到 Internet 时将存储在本地数据库中的离线数据发送到 Web 服务器。我使用下面显示的代码。到目前为止，我已经测试它工作正常，不确定它是否适用于大量记录。我想知道对这段代码的任何调整是否可以提高性能？？？

笔记

  • 我知道这对于离线同步来说是最糟糕的代码，所以试着更好地调整它。

  • 它是从应用程序到服务器的单向同步。

    -(void)FormatAnswersInJSON {

  DMInternetReachability *checkInternet = [[DMInternetReachability alloc] init];
  if ([checkInternet isInternetReachable]) {
     if ([checkInternet isHostReachable:@"www.apple.com"]) {//Change to domain
        responseArray = [[NSMutableArray alloc] init];

        dispatch_async(backgroundQueue, ^(void) {

            NSArray *auditIDArray = [[NSArray alloc] initWithArray: [self getUnuploadedIDs]];
            for (int temp = 0; temp < [auditIDArray count]; temp ++) {

                // Code to post JSON to server

                NSURLResponse *response;
                NSData *urlData=[NSURLConnection sendSynchronousRequest:request returningResponse:&response error:&error];
                if (!error) {
                    NSString *responseID = [[NSString alloc]initWithData:urlData encoding:NSUTF8StringEncoding];
                    if ([responseID isEqualToString:@"ERROR"]) {
                        //Error uploading records
                    } else {
                       [responseArray addObject:responseID];
                    }
                } else {
                   //Error
                   return;
                }
            }
            dispatch_async( backgroundQueue, ^{

                /* Based on return code update local DB */
                for (int temp = 0; temp < [responseArray count]; temp ++) {
                   [self updateRecordsForID:[auditIDArray objectAtIndex:temp] withID:[responseArray objectAtIndex:temp]];
                }
            });
        });
     }
  }
}

- (void)upload { //Called when internet connection available

    if(backgroundQueue){
        dispatch_suspend(backgroundQueue);
        dispatch_release(backgroundQueue);
        backgroundQueue = nil;
    }
    backgroundQueue = dispatch_queue_create("com.XXXX.TestApp.bgqueue", NULL);
    dispatch_async(backgroundQueue, ^(void) {
        [self FormatAnswersInJSON];
    });    
}
4

2 回答 2

1

如果这段代码摆在我面前，我的方法是：

  • 查看用例并定义“大量记录”：一次会定期更新 50 条记录吗？还是会在 1s 和 2s 中？我的用户有 wifi 连接还是通过付费网络连接？等。
  • 如果可能，请在野外进行测试。如果我的用户群足够小，请收集真实数据并让其指导我的决策，或者仅将功能发布给用户/beta 测试和测量的子集。
  • 如果数据告诉您，则优化此代码以提高效率。

我的优化途径是进行组处理。粗略的算法类似于：

for records in groups of X
  collect
  post to server {
    on return:
      gather records that updated successfully
      update locally
  }

这假设您可以修改服务器代码。您可以进行 10、20、50 组等。这一切都取决于发送的数据类型和大小。

组算法意味着更多的预处理客户端，但具有减少 HTTP 请求的优点。如果您只打算获得少量更新，这就是YAGNI和未成熟的优化。

不要让这个决定阻止您发货！

于 2013-03-14T22:55:42.460 回答
0

您的代码有几个问题。一种约定是在测试错误参数之前始终检查返回值。可能会设置错误参数 - 即使方法成功。

当NSURLConnection用于除快速示例或测试之外的任何其他内容时，您还应该始终使用异步样式来处理委托方法。由于NSURLConnection 正确使用可能会很快变得麻烦且容易出错，因此我建议使用第三方框架，该框架将NSURLConnection对象和所有与连接相关的状态信息封装为NSOperation. 您可以在 Apple 示例中找到一个示例实现：QHTTPOperation. 另一个合适的第三方框架是 AFNetworking（在 GitHub 上）。

当您将异步样式与委托或第三方子类一起使用时，您可以取消连接、检索详细的错误或进度信息、执行身份验证等等 - 这是同步 API 无法做到的。

我认为，一旦你完成了这个并且你的方法正常工作，你可以测试性能是否可以接受。但是除非你有大数据——比如>2 MByte——我不会太担心。

如果您的数据变得非常大，例如 > 10 MByte，您需要考虑改进您的方法。例如，您可以将 POST 数据作为文件流而不是NSData对象提供（参见NSURLRequest's property HTTPBodyStream）。使用流避免将所有 POST 数据加载到 RAM 中，这有助于缓解 RAM 受限问题。

如果您有较小的 POST 数据，但可能有很多，您可能会考虑使用NSOperationQueue放置NSOperation连接子类的位置。将最大并发操作数设置为 2。这可能会利用 HTTP 管道 - 如果服务器支持这一点，这实际上会减少延迟。

当然，您的应用程序中可能还有其他部分，例如您创建或检索您必须发送的数据，这可能会影响整体性能。但是，如果您的代码是健全的并且利用调度队列或 NSOperations 让事情以并行方式执行，那么就没有更多的选项来提高连接的性能了。

于 2013-05-15T13:07:48.390 回答