Improve performance of `setNameWithFormat:`

[reactivecocoa]

Improve performance of `setNameWithFormat:`

[mdiep]

The objection to this approach is that it keeps objects alive.

[hfossli]

Are you thinking about the caching?

- (instancetype)setNameBlock:(NSString *(^)(void))nameBlock
{
    __block NSString *cache;
    _nameBlock = [^NSString *{
        if(cache == nil)
        {
            cache = nameBlock();
        }
        return cache;
    } copy];
    return self;
}

Caching can be omitted.

[mdiep]

foo is kept alive in your example:

- (void)testWithSignal
{
    Foo *foo = [Foo new];

    RACSignal *signal = [[RACSignal new] someOperatorWithObject:foo];
    XCTAssertTrue(foo.descriptionRequestedCount == 0, @"");
    XCTAssertEqualObjects([signal name], @"transformed using `someOperatorWithObject` with object <Foo 0xFOOBAR>");
    XCTAssertTrue(foo.descriptionRequestedCount == 1, @"");
    XCTAssertEqualObjects([signal name], @"transformed using `someOperatorWithObject` with object <Foo 0xFOOBAR>");
    XCTAssertTrue(foo.descriptionRequestedCount == 1, @"");
}

[hfossli]

You're right. That means the convenience macro is not a good idea as each object would need to weakly captured.

And my feeling is that this is probably too cumbersome

- (instancetype)someOperatorWithObject:(NSObject *)object
{    
    @weakify(object);
    return [self setNameBlock:^NSString *{ 
        @strongify(object);
        return [NSString stringWithFormat:@"transformed using `someOperatorWithObject` with object %@", [object description]];
    }];
}

[mdiep]

You could probably build the @weakify/@strongify into a macro that sets the name. Feel free to open a PR if you'd like to work on that.

[hfossli]

Yeah, I could do a PR if you guys think this is a good idea. Don't want to invest time and then get it rejected if you fundamentally disagree about the concept :)

[mdiep]

No, that seems like an acceptable approach to me.