Question

我有5个不同的数据源（request，lsitCC，listSEDI，listSEDIFees和XMLRoot加载到相应的C＃Array列表对象中）。我需要根据特定条件组合来自所有这些源的数据来构造JSON请求。我使用NewtonSoft.JSON在C＃中编写了下面的代码。这个LINQ查询的圈复杂度大约是40，这是更高的一面。任何人都可以请求如何减少复杂性。提前致谢。我认为查询非常易于用户阅读，如果需要内联注释，请告诉我。

    var input = from RequestNode in request
              select new
              {
                  Documents = (from objCC in lsitCC
                               where objCC.ID == RequestNode.ID
                               select new
                               {
                                   Request = (from objSEDI in listSEDI
                                                         where objSEDI.ID == objCC.ID && RequestNode.POSTAL.Count(p => p.PID == objSEDI.PID) > 0
                                                         join Config in RequestNode.POSTAL on objSEDI.PID equals Config.PID
                                                         select new
                                                         {
                                                             ReqItem = (Config.ReqItem == null) ? "" : Config.ReqItem,
                                                             Code = (RequestNode.Code == null) ? "" : RequestNode.Code,
                                                             Camp = (RequestNode.Camp == null) ? "" : RequestNode.Camp,
                                                             CCT = new
                                                             {
                                                                 ID = (objCC.ID == null) ? "" : objCC.ID,
                                                                 Band = (RequestNode.Band == null) ? "" : RequestNode.Band,
                                                                 Context = (RequestNode.Context == null) ? 0 : RequestNode.Context,
                                                                 IsActive = (RequestNode.IsActive == null) ? false : RequestNode.IsActive,
                                                                 MaxLimit = (objCC.MaxLimit == null) ? 0 : objCC.MaxLimit,
                                                                 MinLimit = (objCC.MinLimit == null) ? 0 : objCC.MinLimit
                                                             },
                                                             User = RequestNode.User,
                                                             POSTAL = new
                                                             {
                                                                 PID = (objSEDI.PID == null) ? "" : objSEDI.PID,
                                                                 Type = (Config.Type == null) ? "" : Config.Type,
                                                                 Amount = (Config.Amount == null) ? 0 : Config.Amount,
                                                                 IsValid = (Config.IsValid == null) ? false : Config.IsValid,
                                                                 Code = (Config.Code == null) ? "" : Config.Code,                                                                        
                                                                 Infos = new
                                                                 {
                                                                     Info = (from objRoot in XMLRoot
                                                                             where objRoot.ID == objCC.ID && objRoot.Channel == "Channel1" && objRoot.Group == "GROUP_1" && objRoot.Code == Config.Type.Substring(0, 3) && objRoot.PIDCode == Config.Type.Substring(3, 1)
                                                                             select new
                                                                             {
                                                                                 InfoFrom = (objRoot.InfoFrom == null) ? "" : objRoot.InfoFrom,
                                                                                 Selection = (objRoot.Handling == null) ? "" : objRoot.Selection,
                                                                                 Rate = (objRoot.Rate == null) ? "" : objRoot.Rate                                                                                         
                                                                             })
                                                                 },
                                                                 POSTALFee = from objSEDIFee in listSEDIFees
                                                                              where objSEDIFee.ID == objCC.ID && objSEDIFee.PID == objSEDI.PID
                                                                              select new
                                                                              {
                                                                                  BaseValue = (objSEDIFee.BaseValue == null) ? 0 : objSEDIFee.BaseValue,
                                                                                  UpdatedValue = (objSEDIFee.UpdatedValue == null) ? 0 : objSEDIFee.UpdatedValue,
                                                                                  BaseType = (objSEDIFee.BaseType == null) ? "" : objSEDIFee.BaseType,
                                                                                  UpdatedType = (objSEDIFee.UpdatedType == null) ? 0 : objSEDIFee.UpdatedType
                                                                              },
                                                                 OutputRoot = new
                                                                 {
                                                                     Output = from output in outputroot
                                                                             select new
                                                                             {
                                                                                 Type = 0,
                                                                                 SubType = 0,
                                                                                 OutputReference = 0                                                                                        
                                                                             }
                                                                 }
                                                             },

                                                         })
                               })
              };
    var streamRead = JsonConvert.SerializeObject(input, Newtonsoft.Json.Formatting.Indented);

Answer 1

这似乎只是在您的代码库中添加适当数量的翻译器的情况。是翻译很乏味，但是如果你觉得你在这里的一个查询中有太多的逻辑，我建议这可能是你去的方式。

这意味着你必须抛弃anon类型，或者拥抱动态关键字（甚至可以工作吗？！）

您可能还需要查看某种构建模式或中间状态。

在进一步检查时，您似乎在查询的中途引入了一些全局变量，例如XMLRoot＆amp; listSEDIFees。如果这更明确，可能会更好。您还可以通过使用where子句（XMLRoot）的静态部分预过滤objRoot.Channel == "Channel1" && objRoot.Group == "GROUP_1"来减少过多的工作，而不是每次都重新运行。也许像是

var channel1Group1Info = XMLRoot.Where(objRoot=>objRoot.Channel == "Channel1" && objRoot.Group == "GROUP_1").ToArray();

其余的我会一次挑选一件，减少了这个查询的工作量。

需要帮助减少用于构造JSON字符串的LINQ查询的Cyclomatic复杂性

1 个答案: