码迷,mamicode.com
首页 > 其他好文 > 详细

程序员,做艺术家不做码农

时间:2014-06-22 21:22:31      阅读:184      评论:0      收藏:0      [点我收藏+]

标签:c++   重构   

前几天和一个朋友聊天时说:我有强迫症,每次看到不符合自己审美的代码时,总想重构一下。朋友的观点与大多数人相仿,程序只要满足要求,运行正确就可以。
在现实的工作中一样,每当看到一段优美且高效的代码,我总是喜欢多看几遍,满心的享受。但是我们产品中的太多代码让我这个代码洁癖“患者”看了非常的烦躁:
1. 随便一个源文件都是上百行的函数,有些甚至超过500行(含空行和注释)。
2. 大量没有必要的临时变量定义。
3. 习惯性的把所有变量声明定义放到函数前面,一些大函数甚至超过20行都是函数局部变量的声明和定义。想想看,在第一个异常检查之后程序返回,而前面定义的十几个变量变得毫无价值。
4. 功能相仿的重复代码。

诸如此类的代码问题,给程序维护带来无尽的风险,而大多数技术管理者却认为代码重构根本毫无必要,甚至很多人认为只要设计文档写好,代码符合文档要求,顺利通过公司的各种质量评审,这段代码就最好不要动。要命的是,很多大型软件,很多团队共同开发,团队A要在团队B开发的代码中修改,为了省事,根本不考虑重构,甚至函数都不愿意增加一个,任由代码行数累加。
哎,如果我是一个技术管理者,我希望可以把敏捷中对于代码质量的要求贯彻和执行。程序开发者应该吧自己看做是艺术家,而不是一个累代码的民工。

为了说明这个问题,我用一段代码重构结束这篇牢骚。为了不至于引起误会,我采用的是www.cplusplus.com的一段小程序,没有对原作者不敬的意思,相反,我非常尊敬这些给开源做出重大贡献的朋友们。


原代码:

/* va_arg example */
#include <stdio.h> /* printf */
#include <stdarg.h> /* va_list, va_start, va_arg, va_end */
int FindMax (int n, ...)
{
  int i,val,largest;
  va_list vl;
  va_start(vl,n);
  largest=va_arg(vl,int);
  for (i=1;i<n;i++)
  {
    val=va_arg(vl,int);
    largest=(largest>val)?largest:val;
  }
  va_end(vl);
  return largest;
}
int main ()
{
  int m;
  m= FindMax (7,702,422,631,834,892,104,772);
  printf ("The largest value is: %d\n",m);
  return 0;
}


重构后的代码:

/* va_arg example */
#include <stdio.h> /* printf */
#include <stdarg.h> /* va_list, va_start, va_arg, va_end */
#include <stdlib.h>
int getTheLargest(va_list vl, int n, int* pVarray)
{
    int val;
    int largest =0;
   
    for (int i=0;i<n;i++)
    {
        val=va_arg(vl,int);
        largest=(largest>val)?largest:val;
        pVarray[i] = val;
    }
    va_end(vl);
    return largest;
}
int findMax (char a, int n, ...)
{
    int* pVarray = (int*)malloc(sizeof(int)*n);
    va_list vl;
    va_start(vl,n);
 
    return getTheLargest(vl, n, pVarray);
}
int main()
{
    printf ("The largest value is: %d\n", FindMax (‘a‘, 2, 702,822,631,834,892,104,772));
 
    system("PAUSE");
}



控制台输出:
The largest value is: 822
Press any key to continue . . .


PS:
最近两个月一直没有更新博客,一方面工作确实太忙,另一方面花了很多时间在做个人的微信工作号:瑜伽分享汇。希望喜欢瑜伽的朋友们,可以关注。bubuko.com,布布扣

bubuko.com,布布扣

bubuko.com,布布扣

bubuko.com,布布扣bubuko.com,布布扣bubuko.com,布布扣



程序员,做艺术家不做码农,布布扣,bubuko.com

程序员,做艺术家不做码农

标签:c++   重构   

原文地址:http://blog.csdn.net/walle_love_eva/article/details/32345845

(0)
(0)
   
举报
评论 一句话评论(0
登录后才能评论!
© 2014 mamicode.com 版权所有  联系我们:gaon5@hotmail.com
迷上了代码!