2

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

问题

  1. 我的代码写得不好？如果是这样，为什么？如果它更好 - 为什么？（我相信它更干净，即使字段名称更改只要重新生成早期绑定的类文件，我就不必重新编写任何代码）

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

  3. 有没有更好的方法（完全不同的第三种方法）来编写代码？

注意：我建议使用 GetPropertyName，因为他在整个代码中都对属性名称进行了硬编码，因此在另一个不使用早期绑定实体的项目中，我使用结构作为属性名称 - 如下所示。我在 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：还是花在分析上的问题/时间不值得？

4

2 回答 2

3

Early Bound，Late Bound，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没关系，但总是选择，例如我尝试在插件中使用总是后期绑定，也许在一个项目中我更喜欢使用早期绑定，通常有不止一种方法可以解决问题。

快乐的CRM编码！

于 2013-04-29T22:10:26.183 回答
2

虽然GetPropertyName这是一个非常聪明的解决方案，但我不喜欢它，这完全与可读性有关。对我来说更容易理解发生了什么：new ColumnSet(new string[] {"fullname"}).

但这几乎是个人喜好，但重要的是要记住，您不仅仅是为自己编写代码，而是为您的团队编写代码，他们应该能够轻松地理解您所做的工作。

作为一个方面，硬编码的字符串可能在运行时表现更好。我通常对我所有的值进行硬编码，如果 CRM 中的实体模型发生变化，无论如何我都必须重新访问以进行更改。在这种情况下，早绑定和晚绑定没有区别。

我不明白这个功能的意义，

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

它不做任何事情（除了投射已经属于那种类型的东西）。

1.您的代码在需要它之前不必要地创建了一个对象（因为您在 RetrieveEntity 之前使用 new 关键字实例化该对象以便将它与我的 GetProperty 方法一起使用），这是一种不好的编程习惯。在我的代码中，我从未使用过 new 关键字，但只是对其进行强制转换并且强制转换不会创建新对象。

我相信这是指；SystemUser pluginExecutedBy = new SystemUser();我可以在这里看到他/她的观点，在这种情况下new SystemUser()并没有做太多，但如果你实例化的对象做了一些资源密集型的事情（加载文件、打开数据库连接），你可能会做一些“浪费”的事情。SystemUser pluginExecutedBy = null;在这种情况下，如果更改实际上产生了任何显着的性能提升，我会感到惊讶。

2.如果 Common.Retrieve 返回 null，则您的代码分配了不必要的内存，这将导致性能问题

如果这会导致性能问题，我会感到惊讶，无论如何，正如 Guido 指出的那样，该函数在任何情况下都不会返回 null。

总的来说，我强烈认为这段代码几乎没有什么需要改变的——但事情总是会变得更好，并且值得讨论（例如代码审查的要点），尽管你不应该珍惜你的代码。

就我个人而言，我会使用硬编码的属性名称并转储 Common.RetrieveEntity 函数，因为它什么都不做。

pluginExecutedBy = service.Retrieve(SystemUser.EntityLogicalName, localContext.PluginExecutionContext.InitiatingUserId, new ColumnSet(new String[] {"fullname"} ));
于 2013-04-30T12:52:04.727 回答