你这是deserialization,基本很难优化。
想减少代码，别用XML，用protobuf……
【 在 coreman (周文兴) 的大作中提到: 】
: 标  题: 请问如何优化一下这段代码
: 发信站: 水木社区 (Thu Aug 27 23:43:48 2020), 站内
:
: 这是一段从XML文件中提取内容填充到configObject类成员变量的代码，大概有近100个else if比较分支
: 请问如何优化一下，减少代码量
:
: using (XmlReader xr = XmlReader.Create(configXMLReader, settings))
: {
:     Log.Debug("OpHostCIDConfigure.xml contents ");
:     while (xr.Read())
:     {
:         if (xr.MoveToContent() != XmlNodeType.Element)
:             continue;
:
:         if (String.Equals(xr.Name, "agency"))
:         {
:             configObject.Agency = long.Parse(xr.ReadString());
:             Log.Debug("Agency Content {0} ", configObject.Agency);
:         }
: ###
:   <中间省略大量else if比较分支>
: ###  
:  
:         else if (String.Equals(xr.Name, "csc-parameters"))
:         {
:             tmpLong = long.Parse(xr.ReadString());
:             lCscParamList.Add(tmpLong);
:             Log.Debug("CscParameters {0}", tmpLong);
:         }
:         else if (String.Equals(xr.Name, "cid-parameters"))
:         {
:             tmpLong = long.Parse(xr.ReadString());
:             lCidParamList.Add(tmpLong);
:             Log.Debug("CidParameters {0}", tmpLong);
:         }
:     }
: }
: --
:
: ※ 来源:·水木社区 http://www.newsmth.net·[FROM: 14.203.253.*]
--
FROM 76.126.252.*

代码量并没有减少啊？
你还是一堆函数，每个都要实现。
然后这个表建起来的过程也是一大堆代码。

要想减少代码量，必须用反射。如果XML跟数据对象的对应不够严谨就做不了。

推荐用protobuf是因为这套对应关系已经是成熟的了。
【 在 txgx (纸上谈医) 的大作中提到: 】
: 标  题: Re:请问如何优化一下这段代码
: 发信站: 水木社区 (Fri Aug 28 08:41:24 2020), 站内
:
: 可以作成函数指针表格。(string, fn)查表吧。
: 【 在 here080 的大作中提到: 】
: :  
: : 你这是deserialization,基本很难优化。  
: : 想减少代码，别用XML，用protobuf……
:
: 发自「今日水木 on HUAWEI G750-T01」
: --
: 愚昧无知如孔子，乱离神怪不敢谈，
: 更可笑者是世人，不知就说不存在。
:
:
: ※ 来源:·水木社区 http://www.newsmth.net·[FROM: 42.234.92.*]
--
FROM 76.126.252.*

抛砖引玉下：把字符串转为int（比如用map），然后用switch代替循环

性能应该会更好些

【 在 coreman (周文兴) 的大作中提到: 】
: 这是一段从XML文件中提取内容填充到configObject类成员变量的代码，大概有近100个else if比较分支
: 请问如何优化一下，减少代码量
: using (XmlReader xr = XmlReader.Create(configXMLReader, settings))
: ...................
--
FROM 211.86.151.*

边读取边解析边校验不是好习惯
【 在 coreman 的大作中提到: 】
: 这是一段从XML文件中提取内容填充到configObject类成员变量的代码，大概有近100个else if比较分支
: 请问如何优化一下，减少代码量
: using (XmlReader xr = XmlReader.Create(configXMLReader, settings))
: ...................
--
FROM 223.104.3.*

【 在 coreman 的大作中提到: 】
: 这是一段从XML文件中提取内容填充到configObject类成员变量的代码，大概有近100个else if比较分支
: 请问如何优化一下，减少代码量
: using (XmlReader xr = XmlReader.Create(configXMLReader, settings))
: ...................
做一个模板，把100个名字，类型，存储位置罗列出来。
一个for name in template 循环就解决掉。

JAVA就好弄一点，通过反射，匹配类成员名字即可。
--
修改:ylh1969 FROM 221.221.53.*
FROM 221.221.53.*