some function not work about jspatch HOT 9 CLOSED

@cyvoit By now, JSPatch can't add method to an OC class if the method not exists in OC, I'm figuring out a way to solve this problem, would be update soon.

from jspatch.

I think tableView_didSelectRowAtIndexPath is an existing method - (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPathof protocol UITableViewDelegate.

from jspatch.

@cyvoit it's a protocol, not an exists method.
[[[UITableViewController alloc] init] respondsToSelector:@selector(tableView:didSelectRowAtIndexPath:)] == NO

from jspatch.

but why the other method of protocol can be run?

from jspatch.

@wy19901227 because the base class UITableViewController implemented some of the methods.

from jspatch.

    XCTAssertFalse([[[UITableViewController alloc] init] respondsToSelector:@selector(tableView:didSelectRowAtIndexPath:)]);
    XCTAssertTrue([[[UITableViewController alloc] init] respondsToSelector:@selector(tableView:cellForRowAtIndexPath:)]);

Test succeeded!
Maybe because cellForRowAtIndexPath is a required method while didSelectRowAtIndexPath is an optional method.

from jspatch.

how to test？

from jspatch.

@wy19901227 add the test code to a test file, e.g. JSPatchTests.m and run the test target. Maybe it's another topic.

from jspatch.

@cyvoit Solved in 653074f. We can add new method to OC class now.
The tableView_didSelectRowAtIndexPath is work now, but there is something wrong if the protocol method params type is not all id, will support protocol in the future.

from jspatch.