objective-c - NSURLConnectionDelegate 静的クラスラッパーが関数呼び出しを呼び出さない (ARC)

Question

HTTP 通信ロジックをカプセル化する iOS アプリの静的クラスをまとめようとしています。サインを介してObjective-Cで静的メソッドを作成する方法を知って+いますが、クラスがブロックを受け入れて、タスクが完了したときにそれを呼び出すことができるようにしたいと考えています。これまでに取得したクラスは正常にコンパイルされますが、その関数を呼び出すと、特定のポイントに到達するだけで失敗します。これが私の現在のコードです：

#import <UIKit/UIKit.h>

#pragma mark - Globals
#import "Globals.h"

@interface HTTPRequest : NSObject <NSURLConnectionDelegate>
+ (void)GET:(NSMutableURLRequest *)request completionBlock:(void (^)(void))block;
+ (void)POST:(NSMutableURLRequest *)request andBody:(NSString *)requestString completionBlock:(void (^)(void))block;
@end

#import "HTTPRequest.h"

@interface HTTPRequest () {
    NSInteger statusCode;
}
@end

static void (^completionBlock)(void);

@implementation HTTPRequest
+ (void)GET:(NSMutableURLRequest *)request completionBlock:(void (^)(void))block {
    [[UIApplication sharedApplication] setNetworkActivityIndicatorVisible:YES];

    [request setHTTPMethod:@"GET"];

    NSURLConnection *connection = [[NSURLConnection alloc] initWithRequest:request
                                                                  delegate:self];

    if (connection) {
        responseData = [[NSMutableData alloc] init];
    } else {
        NSLog(@"NSURLConnection INSTANCE FAILED");
    };

    completionBlock = block;
}

+ (void)POST:(NSMutableURLRequest *)request andBody:(NSString *)requestString completionBlock:(void (^)(void))block {
    [[UIApplication sharedApplication] setNetworkActivityIndicatorVisible:YES];

    NSData *requestData = [NSData dataWithBytes:[requestString UTF8String]
                                         length:[requestString length]];

    [request setHTTPMethod:@"POST"];
    [request setValue:@"application/x-www-form-urlencoded" forHTTPHeaderField:@"content-type"];
    [request setHTTPBody:requestData];

    NSURLConnection *connection = [[NSURLConnection alloc] initWithRequest:request
                                                                  delegate:self];

    if (connection) {
        responseData = [[NSMutableData alloc] init];
    } else {
        NSLog(@"NSURLConnection INSTANCE FAILED");
    };

    completionBlock = block;
}

#pragma mark - NSURLConnectionDelegate Methods
- (void)connection:(NSURLConnection *)connection didReceiveResponse:(NSURLResponse *)response {
    [responseData setLength:0];

    statusCode = [((NSHTTPURLResponse *)response) statusCode];
}

- (void)connection:(NSURLConnection *)connection didReceiveData:(NSData *)data {
    [responseData appendData:data];
}

- (void)connection:(NSURLConnection *)connection didFailWithError:(NSError *)error {
    NSLog(@"CONNECTION ERROR: %@ %@", [error localizedDescription], [[error userInfo] objectForKey:NSURLErrorFailingURLStringErrorKey]);
}

- (void)connectionDidFinishLoading:(NSURLConnection *)connection {
    NSLog(@"RESPONSE STRING: %@", [[NSString alloc] initWithData:responseData encoding:NSUTF8StringEncoding]);

    [[UIApplication sharedApplication] setNetworkActivityIndicatorVisible:NO];

    if (statusCode == 200) {
        completionBlock();
    } else {
        NSLog(@"RESPONSE ERROR: %i", statusCode);
    };
}
@end

GET および POST 関数が呼び出され、それらがオブジェクトを作成することを知っていNSURLConnectionます。その後、どのNSURLConnectionDelegate関数も呼び出されません。

誰でも問題を見つけることができますか、あるいは、私がこれについて完全に間違っている場合は、私に知らせてください. 前もって感謝します！

アップデート

アプリケーションが ARC ベースであることを指摘しておく必要があります。関数が呼び出されていないことについて、ここで他の質問を読みましたNSURLConnectionDelegateが、それらのほとんどは、開発者がオブジェクトをとして設定したことが原因のようですautorelease。私はそれを制御できないので、問題はないと思います。

更新 2 (書き換え?)

静的関数からクラスのインスタンスを作成し、そのインスタンス関数を使用するいくつかの変更を加えました。これでNSURLConnectionDelegate関数が呼び出され、onSuccessBlockブロックも呼び出されていることがわかったので、クラス全体が最後まで実行されています。率直に言って、これが正当なコードであるかどうかはわかりませんが、コンパイラは文句を言っておらず、期待どおりのログメッセージが表示されているので、そのまま使用します。現在の実装は次のとおりです。

#import <UIKit/UIKit.h>

@interface HTTPRequest : NSObject <NSURLConnectionDelegate>
+ (void)requestAsGET:(NSMutableURLRequest *)request onSuccess:(void (^)(void))successBlock onFail:(void (^)(void))failBlock;
- (void)requestAsGET:(NSMutableURLRequest *)request onSuccess:(void (^)(void))successBlock onFail:(void (^)(void))failBlock;
+ (void)requestAsPOST:(NSMutableURLRequest *)request body:(NSString *)body onSuccess:(void (^)(void))successBlock onFail:(void (^)(void))failBlock;
- (void)requestAsPOST:(NSMutableURLRequest *)request body:(NSString *)body onSuccess:(void (^)(void))successBlock onFail:(void (^)(void))failBlock;
@end

#import "HTTPRequest.h"

@interface HTTPRequest () {
    NSMutableData *responseData;
    NSInteger statusCode;

    void (^onSuccessBlock)(void);
    void (^onFailBlock)(void);
}
@end

@implementation HTTPRequest
- (id)init {
    self = [super init];

    if (self) {
    };

    return self;
}

+ (void)requestAsGET:(NSMutableURLRequest *)request onSuccess:(void (^)(void))successBlock onFail:(void (^)(void))failBlock {
    [[[HTTPRequest alloc] init] requestAsGET:request
                                   onSuccess:successBlock
                                      onFail:failBlock];
}

- (void)requestAsGET:(NSMutableURLRequest *)request onSuccess:(void (^)(void))successBlock onFail:(void (^)(void))failBlock {
    [[UIApplication sharedApplication] setNetworkActivityIndicatorVisible:YES];

    [request setHTTPMethod:@"GET"];

    NSURLConnection *connection = [[NSURLConnection alloc] initWithRequest:request
                                                                  delegate:self];

    if (connection) {
        onSuccessBlock = successBlock;
        onFailBlock = failBlock;
    } else {
        NSLog(@"NSURLConnection INSTANCE FAILED");
    };
}

+ (void)requestAsPOST:(NSMutableURLRequest *)request body:(NSString *)body onSuccess:(void (^)(void))successBlock onFail:(void (^)(void))failBlock {
    [[[HTTPRequest alloc] init] requestAsPOST:request
                                         body:body
                                    onSuccess:successBlock
                                       onFail:failBlock];
}

- (void)requestAsPOST:(NSMutableURLRequest *)request body:(NSString *)body onSuccess:(void (^)(void))successBlock onFail:(void (^)(void))failBlock {
    [[UIApplication sharedApplication] setNetworkActivityIndicatorVisible:YES];

    NSData *requestData = [NSData dataWithBytes:[body UTF8String]
                                         length:[body length]];

    [request setHTTPMethod:@"POST"];
    [request setValue:@"application/x-www-form-urlencoded" forHTTPHeaderField:@"content-type"];
    [request setHTTPBody:requestData];

    NSURLConnection *connection = [[NSURLConnection alloc] initWithRequest:request
                                                                  delegate:self];

    if (connection) {
        onSuccessBlock = successBlock;
        onFailBlock = failBlock;
    } else {
        NSLog(@"NSURLConnection INSTANCE FAILED");
    };
}

#pragma mark - NSURLConnectionDelegate Methods
- (void)connection:(NSURLConnection *)connection didReceiveResponse:(NSURLResponse *)response {
    [responseData setLength:0];

    statusCode = [((NSHTTPURLResponse *)response) statusCode];
}

- (void)connection:(NSURLConnection *)connection didReceiveData:(NSData *)data {
    [responseData appendData:data];
}

- (void)connection:(NSURLConnection *)connection didFailWithError:(NSError *)error {
    NSLog(@"CONNECTION ERROR: %@ %@", [error localizedDescription], [[error userInfo] objectForKey:NSURLErrorFailingURLStringErrorKey]);
}

- (void)connectionDidFinishLoading:(NSURLConnection *)connection {
    NSLog(@"RESPONSE STRING: %@", [[NSString alloc] initWithData:responseData encoding:NSUTF8StringEncoding]);

    [[UIApplication sharedApplication] setNetworkActivityIndicatorVisible:NO];

    if (statusCode == 200) {
        //  Pass responseData into the block... How?

        onSuccessBlock();
    } else {
        //  Pass statusCode into the block?

        onFailBlock();
    };
}
@end

score 1 · Accepted Answer

静的クラスの代わりにシングルトンパターンを使用します。そうすれば、別のリクエストが以前のデータを上書きすることを心配する必要はありません。

たとえば、 NSMutableData responseDataを取ります。接続のライフサイクルを通じてそのデータを追跡するには、インスタンス変数が必要になります。見たことはありませんが、持っている場合は、他の接続がそれを更新できないことを確認する必要があります. つまり、最初の反応である可能性のある静的変数にすることはできません。

いずれにせよ、NSUrlConnection をキーとしてresponseData変数をディクショナリに入れる必要があります。そうすれば、各デリゲートメソッドの実装で、接続からデータを検索し、必要なことを実行できます。

したがって、このシナリオでは、次のようなことができます。

NSURLConnection *connection = [[NSURLConnection alloc] initWithRequest:request
                                                              delegate:self];

if (connection) {
    responseData = [[NSMutableData alloc] init];

    // So this would be a NSMutableDictionary you would store your connection in 
    [requests setValue:responseData forKey:connection];

} else {
    NSLog(@"NSURLConnection INSTANCE FAILED");
};

次に、デリゲートメソッドでこれを行うことができます

- (void) connection:(NSURLConnection *)connection didReceiveData:(NSData *)data {
NSMutableData *responseData = [requests valueForKey:connection];

[responseData appendData:data];
}

私の個人的な実装では、NSMutableDictionary は追加されたオブジェクトを保持しないと考えているため、NSMutableDictionary の代わりに CFMutableDictionaryRef を使用することにしました。私のコードは ARC より前に書かれているので、心配する必要はないかもしれません。この方法を試して EXC_BAD_ACCESS を取得し始めたり、メモリの問題が発生した場合は、CFMutableDictionaryRef を使用してください。

これが正しい方向に進むことを願っています。あなたの頭は間違いなく正しい場所にあります:)

リライトベース

それで、あなたの書き直しに基づいて、私が上で述べたことをよりよく説明しようとしましょう：

- (void)requestAsGET:(NSMutableURLRequest *)request onSuccess:(void (^)(void))successBlock onFail:(void (^)(void))failBlock {
    [[UIApplication sharedApplication] setNetworkActivityIndicatorVisible:YES];

    [request setHTTPMethod:@"GET"];

    NSURLConnection *connection = [[NSURLConnection alloc] initWithRequest:request
                                                              delegate:self];

    if (connection) {

       /** This is your problem. You are trying to save data to some instance variables
           but there's a data integrity issue here.  If you fire off two requests, the 
           second request will overwrite these two values which would, at the time, contain
           information from the first request */

        onSuccessBlock = successBlock;
        onFailBlock = failBlock;

    } else {
        NSLog(@"NSURLConnection INSTANCE FAILED");
    };
}

したがって、最初のステップで設計上の問題が発生します。この問題を解決するまで、そこから何が起こるかはあまり重要ではありません。私の提案は、現在の NSURLConnection をキーとして使用して、** HTTPRequest** 値オブジェクトを辞書に保存することです。このように、各デリゲートメソッドで、その接続に固有のデータを検索して変更することができます。他のリクエストがデータを消去することを心配する必要はありません。

このようなもの：

if (connection) {

    // Store the data in an object... call it whatever you want
    SomeObject *someObject = [[SomeObject all] init];
    someObject.responseData = [[NSMutableData alloc] init];
    someObject.onSuccessBlock = successBlock;
    someObject.onFailBlock = failBlock;

    /** Have an NSMutableDictionary that stores all the value objects
        under the corresponding connection since ALL NSURLConnectionDelegate
        methods include a reference to the NSURLConnection for the request */
    [requests setValue:someObject forKey:connection];

} else {
    NSLog(@"NSURLConnection INSTANCE FAILED");
};

score 1 · Accepted Answer

https://github.com/nst/STHTTPRequestなどの単純な NSURLConnection ラッパーからインスピレーションを得てください。

また、ARC プロジェクトで非 ARC コードを使用することもできます。このために、ファイル単位で ARC を使用しないようにコンパイラに指示できます。 Disable Automatic Reference Counting for Some Files。

objective-c - NSURLConnectionDelegate 静的クラス ラッパーが関数呼び出しを呼び出さない (ARC)

2 に答える 2

Related

Reference

objective-c - NSURLConnectionDelegate 静的クラスラッパーが関数呼び出しを呼び出さない (ARC)