Added the ability to merge cells with the OBJECT type

[stv233]

Added the ability to merge cells with the OBJECT type if the required operation is defined for the ObjectType of the cells. If the required operation is not defined in the type, the cells will be merged as a string.

[vassilych]

Привет, Михаил! Благодарю за контрибуцию к проекту. Вопрос, почему так сложно: typeof(Parser).GetMethod("MergeObjects", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Static) .MakeGenericMethod(leftCell.ObjectType) .Invoke(null, new object[] { leftCell, rightCell, script }); А не просто MergeObjects(...); Overhead ведь из-за Reflection? Василий

…

On Sat, Nov 12, 2022 at 5:08 PM Mikhail Glazatov ***@***.***> wrote: Added the ability to merge cells with the OBJECT type if the required operation is defined for the ObjectType of the cells. If the required operation is not defined in the type, the cells will be merged as a string. ------------------------------ You can view, comment on, or merge this pull request online at: #31 Commit Summary - 88159ec <88159ec> Added the ability to merge cells with the OBJECT type File Changes (1 file <https://github.com/vassilych/cscs/pull/31/files>) - *M* cscs/Parser.cs <https://github.com/vassilych/cscs/pull/31/files#diff-320ca16e8a6637a015536eb56c4ec4f4df1a25cbb25215c37d465b572893634a> (75) Patch Links: - https://github.com/vassilych/cscs/pull/31.patch - https://github.com/vassilych/cscs/pull/31.diff — Reply to this email directly, view it on GitHub <#31>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AE74VFGZ45ZIAEP6KKIY7YDWH66G5ANCNFSM6AAAAAAR6MHBJI> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[stv233]

Здравствуйте Василий.

Я использовал Reflection в этом случае поскольку мне был необходим индификатор типа, который хранит переменная, для использования в делегате Func в выражениях. Для приведения экземпляра Type, хранимого в переменной, в индификатор типа мне пришлось использовать метод MakeGenericMethod из Reflection.

Михаил

[vassilych]

Михаил, добрый вечер! Я мигрировал код с объектами и также сделал изменение, чтоб это работало без Рефлекции. Дайте знать, если работает. А также, какой лучше всего скрипт для тестирования правильных операций с двумя объектами? Василий

…

On Sun, Nov 13, 2022 at 4:43 PM Mikhail Glazatov ***@***.***> wrote: Здравствуйте Василий, Я использовал Reflection в этом случае поскольку мне был необходим индификатор типа, который хранит переменная, для использования в делегате Func в выражениях. Для приведения экземпляра Type, хранимого в переменной, в индификатор типа мне пришлось использовать метод MakeGenericMethod из Reflection. Михаил — Reply to this email directly, view it on GitHub <#31 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AE74VFB3Z45UQZMB7PZM2LLWIEEBXANCNFSM6AAAAAAR6MHBJI> . You are receiving this because you commented.Message ID: ***@***.***>