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:

https://github.com/vassilych/cscs/pull/31 Commit Summary

• 88159ec https://github.com/vassilych/cscs/pull/31/commits/88159ec63b8ac3d3b2e4a3bb631413de6732ec45 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 https://github.com/vassilych/cscs/pull/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 https://github.com/vassilych/cscs/pull/31#issuecomment-1312760175, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE74VFB3Z45UQZMB7PZM2LLWIEEBXANCNFSM6AAAAAAR6MHBJI . You are receiving this because you commented.Message ID: @.***>