René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Proper prototype handling","articleBody":"# Problem\r\n\r\nCurrently JavaScript.Net does not fully support prototype related use cases:\r\n\r\nWhile it is possible to register a function which creates a new object like this:\r\n\r\n```c#\r\n_context.SetParameter(\"TestClass\", new Func\u003cTestClass\u003e(() =\u003e new TestClass()));\r\n\r\n_context.Run(@\"\r\n    let c = new TestClass();\r\n    c.foo()\").Should().Be(true);\r\n```\r\n\r\nproblems with this approach:\r\n\r\n* `instanceOf` checks are not working. `c instanceof TestClass` is false\r\n* Prototype extensions are not working (`TestClass.prototype.newMethod = function() {...}`)\r\n\r\n# Things to consider for the solution\r\n\r\n* Objects which are created by C# code should also be able to pass to JavaScript under an existing prototype\r\n* Fine grained control about which types are constructable by JS is required\r\n\r\n# Solution proposal\r\n\r\n* add method like `SetConstructor(ConstructorInfo constructor)` to the context which generates a prototype and registers under the name\r\n* when wrapping an object:\r\n  - lookup whether a prototype already exists for the type\r\n  - if no prototype is known: generate a prototype (but do not make it available to the context under a name)\r\n  - wrap object by using the newly generated or existing prototype\r\n\r\n## Side notes\r\n\r\nThis could also improve performance of wrapping objects since the analysis of type is only required once, repetitive passing could become a lot faster.\r\n\r\n---\r\n\r\n/cc @spahnke \r\n\r\n@oliverbock do you think the proposal above is sane? Then I would start to implement it.","author":{"url":"https://github.com/Ablu","@type":"Person","name":"Ablu"},"datePublished":"2018-10-05T07:15:17.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":5},"url":"https://github.com/60/Javascript.Net/issues/60"}