- 主题:重构了屎山代码
看过一个20000行的c函数。
关键是 那是80年代的代码，
我不知道当时的人是怎么做到十几层的括号对齐的
【 在 iwantfly 的大作中提到: 】
: 最近完成了重构某位二十年工作经验人员写的代码
: 其特点是特别善于使用stl,善于使用各种先进特性
: 主要优点有：
: ...................
--
FROM 115.183.224.*
vim
哈哈
【 在 liangyue 的大作中提到: 】
: 看过一个20000行的c函数。
: 关键是 那是80年代的代码，
: 我不知道当时的人是怎么做到十几层的括号对齐的
: ...................
--
FROM 101.86.156.*
我倒觉得这个代码还可以。非常清晰地说明这个代码里面是解析 xml 的代码，并且回复。
无论是 Qt 还是 JAVA，碰到解析 xml 都特别麻烦。除非使用 python 才简单一些，几个 query 就搞定了。业务代码就是烦。
【 在 iwantfly 的大作中提到: 】
: bool CUSBLProcess::ReplyTransponderState(QByteArray byUSBLData)
: {
: bool bParseOK = false;
: ...................
--
FROM 59.60.25.*
这代码的条件语句纯属闲的蛋疼
判断条件不符合，直接退出就减少了嵌套
【 在 iwantfly 的大作中提到: 】
: bool CUSBLProcess::ReplyTransponderState(QByteArray byUSBLData)
: {
: bool bParseOK = false;
: ...................
--
FROM 221.218.215.*
其实他没干啥呀，就是用了move和lambda，还好吧
大部分都是xml的常规代码
【 在 iwantfly 的大作中提到: 】
: bool CUSBLProcess::ReplyTransponderState(QByteArray byUSBLData)
: {
: bool bParseOK = false;
: ...................
--
FROM 123.114.94.*
在这里你看不出玄妙
拷贝到文本编辑器里不换行才知道酸爽
【 在 Zinux 的大作中提到: 】
: 其实他没干啥呀，就是用了move和lambda，还好吧
: 大部分都是xml的常规代码
--
FROM 111.199.185.*
别的不说，static 效率稍微高那么一丢丢吧？
【 在 z16166 的大作中提到: 】
: 说不定每个人看别人的代码都是有点shi的
:
: 我觉得有点shi的，现在能忍就忍，没工夫去理
:
: 之前是发现有个类，被搞成单例，但是每个成员都是static，没忍住，把static全都去掉了。这纯粹是c++还没入门的写的，但是没人干活，还得顶上
:
: --
发自「今日水木 on M2007J17C」
※ 来源:·水木社区
http://www.mysmth.net·[FROM: 124.127.77.*]
FROM 124.127.77.*
这是重构后的代码
QDomElement node = doc.documentElement()
.firstChildElement("SynchStatus")
.firstChildElement("Job")
.firstChildElement("Objects");
bParseOK = !node.isNull();
【 在 hgoldfish 的大作中提到: 】
: 我倒觉得这个代码还可以。非常清晰地说明这个代码里面是解析 xml 的代码，并且回复。
: 无论是 Qt 还是 JAVA，碰到解析 xml 都特别麻烦。除非使用 python 才简单一些，几个 query 就搞定了。业务代码就是烦。
--
FROM 111.199.185.*
static成员是crt init时构造，单例一般是搞成第一次引用时构造
【 在 GoGoRoger 的大作中提到: 】
: 别的不说，static 效率稍微高那么一丢丢吧？
发自「今日水木 on M2007J17C」
--
FROM 114.240.244.*
看错了，还以为是函数方法
【 在 z16166 的大作中提到: 】
: static成员是crt init时构造，单例一般是搞成第一次引用时构造
:
: 发自「今日水木 on M2007J17C」
: --
发自「今日水木 on M2007J17C」
※ 来源:·水木社区
http://www.mysmth.net·[FROM: 124.127.77.*]
FROM 124.127.77.*