第一，避免无故的函数调用和重复计算，看下面的代码：
for(int i=0;i<strlen(s);i++) { cout<<s[i]<<endl; } int l=strlen(s); for(int j=0;j<l;j++) { cout<<s[j]<<endl; } int k=0; while(s[k]!=‘\0‘) { cout<<s[k]<<endl; k++; }
由于字符串s的长度是固定的，对于第一个循环，每次循环都调用strlen计算一遍它的长度，strlen大概与字符串长度相关，那么这样写的后果就是循环的时间效率与字符串长度的平方相关！也就意味着看着单循环实际是双循环的尴尬境地。毫无疑问，这种写法不可以让人接受！
对于第二个循环，把计算长度的代码提出来赋予一个临时变量，那么对于字符串的长度就只是计算了一遍而已，相比于上面的循环快了不止一星半点，事实证明效率至少提高了百分之六七十！
对于第三个循环，这是进一步优化，由于字符串都是以‘\0‘结尾，所以没有必要计算长度，但是这一步的优化在效率上并没有显著提高（相比于第二个）。
总结成一句话就是，绝不写重复计算的代码，绝不做无谓的重复的函数调用。
第二，避免无味的存储器引用，接着看代码：
bool arraysum1(int a[],int l,int *ans) { if(l==0) return false; for(int i=0;i<l;i++) { *ans += a[i]; } return true; } bool arraysum2(int a[],int l,int *ans) { if(l==0) return 0; int tempsum = 0; for(int i=0;i<l;i++) { tempsum += a[i]; } *ans = tempsum; return true; }
指针会导致频繁的存储器访问，因此arraysum1的时间效率要比arraysum2慢两到三个数量级！频繁的存储器访问非常的占用cpu时间，绝对是不好的习惯。
原文：http://www.cnblogs.com/liboyan/p/4985092.html