软件测试 一个让我印象深刻的错误
2016-03-07 16:11
274 查看
由于C++语言的高效性与便利性,在进行一些算法设计的时候我总是会选择用C++。其中我遇到过的一个错误让我印象深刻。这是一个关于线段树和运算符优先级的故事。
基本的递归式的线段树看起来大致像这样:
当然,node结点中你会添加上你需要维护的变量,例如sum、max、min等各种稀奇古怪的东西,这里不做讨论。
但是,一个很明显的问题就是:结点的数量往往很多,动辄几百万,还是递归形式的。而改成非递归显得有点难。考虑到其中有大量的乘法运算,于是我写出了如下改进的代码:
用移位运算符来代替乘除法,貌似是个不错的选择。但是这组程序却无法通过测试用例。
通过跟踪建树过程,发现原因是移位运算符 << 的优先级要低于 +。
所以 build( i << 1 + 1, mid + 1, r ); 其实是 build( i << ( 1 + 1 ), mid + 1, r );
而我本意则是 build( ( i << 1 ) + 1, mid + 1, r );
原来加括号很重要!尤其是当你不知道优先级的时候。
但是当你知道了优先级以后,你可能会写出更加优雅、快速的代码:
build( i << 1 | 1, mid + 1, r );
原来,优先级很重要!
基本的递归式的线段树看起来大致像这样:
#include <iostream> #include <cstring> #include <cstdio> using namespace std; const int N = 1000001; struct Node { int l, r; } node[N * 4]; void build( int i, int l, int r ) { node[i].l = l, node[i].r = r; if ( l == r ) return ; int mid = ( l + r ) / 2; build( i * 2, l, mid ); build( i * 2 + 1, mid + 1, r ); } int main () { build( 1, 1, N - 1 ); return 0; }
当然,node结点中你会添加上你需要维护的变量,例如sum、max、min等各种稀奇古怪的东西,这里不做讨论。
但是,一个很明显的问题就是:结点的数量往往很多,动辄几百万,还是递归形式的。而改成非递归显得有点难。考虑到其中有大量的乘法运算,于是我写出了如下改进的代码:
#include <iostream> #include <cstring> #include <cstdio> using namespace std; const int N = 1000001; struct Node { int l, r; } node[N << 2]; void build( int i, int l, int r ) { node[i].l = l, node[i].r = r; if ( l == r ) return ; int mid = l + r >> 1; build( i << 1, l, mid ); build( i << 1 + 1, mid + 1, r ); } int main () { build( 1, 1, N - 1 ); return 0; }
用移位运算符来代替乘除法,貌似是个不错的选择。但是这组程序却无法通过测试用例。
通过跟踪建树过程,发现原因是移位运算符 << 的优先级要低于 +。
所以 build( i << 1 + 1, mid + 1, r ); 其实是 build( i << ( 1 + 1 ), mid + 1, r );
而我本意则是 build( ( i << 1 ) + 1, mid + 1, r );
原来加括号很重要!尤其是当你不知道优先级的时候。
但是当你知道了优先级以后,你可能会写出更加优雅、快速的代码:
build( i << 1 | 1, mid + 1, r );
原来,优先级很重要!
相关文章推荐
- c#---传址
- NETBIOS名 和 Host名的不同
- H264三种码率控制方法(CBR, VBR, CVBR)
- 2016.3.7 16.14复习篇之三(蓝桥杯恶补)
- MyEclipse如何从svn上下载项目
- react:undefined is not a function(this.State({flag:true,}))
- 2016数据结构试验1.2
- 用U盘制作启动盘后空间变小的恢复方法
- Android安全输入设计与思考
- 10款无限滚动自动翻页jquery插件
- 【百度地图API】搜索建议,如何使用suggestion--下拉列表方式的搜索建议
- 用nginx指定访问tomcat日志
- Apk去签名校验详解
- Cpp--处理字符串的自定义类
- 不使用递归和栈遍历二叉树
- uva10817
- process launch failed: Security 解决方案
- 关于fopen函数的总结
- 使用cout输出控制小数位的方法
- 在标准输入流与输出流中使用控制符笔记