c#-4.0 - CRM 2011 插件 - 将早期绑定(bind)实体用于属性名称是否会导致内存问题？

Question

在我的插件代码中，我使用了早期绑定(bind)实体(通过 crmsvcutil 生成)。在我的代码中，我使用 MemberExpression 来检索属性的名称。例如，如果我想要启动插件的用户的全名，我会执行以下操作

SystemUser pluginExecutedBy = new SystemUser();
pluginExecutedBy = Common.RetrieveEntity(service
                                        , SystemUser.EntityLogicalName
                                        , new ColumnSet(new string[] {Common.GetPropertyName(() => pluginExecutedBy.FullName)})
                                        , localContext.PluginExecutionContext.InitiatingUserId).ToEntity<SystemUser>();

GetPropertyName的代码如下

    public static string GetPropertyName<T>(Expression<Func<T>> expression)
    {
        MemberExpression body = (MemberExpression)expression.Body;
        return body.Member.Name.ToLower();
    }

RetrieveEntity的代码如下

public static Entity RetrieveEntity(IOrganizationService xrmService, string entityName, ColumnSet columns, Guid entityId)
        {
            return (Entity)xrmService.Retrieve(entityName, entityId, columns);
        }

我的解决方案架构师的评论:

与其像上面那样编写代码，不如这样写(硬编码字段名称 - 或者使用结构)。

SystemUser pluginExecutedBy = null;
pluginExecutedBy = Common.RetrieveEntity(service
                                        , SystemUser.EntityLogicalName
                                        , new ColumnSet(new string[] {"fullname"})
                                        , localContext.PluginExecutionContext.InitiatingUserId).ToEntity<SystemUser>();

原因:

1. 您的代码在需要对象之前就不必要地创建对象(因为您在 RetrieveEntity 之前使用 new 关键字实例化对象以便将其与我的 GetProperty 方法一起使用)这是糟糕的编程习惯。在我的代码中，我从未使用过 new 关键字，只是对其进行强制转换，而强制转换不会创建新对象。 现在，我不是 C# 或 .NET 方面的专家，但我喜欢阅读和尝试不同的东西。因此，我查找了 Microsoft.Xrm.Sdk.dll，发现 Sdk 中的 ToEntity 实际上确实使用关键字 new 创建了一个新实体。
2. 如果 Common.Retrieve 返回 null，您的代码分配了不必要的内存，这会导致性能问题，而我的不会？像 C# 这样的托管语言为我“管理内存”，不是吗？

问题

1. 我的代码写得不好吗？如果是这样，为什么？如果更好-为什么？ (我相信这样会干净很多，即使字段名发生变化，只要重新生成早期绑定(bind)的类文件，我也不必重新编写任何代码)

2. 我同意强制转换不会创建新对象，但我的代码是否会不必要地创建对象？

3. 有没有更好的方式(完全不同的第三种方式)来写代码？

注意:我建议使用 GetPropertyName，因为他在他的所有代码中都对属性名称进行了硬编码，因此在一个不使用早期绑定(bind)实体的不同项目中，我使用结构作为属性名称 - 如下所示。我在 CRM 2011 的新工作中做了这 3 周，但后来发现了 MemberExpression 的魔力.他正在为他在他的插件中使用的每个实体编写一个巨大的 cs 文件，我告诉他他不必做任何这些因为他可以在他的插件中使用我的 GetPropertyName 方法并获取所有必需的字段和提示此代码审查评论。通常他不做代码审查。

public class ClientName
{
    public struct EntityNameA
    {
        public const string LogicalName = "new_EntityNameA";
        public struct Attributes
        {
            public const string Name = "new_name";
            public const string Status = "new_status";
        }
    }
}

PS:还是花在分析上的问题/时间不值得？

Answer 1

早绑定(bind)、晚绑定(bind)、MemberExpression、bla bla bla :)

我能理解“哲学”，但看着你的代码，我的脑海中弹出了一个巨大的警报:

public static Entity RetrieveEntity(IOrganizationService xrmService, string entityName, ColumnSet columns, Guid entityId)
        {
            return (Entity)xrmService.Retrieve(entityName, entityId, columns);
        }

如果找不到记录，Retrieve 会抛出异常。

关于其他的东西，GetPropertyName 没问题，但总是选择，例如我尝试在插件中使用 always late bound，也许在一个项目中我更喜欢使用 early bound，经常出现解决问题的方法不止一种。

快乐的 crm 编码!