ฉันได้รับคำเตือนต่อไปนี้โดยคอมไพเลอร์ ARC:

"performSelector may cause a leak because its selector is unknown".

นี่คือสิ่งที่ฉันทำ:

[_controller performSelector:NSSelectorFromString(@"someMethod")];

เหตุใดฉันจึงได้รับคำเตือนนี้ ฉันเข้าใจว่าคอมไพเลอร์ไม่สามารถตรวจสอบว่ามีตัวเลือกอยู่หรือไม่ แต่ทำไมมันถึงทำให้เกิดการรั่ว และฉันจะเปลี่ยนรหัสของฉันอย่างไรเพื่อที่ฉันจะไม่ได้รับคำเตือนนี้อีกต่อไป?

สารละลาย

คอมไพเลอร์เตือนเกี่ยวกับเรื่องนี้ด้วยเหตุผล มันยากมากที่คำเตือนนี้ควรถูกเพิกเฉยและง่ายต่อการแก้ไข นี่คือวิธี:

if (!_controller) { return; }
SEL selector = NSSelectorFromString(@"someMethod");
IMP imp = [_controller methodForSelector:selector];
void (*func)(id, SEL) = (void *)imp;
func(_controller, selector);

หรือมากกว่านั้น (อ่านยากและไม่มีผู้พิทักษ์):

SEL selector = NSSelectorFromString(@"someMethod");
((void (*)(id, SEL))[_controller methodForSelector:selector])(_controller, selector);

คำอธิบาย

สิ่งที่เกิดขึ้นที่นี่คือคุณกำลังถามตัวควบคุมสำหรับตัวชี้ฟังก์ชัน C สำหรับวิธีการที่สอดคล้องกับตัวควบคุม NSObjectตอบกลับทั้งหมดmethodForSelector:แต่คุณยังสามารถใช้class_getMethodImplementationใน Objective-C runtime (มีประโยชน์หากคุณมีการอ้างอิงโปรโตคอลเท่านั้นid<SomeProto>) เหล่านี้คำแนะนำการทำงานที่เรียกว่าIMPs และมีความเรียบง่ายtypedefคำแนะนำการทำงานเอ็ด ( id (*IMP)(id, SEL, ...)) 1 นี่อาจใกล้เคียงกับลายเซ็นวิธีการที่แท้จริงของวิธีการ แต่จะไม่ตรงกันทุกครั้ง

เมื่อคุณมีIMPแล้วคุณจะต้องโยนมันไปยังตัวชี้ฟังก์ชั่นที่มีรายละเอียดทั้งหมดที่ ARC ต้องการ (รวมถึงอาร์กิวเมนต์ที่ซ่อนอยู่สองนัยselfและ_cmdของการเรียกเมธอด Objective-C ทุกครั้ง) สิ่งนี้ถูกจัดการในบรรทัดที่สาม ( (void *)ทางด้านขวามือจะบอกคอมไพเลอร์ว่าคุณรู้ว่าคุณกำลังทำอะไรและไม่สร้างคำเตือนเนื่องจากประเภทตัวชี้ไม่ตรงกัน)

สุดท้ายคุณเรียกตัวชี้ฟังก์ชั่น2

ตัวอย่างที่ซับซ้อน

เมื่อตัวเลือกรับอาร์กิวเมนต์หรือคืนค่าคุณจะต้องเปลี่ยนสิ่งต่าง ๆ เล็กน้อย:

SEL selector = NSSelectorFromString(@"processRegion:ofView:");
IMP imp = [_controller methodForSelector:selector];
CGRect (*func)(id, SEL, CGRect, UIView *) = (void *)imp;
CGRect result = _controller ?
  func(_controller, selector, someRect, someView) : CGRectZero;

เหตุผลสำหรับคำเตือน

เหตุผลสำหรับการเตือนนี้คือด้วย ARC รันไทม์จำเป็นต้องรู้ว่าจะทำอย่างไรกับผลของวิธีการที่คุณโทร ผลที่ได้จะเป็นอะไร: void, int, char, NSString *, idฯลฯ ARC ปกติได้รับข้อมูลนี้จากส่วนหัวของชนิดของวัตถุที่คุณกำลังทำงานกับ ³

มีเพียง 4 สิ่งเท่านั้นที่ ARC จะพิจารณาสำหรับค่าส่งคืน: ⁴

1. ประเภทไม่สนใจไม่ใช่วัตถุ ( void, intฯลฯ )
2. เก็บค่าของออบเจ็กต์จากนั้นปล่อยเมื่อไม่ใช้งานอีกต่อไป (ข้อสมมติฐานมาตรฐาน)
3. ปล่อยค่าออบเจคใหม่เมื่อไม่ใช้งานอีกต่อไป (วิธีการในinit/ copyfamily หรือประกอบกับns_returns_retained)
4. ทำอะไรและถือว่าคุ้มค่าวัตถุกลับจะถูกต้องอยู่ในขอบเขตท้องถิ่น (จนถึงภายในสระว่ายน้ำเปิดตัวมากที่สุดคือการระบายน้ำมาประกอบกับns_returns_autoreleased)

การเรียกmethodForSelector:สมมติว่าค่าส่งคืนของวิธีการที่เรียกเป็นวัตถุ แต่ไม่เก็บ / ปล่อยมัน ดังนั้นคุณสามารถสร้างการรั่วไหลได้หากวัตถุของคุณควรได้รับการปล่อยตัวตามที่กล่าวไว้ในข้อ 3 ข้างต้น (นั่นคือวิธีที่คุณใช้เรียกคืนวัตถุใหม่)

สำหรับตัวเลือกที่คุณพยายามโทรหาการส่งคืนนั้นvoidหรือวัตถุอื่น ๆ ที่ไม่ใช่คุณสามารถเปิดใช้งานคุณสมบัติคอมไพเลอร์เพื่อละเว้นคำเตือน แต่อาจเป็นอันตรายได้ ฉันเคยเห็นเสียงดังกราวไปทำซ้ำสองสามครั้งว่าวิธีจัดการกับค่าส่งคืนที่ไม่ได้กำหนดให้กับตัวแปรท้องถิ่น ไม่มีเหตุผลที่เปิดใช้งาน ARC ไม่สามารถเก็บและปล่อยค่าวัตถุที่ส่งคืนจากmethodForSelector:แม้ว่าคุณจะไม่ต้องการใช้ จากมุมมองของคอมไพเลอร์มันเป็นวัตถุหลังจากทั้งหมด ซึ่งหมายความว่าหากวิธีการที่คุณโทรกำลังsomeMethodส่งคืนวัตถุที่ไม่ใช่ (รวมถึงvoid) คุณสามารถจบลงด้วยค่าตัวชี้ขยะที่ถูกเก็บไว้ / ปล่อยและผิดพลาด

ข้อโต้แย้งเพิ่มเติม

สิ่งหนึ่งที่ควรพิจารณาคือนี่คือคำเตือนเดียวกันที่จะเกิดขึ้นperformSelector:withObject:และคุณสามารถพบปัญหาที่คล้ายกันโดยไม่ต้องประกาศว่าวิธีนั้นใช้พารามิเตอร์อย่างไร ARC อนุญาตให้ประกาศพารามิเตอร์ที่ใช้ไปและหากวิธีการนั้นใช้พารามิเตอร์คุณอาจส่งข้อความไปยังซอมบี้และในที่สุด มีวิธีแก้ไขปัญหานี้กับการคัดเลือกนักแสดงบริดจ์ แต่จริงๆแล้วมันจะดีกว่าถ้าใช้วิธีการIMPและตัวชี้ฟังก์ชันด้านบน เนื่องจากพารามิเตอร์ที่ใช้แล้วไม่ค่อยมีปัญหาจึงไม่น่าจะเกิดขึ้น

ตัวเลือกแบบคงที่

ที่น่าสนใจคอมไพเลอร์จะไม่บ่นเกี่ยวกับตัวเลือกที่ประกาศแบบคงที่:

[_controller performSelector:@selector(someMethod)];

เหตุผลนี้เป็นเพราะคอมไพเลอร์จริงสามารถบันทึกข้อมูลทั้งหมดเกี่ยวกับตัวเลือกและวัตถุในระหว่างการรวบรวม ไม่จำเป็นต้องตั้งสมมติฐานเกี่ยวกับอะไรเลย (ฉันตรวจสอบสิ่งนี้เป็นปีที่แล้วโดยดูจากแหล่งที่มา แต่ตอนนี้ยังไม่มีการอ้างอิง)

การปราบปราม

ในการพยายามนึกถึงสถานการณ์ที่การยับยั้งคำเตือนนี้จะมีความจำเป็นและการออกแบบรหัสที่ดีฉันกำลังจะว่างเปล่า บางคนได้โปรดแบ่งปันหากพวกเขามีประสบการณ์ในการปิดเสียงเตือนนี้เป็นสิ่งที่จำเป็น

มากกว่า

เป็นไปได้ที่จะสร้างขึ้นNSMethodInvocationเพื่อจัดการกับสิ่งนี้เช่นกัน แต่การทำเช่นนั้นต้องใช้การพิมพ์มากขึ้นและช้าลงดังนั้นจึงมีเหตุผลเล็กน้อยที่จะทำเช่นนั้น

ประวัติศาสตร์

เมื่อperformSelector:ตระกูลของวิธีการถูกเพิ่มลงใน Objective-C เป็นครั้งแรก ARC ไม่มีอยู่จริง ในขณะที่สร้าง ARC นั้น Apple ตัดสินใจว่าควรสร้างคำเตือนสำหรับวิธีการเหล่านี้เพื่อเป็นแนวทางในการแนะนำผู้พัฒนาไปสู่การใช้วิธีการอื่นเพื่อกำหนดวิธีจัดการหน่วยความจำอย่างชัดเจนเมื่อส่งข้อความโดยผ่านตัวเลือกที่กำหนด ใน Objective-C ผู้พัฒนาสามารถทำสิ่งนี้ได้โดยใช้การใช้งานลักษณะ C บนตัวชี้ฟังก์ชันดิบ

ด้วยการแนะนำของสวิฟท์, แอปเปิ้ลมีเอกสารperformSelector:ครอบครัวของวิธีการเป็น "ไม่ปลอดภัยโดยเนื้อแท้" และพวกเขาจะไม่สามารถใช้ได้กับสวิฟท์

เมื่อเวลาผ่านไปเราได้เห็นความก้าวหน้านี้:

1. อนุญาตให้ Objective-C เวอร์ชันก่อนหน้าperformSelector:(การจัดการหน่วยความจำด้วยตนเอง)
2. Objective-C กับ ARC เตือนให้ใช้ performSelector:
3. Swift ไม่สามารถเข้าถึงperformSelector:และจัดทำเอกสารวิธีการเหล่านี้ในฐานะ "ไม่ปลอดภัยโดยเนื้อแท้"

แนวคิดของการส่งข้อความตามตัวเลือกที่ระบุชื่อนั้นไม่ใช่คุณสมบัติ "ไม่ปลอดภัยโดยเนื้อแท้" แนวคิดนี้ถูกนำมาใช้อย่างประสบความสำเร็จเป็นเวลานานใน Objective-C รวมถึงภาษาการเขียนโปรแกรมอื่น ๆ อีกมากมาย

¹เมธอด Objective-C ทั้งหมดมีอาร์กิวเมนต์ที่ซ่อนอยู่สองข้อselfและ_cmdที่เพิ่มเข้ามาโดยนัยเมื่อคุณเรียกใช้เมธอด

^{2 การ}เรียกใช้NULLฟังก์ชั่นนั้นไม่ปลอดภัยใน C. ตัวป้องกันที่ใช้ในการตรวจสอบว่ามีตัวควบคุมอยู่หรือไม่เพื่อให้แน่ใจว่าเรามีวัตถุ ดังนั้นเราจึงรู้ว่าเราจะได้รับIMPจากmethodForSelector:(แม้ว่าอาจเป็นไปได้ที่จะ_objc_msgForwardเข้าสู่ระบบการส่งต่อข้อความ) โดยพื้นฐานแล้วด้วยยามที่เรารู้ว่าเรามีฟังก์ชั่นการโทร

^{3 ที่}จริงแล้วอาจเป็นไปได้ที่จะได้รับข้อมูลที่ไม่ถูกต้องหากประกาศว่าคุณเป็นวัตถุidและคุณไม่ได้นำเข้าส่วนหัวทั้งหมด คุณสามารถจบลงด้วยการล่มในรหัสที่คอมไพเลอร์คิดว่าดี นี่เป็นของหายากมาก แต่อาจเกิดขึ้นได้ โดยปกติคุณจะได้รับคำเตือนว่าจะไม่ทราบว่าจะเลือกลายเซ็นวิธีใดจากสองวิธี

⁴ดูการอ้างอิง ARC เกี่ยวกับค่าส่งคืนที่เก็บไว้และค่าส่งคืนที่ไม่ได้รับสำหรับรายละเอียดเพิ่มเติม

ในคอมไพเลอร์ LLVM 3.0 ใน Xcode 4.2 คุณสามารถระงับคำเตือนดังต่อไปนี้:

#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Warc-performSelector-leaks"
    [self.ticketTarget performSelector: self.ticketAction withObject: self];
#pragma clang diagnostic pop

หากคุณได้รับข้อผิดพลาดในหลาย ๆ ที่และต้องการใช้ระบบมาโคร C เพื่อซ่อน pragmas คุณสามารถกำหนดแมโครเพื่อให้ง่ายต่อการระงับคำเตือน:

#define SuppressPerformSelectorLeakWarning(Stuff) \
    do { \
        _Pragma("clang diagnostic push") \
        _Pragma("clang diagnostic ignored \"-Warc-performSelector-leaks\"") \
        Stuff; \
        _Pragma("clang diagnostic pop") \
    } while (0)

คุณสามารถใช้แมโครดังนี้:

SuppressPerformSelectorLeakWarning(
    [_target performSelector:_action withObject:self]
);

หากคุณต้องการผลลัพธ์ของข้อความที่ดำเนินการคุณสามารถทำได้:

id result;
SuppressPerformSelectorLeakWarning(
    result = [_target performSelector:_action withObject:self]
);

ฉันเดาว่านี่คือสิ่งนี้: เนื่องจากตัวเลือกไม่รู้จักคอมไพเลอร์ ARC จึงไม่สามารถบังคับใช้การจัดการหน่วยความจำที่เหมาะสม

ในความเป็นจริงมีบางครั้งที่การจัดการหน่วยความจำถูกผูกไว้กับชื่อของวิธีการโดยการประชุมที่เฉพาะเจาะจง โดยเฉพาะฉันคิดว่าตัวสร้างความสะดวกสบายเมื่อเทียบกับวิธีการทำ ; อดีตกลับมาจากการประชุมวัตถุอัตโนมัติ หลังวัตถุที่ถูกเก็บรักษาไว้ หลักการประชุมขึ้นอยู่กับชื่อของตัวเลือกดังนั้นหากคอมไพเลอร์ไม่รู้จักตัวเลือกดังนั้นจึงไม่สามารถบังคับใช้กฎการจัดการหน่วยความจำที่เหมาะสม

หากสิ่งนี้ถูกต้องฉันคิดว่าคุณสามารถใช้รหัสของคุณได้อย่างปลอดภัยหากคุณตรวจสอบให้แน่ใจว่าทุกอย่างใช้ได้ในการจัดการหน่วยความจำ (เช่นวิธีการของคุณจะไม่คืนค่าวัตถุที่พวกเขาจัดสรร)

ในโครงการของคุณตั้งค่ารูปร่างภายใต้ธงคำเตือนอื่น ๆ ( WARNING_CFLAGS), เพิ่ม
-Wno-arc-performSelector-leaks

ตอนนี้ให้แน่ใจว่าตัวเลือกที่คุณโทรไม่ทำให้วัตถุของคุณถูกเก็บหรือคัดลอก

เป็นวิธีแก้ปัญหาจนกว่าคอมไพเลอร์อนุญาตให้แทนที่คำเตือนคุณสามารถใช้รันไทม์

objc_msgSend(_controller, NSSelectorFromString(@"someMethod"));

แทน

[_controller performSelector:NSSelectorFromString(@"someMethod")];

คุณจะต้อง

#import <objc/message.h>

หากต้องการละเว้นข้อผิดพลาดเฉพาะในไฟล์ที่มีตัวเลือกการดำเนินการให้เพิ่ม #pragma ดังนี้:

#pragma clang diagnostic ignored "-Warc-performSelector-leaks"

สิ่งนี้จะเพิกเฉยต่อคำเตือนในบรรทัดนี้ แต่ก็ยังอนุญาตให้ใช้ได้ตลอดทั้งโครงการที่เหลือของคุณ

แปลก แต่จริง: ถ้ายอมรับได้ (เช่นผลลัพธ์เป็นโมฆะและคุณไม่รังเกียจที่จะให้วงจร runloop หนึ่งครั้ง) เพิ่มความล่าช้าแม้ว่าจะเป็นศูนย์:

[_controller performSelector:NSSelectorFromString(@"someMethod")
    withObject:nil
    afterDelay:0];

สิ่งนี้จะลบคำเตือนน่าจะเป็นเพราะมันทำให้คอมไพเลอร์มั่นใจว่าไม่มีวัตถุใดที่สามารถส่งคืนได้และจัดการผิดพลาด

นี่คือมาโครที่อัปเดตตามคำตอบที่ระบุไว้ด้านบน อันนี้ควรอนุญาตให้คุณล้อมโค้ดของคุณแม้ว่าจะมีคำสั่ง return

#define SUPPRESS_PERFORM_SELECTOR_LEAK_WARNING(code)                        \
    _Pragma("clang diagnostic push")                                        \
    _Pragma("clang diagnostic ignored \"-Warc-performSelector-leaks\"")     \
    code;                                                                   \
    _Pragma("clang diagnostic pop")                                         \


SUPPRESS_PERFORM_SELECTOR_LEAK_WARNING(
    return [_target performSelector:_action withObject:self]
);

รหัสนี้ไม่เกี่ยวข้องกับการตั้งค่าสถานะคอมไพเลอร์หรือโทรโดยตรงรันไทม์:

SEL selector = @selector(zeroArgumentMethod);
NSMethodSignature *methodSig = [[self class] instanceMethodSignatureForSelector:selector];
NSInvocation *invocation = [NSInvocation invocationWithMethodSignature:methodSig];
[invocation setSelector:selector];
[invocation setTarget:self];
[invocation invoke];

NSInvocation อนุญาตให้ตั้งค่าอาร์กิวเมนต์หลายรายการซึ่งแตกต่างจาก performSelectorวิธีนี้ในวิธีการใด ๆ

มีคำตอบมากมายที่นี่ แต่เนื่องจากนี่แตกต่างกันเล็กน้อยรวมคำตอบสองสามข้อที่ฉันคิดว่าฉันใส่ไว้ฉันใช้หมวดหมู่ NSObject ซึ่งตรวจสอบเพื่อให้แน่ใจว่าตัวเลือกคืนค่าเป็นโมฆะและยังยับยั้งคอมไพเลอร์ คำเตือน.

#import <Foundation/Foundation.h>
#import <objc/runtime.h>
#import "Debug.h" // not given; just an assert

@interface NSObject (Extras)

// Enforce the rule that the selector used must return void.
- (void) performVoidReturnSelector:(SEL)aSelector withObject:(id)object;
- (void) performVoidReturnSelector:(SEL)aSelector;

@end

@implementation NSObject (Extras)

// Apparently the reason the regular performSelect gives a compile time warning is that the system doesn't know the return type. I'm going to (a) make sure that the return type is void, and (b) disable this warning
// See http://stackoverflow.com/questions/7017281/performselector-may-cause-a-leak-because-its-selector-is-unknown

- (void) checkSelector:(SEL)aSelector {
    // See http://stackoverflow.com/questions/14602854/objective-c-is-there-a-way-to-check-a-selector-return-value
    Method m = class_getInstanceMethod([self class], aSelector);
    char type[128];
    method_getReturnType(m, type, sizeof(type));

    NSString *message = [[NSString alloc] initWithFormat:@"NSObject+Extras.performVoidReturnSelector: %@.%@ selector (type: %s)", [self class], NSStringFromSelector(aSelector), type];
    NSLog(@"%@", message);

    if (type[0] != 'v') {
        message = [[NSString alloc] initWithFormat:@"%@ was not void", message];
        [Debug assertTrue:FALSE withMessage:message];
    }
}

- (void) performVoidReturnSelector:(SEL)aSelector withObject:(id)object {
    [self checkSelector:aSelector];

#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Warc-performSelector-leaks"
    // Since the selector (aSelector) is returning void, it doesn't make sense to try to obtain the return result of performSelector. In fact, if we do, it crashes the app.
    [self performSelector: aSelector withObject: object];
#pragma clang diagnostic pop    
}

- (void) performVoidReturnSelector:(SEL)aSelector {
    [self checkSelector:aSelector];

#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Warc-performSelector-leaks"
    [self performSelector: aSelector];
#pragma clang diagnostic pop
}

@end

เพื่อลูกหลานของฉันฉันตัดสินใจที่จะโยนหมวกของฉันลงในแหวน :)

เมื่อเร็ว ๆ นี้ฉันได้เห็นการปรับโครงสร้างมากขึ้นเรื่อย ๆ จากtarget/ selectorกระบวนทัศน์เพื่อสนับสนุนสิ่งต่าง ๆ เช่นโพรโทคอลบล็อก ฯลฯ อย่างไรก็ตามมีการแทนที่แบบหล่นลงหนึ่งperformSelectorที่ฉันใช้ไม่กี่ครั้ง:

[NSApp sendAction: NSSelectorFromString(@"someMethod") to: _controller from: nil];

เหล่านี้ดูเหมือนจะสะอาด ARC ปลอดภัยและเกือบจะเหมือนกับแทนโดยไม่ต้องมากเกี่ยวกับperformSelectorobjc_msgSend()

แม้ว่าฉันจะไม่รู้ว่ามีอะนาล็อกใน iOS

คำตอบของ Matt Galloway ในหัวข้อนี้อธิบายถึงสาเหตุ:

พิจารณาสิ่งต่อไปนี้:

id anotherObject1 = [someObject performSelector:@selector(copy)];
id anotherObject2 = [someObject performSelector:@selector(giveMeAnotherNonRetainedObject)];

ทีนี้ ARC จะรู้ได้อย่างไรว่าอันแรกส่งคืนอ๊อบเจ๊กต์ที่มีจำนวนนับคงที่ 1 แต่อันที่สองส่งคืนออบเจ็กต์ที่ถูกลบอัตโนมัติ

ดูเหมือนว่าโดยทั่วไปจะปลอดภัยที่จะระงับคำเตือนหากคุณไม่สนใจค่าส่งคืน ฉันไม่แน่ใจว่าวิธีปฏิบัติที่ดีที่สุดคืออะไรถ้าคุณต้องการรับวัตถุที่เก็บไว้จากตัวเลือกตัวเลือก - นอกจาก "อย่าทำอย่างนั้น"

@ C-ถนนให้การเชื่อมโยงทางด้านขวาที่มีคำอธิบายปัญหาที่นี่ ด้านล่างคุณสามารถดูตัวอย่างของฉันเมื่อ performerelector ทำให้หน่วยความจำรั่ว

@interface Dummy : NSObject <NSCopying>
@end

@implementation Dummy

- (id)copyWithZone:(NSZone *)zone {
  return [[Dummy alloc] init];
}

- (id)clone {
  return [[Dummy alloc] init];
}

@end

void CopyDummy(Dummy *dummy) {
  __unused Dummy *dummyClone = [dummy copy];
}

void CloneDummy(Dummy *dummy) {
  __unused Dummy *dummyClone = [dummy clone];
}

void CopyDummyWithLeak(Dummy *dummy, SEL copySelector) {
  __unused Dummy *dummyClone = [dummy performSelector:copySelector];
}

void CloneDummyWithoutLeak(Dummy *dummy, SEL cloneSelector) {
  __unused Dummy *dummyClone = [dummy performSelector:cloneSelector];
}

int main(int argc, const char * argv[]) {
  @autoreleasepool {
    Dummy *dummy = [[Dummy alloc] init];
    for (;;) { @autoreleasepool {
      //CopyDummy(dummy);
      //CloneDummy(dummy);
      //CloneDummyWithoutLeak(dummy, @selector(clone));
      CopyDummyWithLeak(dummy, @selector(copy));
      [NSThread sleepForTimeInterval:1];
    }} 
  }
  return 0;
}

วิธีเดียวที่ทำให้หน่วยความจำรั่วในตัวอย่างของฉันคือ CopyDummyWithLeak เหตุผลก็คือ ARC ไม่ทราบว่า copySelector ส่งคืนวัตถุที่ถูกเก็บไว้

หากคุณเรียกใช้ Memory Leak Tool คุณจะเห็นภาพต่อไปนี้: ... และไม่มีการรั่วไหลของหน่วยความจำในกรณีอื่น ๆ :

เพื่อทำให้มาโครของ Scott Thompson เป็นเรื่องทั่วไปมากขึ้น:

// String expander
#define MY_STRX(X) #X
#define MY_STR(X) MY_STRX(X)

#define MYSilenceWarning(FLAG, MACRO) \
_Pragma("clang diagnostic push") \
_Pragma(MY_STR(clang diagnostic ignored MY_STR(FLAG))) \
MACRO \
_Pragma("clang diagnostic pop")

จากนั้นใช้แบบนี้:

MYSilenceWarning(-Warc-performSelector-leaks,
[_target performSelector:_action withObject:self];
                )

อย่าระงับคำเตือน!

มีทางเลือกไม่น้อยกว่า12วิธีในการแก้ไขปัญหาด้วยคอมไพเลอร์
ในขณะที่คุณฉลาดในขณะที่มีการใช้งานครั้งแรกวิศวกรเพียงไม่กี่คนบนโลกสามารถติดตามรอยเท้าของคุณและรหัสนี้จะแตกในที่สุด

เส้นทางที่ปลอดภัย:

โซลูชันทั้งหมดเหล่านี้จะทำงานได้โดยมีระดับความผันแปรตามเจตนาดั้งเดิมของคุณ สมมติว่าparamเป็นได้nilถ้าคุณต้องการ:

เส้นทางที่ปลอดภัยพฤติกรรมที่เป็นแนวคิดเดียวกัน:

// GREAT
[_controller performSelectorOnMainThread:selector withObject:anArgument waitUntilDone:YES];
[_controller performSelectorOnMainThread:selector withObject:anArgument waitUntilDone:YES modes:@[(__bridge NSString *)kCFRunLoopDefaultMode]];

[_controller performSelector:selector onThread:[NSThread mainThread] withObject:anArgument waitUntilDone:YES];
[_controller performSelector:selector onThread:[NSThread mainThread] withObject:anArgument waitUntilDone:YES modes:@[(__bridge NSString *)kCFRunLoopDefaultMode]];

เส้นทางที่ปลอดภัยพฤติกรรมที่แตกต่างกันเล็กน้อย:

(ดูที่นี้การตอบสนอง)
ใช้กระทู้ใด ๆ [NSThread mainThread]แทน

// GOOD
[_controller performSelector:selector withObject:anArgument afterDelay:0];
[_controller performSelector:selector withObject:anArgument afterDelay:0 inModes:@[(__bridge NSString *)kCFRunLoopDefaultMode]];

[_controller performSelectorOnMainThread:selector withObject:anArgument waitUntilDone:NO];
[_controller performSelectorOnMainThread:selector withObject:anArgument waitUntilDone:NO];
[_controller performSelectorOnMainThread:selector withObject:anArgument waitUntilDone:NO modes:@[(__bridge NSString *)kCFRunLoopDefaultMode]];

[_controller performSelectorInBackground:selector withObject:anArgument];

[_controller performSelector:selector onThread:[NSThread mainThread] withObject:anArgument waitUntilDone:NO];
[_controller performSelector:selector onThread:[NSThread mainThread] withObject:anArgument waitUntilDone:NO modes:@[(__bridge NSString *)kCFRunLoopDefaultMode]];

เส้นทางอันตราย

ต้องใช้การคอมไพล์เลอร์แบบเงียบ ๆ ซึ่งถูกทำลาย ทราบว่าในช่วงเวลาปัจจุบันก็ไม่ทำลายในสวิฟท์

// AT YOUR OWN RISK
[_controller performSelector:selector];
[_controller performSelector:selector withObject:anArgument];
[_controller performSelector:selector withObject:anArgument withObject:nil];

เนื่องจากคุณใช้ ARC คุณต้องใช้ iOS 4.0 ขึ้นไป หมายความว่าคุณสามารถใช้บล็อกได้ หากแทนที่จะจดจำตัวเลือกให้ดำเนินการแทนบล็อก ARC จะสามารถติดตามสิ่งที่เกิดขึ้นจริงได้ดีขึ้นและคุณไม่ต้องเสี่ยงกับการแนะนำหน่วยความจำรั่วไหลโดยไม่ตั้งใจ

แทนที่จะใช้วิธีบล็อกซึ่งทำให้ฉันมีปัญหา:

    IMP imp = [_controller methodForSelector:selector];
    void (*func)(id, SEL) = (void *)imp;

ฉันจะใช้ NSInvocation แบบนี้:

    -(void) sendSelectorToDelegate:(SEL) selector withSender:(UIButton *)button 

    if ([delegate respondsToSelector:selector])
    {
    NSMethodSignature * methodSignature = [[delegate class]
                                    instanceMethodSignatureForSelector:selector];
    NSInvocation * delegateInvocation = [NSInvocation
                                   invocationWithMethodSignature:methodSignature];


    [delegateInvocation setSelector:selector];
    [delegateInvocation setTarget:delegate];

    // remember the first two parameter are cmd and self
    [delegateInvocation setArgument:&button atIndex:2];
    [delegateInvocation invoke];
    }

ถ้าคุณไม่จำเป็นต้องผ่านการขัดแย้งใด ๆ valueForKeyPathวิธีแก้ปัญหาที่ง่ายคือการใช้งาน สิ่งนี้สามารถเกิดขึ้นได้กับClassวัตถุ

NSString *colorName = @"brightPinkColor";
id uicolor = [UIColor class];
if ([uicolor respondsToSelector:NSSelectorFromString(colorName)]){
    UIColor *brightPink = [uicolor valueForKeyPath:colorName];
    ...
}

คุณสามารถใช้โปรโตคอลได้ที่นี่ ดังนั้นสร้างโปรโตคอลอย่างเช่น:

@protocol MyProtocol
-(void)doSomethingWithObject:(id)object;
@end

ในชั้นเรียนของคุณที่ต้องโทรหาตัวเลือกของคุณคุณจะมี @property

@interface MyObject
    @property (strong) id<MyProtocol> source;
@end

เมื่อคุณต้องการโทร@selector(doSomethingWithObject:)ใน MyObject ให้ทำดังนี้

[self.source doSomethingWithObject:object];