重构老大的代码，需要一点点信心

[ 619 查看 / 1 回复 ]

返回列表
发新话题 回复该主题
shenyoujia
头像

陈健

  • 管理员
  • 14848
  • 1423
  • 2009-11-07
2010-11-03 08:53 |只看楼主 1#
t T
老大写了方法交由偶调用，发现还是有可以 重构 的地方：
  1.         /// <summary>
  2.         /// 解析表达式的参数
  3.         /// </summary>
  4.         /// <param name="Express">表达式</param>
  5.         /// <returns>参数列表</returns>
  6.         public List<string> GetExpressParams(string Express)
  7.         {
  8.             char[] SplitFilter = new char[] { '(', ')', ',' };
  9.             string[] buffer = Express.Split(SplitFilter);
  10.             List<string> ParamsList = new List<string>();

  12.             for (int i = 1; i < buffer.Length; i++)
  13.             {
  14.                 if (!string.IsNullOrEmpty(buffer[i]))
  15.                 {
  16.                     ParamsList.Add(buffer[i]);
  17.                 }
  18.             }

  20.             return ParamsList;
  21.         }
复制代码
TOP
shenyoujia
头像

陈健

  • 管理员
  • 14848
  • 1423
  • 2009-11-07
2010-11-03 08:57 |只看楼主 2#
t T

  1. private static IEnumerable<string> GetExpressParams(string Express)
            {
                string[] stringArray = Express.Split('(', ')', ',');
                foreach (var item in stringArray)
                    if (String.IsNullOrEmpty(item))
                        yield return item;
            }
            private static List<string> GetSplitParams(string Express)
            {
                return new List<string>(CalcPackage.GetExpressParams(Express));
            }
复制代码
如果调用方可以接受 IEnumerable<string> 这个类型，则 更方便了
TOP
返回列表
上一主题 | 下一主题