3

我需要用来NSURLSession拨打网络电话。基于某些事情，我收到响应后，需要返回一个NSError对象。

我正在使用信号量使异步调用同步运行。问题是，错误在调用内部正确设置，但是一旦信号量结束（之后

dispatch_semaphore_wait（信号量，DISPATCH_TIME_FOREVER）；

)，则err变为 nil。

请帮忙

代码：

-(NSError*)loginWithEmail:(NSString*)email Password:(NSString*)password
{
    NSError __block *err = NULL;

        // preparing the URL of login
        NSURL *Url              =       [NSURL URLWithString:urlString];

        NSData *PostData        =       [Post dataUsingEncoding:NSASCIIStringEncoding allowLossyConversion:YES];

        // preparing the request object
        NSMutableURLRequest *Request = [[NSMutableURLRequest alloc] init];
        [Request setURL:Url];
        [Request setHTTPMethod:@"POST"];
        [Request setValue:postLength forHTTPHeaderField:@"Content-Length"];
        [Request setHTTPBody:PostData];

        NSMutableDictionary __block *parsedData = NULL; // holds the data after it is parsed

        dispatch_semaphore_t semaphore = dispatch_semaphore_create(0);

        NSURLSessionConfiguration *config = [NSURLSessionConfiguration defaultSessionConfiguration];
        config.TLSMinimumSupportedProtocol = kTLSProtocol11;

        NSURLSession *session = [NSURLSession sessionWithConfiguration:config delegate:nil delegateQueue:nil];

        NSURLSessionDataTask *task = [session dataTaskWithRequest:Request completionHandler:^(NSData *data, NSURLResponse *response1, NSError *err){
                if(!data)
                {
                    err = [NSError errorWithDomain:@"Connection Timeout" code:200 userInfo:nil];
                }
                else
                {
                    NSString *formattedData = [[NSString alloc] initWithData:data encoding:NSUTF8StringEncoding];

                    NSLog(@"%@", formattedData);

                    if([formattedData rangeOfString:@"<!DOCTYPE"].location != NSNotFound || [formattedData rangeOfString:@"<html"].location != NSNotFound)
                    {
                        loginSuccessful = NO;
                        //*errorr = [NSError errorWithDomain:@"Server Issue" code:201 userInfo:nil];
                        err = [NSError errorWithDomain:@"Server Issue" code:201 userInfo:nil];
                    }
                    else
                    {
                        parsedData = [NSJSONSerialization JSONObjectWithData:data options:NSJSONReadingAllowFragments error:&err];
                        NSMutableDictionary *dict = [parsedData objectForKey:@"User"];

                        loginSuccessful = YES;
                }
            dispatch_semaphore_signal(semaphore);
        }];
        [task resume];

        // but have the thread wait until the task is done

        dispatch_semaphore_wait(semaphore, DISPATCH_TIME_FOREVER);

    return err;
}
4

3 回答 3

4

Rob 的回答告诉你如何正确地做，但不是你犯了什么错误：

您有两个名为 err 的变量，它们完全不相关。看来您还没有打开一些重要的警告，否则您的代码甚至都不会编译。

传递给完成块的参数 err 是来自 URL 请求的错误。您在没有考虑超时错误的情况下替换它 - 所以真正的错误现在丢失了。考虑到超时不是唯一的错误。

但是您设置的所有错误仅设置了在完成块中传递给您的局部变量 err ；他们根本不会触及调用者中的变量 err 。

PS。您的 JSON 处理中有几个严重错误。JSON 可以采用 UTF-16 或 UTF-32，在这种情况下，formattedData 将为 nil，并且您错误地打印“服务器问题”。如果数据不是 JSON，则无法保证它包含 DOCTYPE 或 html，那么该测试绝对是垃圾。昵称 JoeSmith 的用户会讨厌你。

将 NSJSONReadingAllowFragments 传递给 NSJSONSerialization 是无稽之谈。dict 是不可变的；如果您尝试修改它，您的应用程序将崩溃。您不检查解析器是否返回了字典，不检查键“User”是否存在值，也不检查该值是否是字典。这是您的应用程序崩溃的多种方式。

于 2015-07-23T14:47:00.437 回答
2

我建议打破僵局：您不应该使用信号量来使异步方法同步运行。采用异步模式，例如使用完成处理程序：

- (void)loginWithEmail:(NSString *)email password:(NSString*)password completionHandler:(void (^ __nonnull)(NSDictionary *userDictionary, NSError *error))completionHandler
{
    NSString *post   = ...; // build your `post` here, making sure to percent-escape userid and password if this is x-www-form-urlencoded request
    
    NSURL  *url      = [NSURL URLWithString:urlString];
    NSData *postData = [post dataUsingEncoding:NSASCIIStringEncoding allowLossyConversion:YES];
    
    NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:url];
    [request setHTTPMethod:@"POST"];
    // [request setValue:postLength forHTTPHeaderField:@"Content-Length"];                       // not needed to set length ... this is done for you
    [request setValue:@"application/x-www-form-urlencoded" forHTTPHeaderField:@"Content-Type"];  // but it is best practice to set the `Content-Type`; use whatever `Content-Type` appropriate for your request
    [request setValue:@"text/json" forHTTPHeaderField:@"Accept"];                                // and it's also best practice to also inform server of what sort of response you'll accept
    [request setHTTPBody:postData];
    
    NSURLSessionConfiguration *config = [NSURLSessionConfiguration defaultSessionConfiguration];
    config.TLSMinimumSupportedProtocol = kTLSProtocol11;
    
    NSURLSession *session = [NSURLSession sessionWithConfiguration:config delegate:nil delegateQueue:nil];
    
    NSURLSessionDataTask *task = [session dataTaskWithRequest:request completionHandler:^(NSData *data, NSURLResponse *response, NSError *err) {
        if (!data) {
            dispatch_async(dispatch_get_main_queue(), ^{
                completionHandler(nil, [NSError errorWithDomain:@"Connection Timeout" code:200 userInfo:nil]);
            });
        } else {
            NSError *parseError;
            NSDictionary *parsedData = [NSJSONSerialization JSONObjectWithData:data options:NSJSONReadingAllowFragments error:&parseError];
            
            dispatch_async(dispatch_get_main_queue(), ^{
                if (parsedData) {
                    NSDictionary *dict = parsedData[@"User"];
                    completionHandler(dict, nil);
                } else {
                    completionHandler(nil, [NSError errorWithDomain:@"Server Issue" code:201 userInfo:nil]);
                }
            });
        }
    }];
    [task resume];
}

然后这样称呼它：

[self loginWithEmail:userid password:password completionHandler:^(NSDictionary *userDictionary, NSError *error) {
    if (error) {
        // do whatever you want on error here
    } else {
        // successful, use `userDictionary` here
    }
}];

// but don't do anything reliant on successful login here; put it inside the block above

笔记：

  1. 我知道您会反对将此恢复为异步方法，但将其设为同步是一个非常糟糕的主意。首先，这是一个糟糕的用户体验（应用程序将冻结，用户不知道它是否真的在做某事或它是否已经死了），如果你的网络速度很慢，你可能会遇到各种各样的问题（例如看门狗进程可以杀死如果您在错误的时间执行此操作，您的应用程序）。

    所以，保持这个异步。理想情况下，UIActivityIndicatorView在开始异步登录之前显示，并在completionHandler. 这completionHandler也将启动过程中的下一步（例如performSegueWithIdentifier）。

  2. 我不费心测试 HTML 内容；尝试解析 JSON 并查看它是否成功更容易。您还可以通过这种方式捕获更广泛的错误。

  3. 就个人而言，我不会返回我自己的错误对象。我会继续返回操作系统给我的错误对象。这样，如果调用者必须区分不同的错误代码（例如，没有连接与服务器错误），你可以。

    如果您使用自己的错误代码，我建议不要更改domain. 应该涵盖一整domain类错误（例如，可能为您的应用程序自己的所有内部错误提供一个自定义域），而不是从一个错误到另一个错误。将该domain字段用于错误消息之类的内容不是一个好习惯。如果您想在NSError对象中添加更多描述性的内容，请将错误消息的文本放入userInfo字典中。

  4. 我可能会建议方法/变量名称符合 Cocoa 命名约定（例如，类以大写字母开头，变量和方法名称以及参数以小写字母开头）。

  5. 无需设置Content-Length（为您完成了），但设置Content-Typeand是一种很好的做法Accept（尽管不是必需的）。

于 2015-07-23T14:56:46.560 回答
1

您需要让编译器知道您将修改err. 它需要一些特殊的处理来保护它超出块的生命周期。声明它__block：

__block NSError *err = NULL;

有关更多详细信息，请参阅块编程主题中的块和变量。

于 2015-07-23T13:49:05.503 回答