登录
首页精彩阅读程序员的基础和解决问题的思维很重要
程序员的基础和解决问题的思维很重要
2016-04-08
收藏

程序员的基础和解决问题的思维很重要

为什么今天突然想说这个话题,那是因为在工作当中,被队友坑过。同样的一个功能,你让不同的人去实现,可能会有不同的实现方式,同样,就会得到不一样的结果。有些人做出来性能好,代码精简,可读性好,易于维护等,有些人写的代码不独代码凌乱堪,更能让服务器崩溃。程序员的基础和思维很重要,解决问题的思维,要多思考,而不能只考虑完成功能,其它就啥都不管了,说得严重点,给人条活路好么?人都是爹妈生的!

备注:本文采用C#编程语言

1、我先列一个最最简单的基础编程题,本来很简单,至于为什么写成了这样,不要问我,问写的那位大神吧,我完全相信我现在的公司中有人看到这样的代码(虽然工作几年了...)会给我一个错误的答案

int a = 5;

int b = a++ + (++a) * 2 + ++a;

Console.WriteLine(a);

Console.WriteLine(b);

解:5+ a=a+1  a=a+1 a=7  a=a+1=5 +7*2+8;

注意优先顺序,从左至右,一元运算符高于二元运算符

答案是:

8

27


一眼看上去,精简是精简了,可是可读性呢?这是给人看的么?如果你口算不出来,请复习下这一块的基础知识,补一补。

算运运算符  ++ --

++;分为前++和后++,不管是前++还是后++,最终的结果都是这个变量的值自身加1.

区别:如果在一个表达式中遇到了++运算符,

如果是前++,则首先这个变量的值自身加一,然后拿着这个加一后的值去参与运算。

如果是后++,则首先拿原值参与运算,运算完成后,自身再加一。

这里只是举个例子,现实中看到了类似的代码,我呢个去,我费了好大劲........

我用reflector工具查看了下,解析如下:

计算机会理解成如下:

int num = 5;

int num2 = (num++ + (++num * 2)) + ++num;

也就是说:

++在后,也就是说num依旧为5,等+运算后再执行++

num=num+1;//6

num=num+1; //7

两次++,num=7

5+7*2

num=num+1;//8

num2=5+7*2+8

IL语言如下:

.method private hidebysig static void Main(string[] args) cil managed

{

    .entrypoint

    .maxstack 3

    .locals init (

        [0] int32 num,

        [1] int32 num2)

    L_0000: nop 

    L_0001: ldc.i4.5 

    L_0002: stloc.0 

    L_0003: ldloc.0 

    L_0004: dup 

    L_0005: ldc.i4.1 

    L_0006: add 

    L_0007: stloc.0 

    L_0008: ldloc.0 

    L_0009: ldc.i4.1 

    L_000a: add 

    L_000b: dup 

    L_000c: stloc.0 

    L_000d: ldc.i4.2 

    L_000e: mul 

    L_000f: add 

    L_0010: ldloc.0 

    L_0011: ldc.i4.1 

    L_0012: add 

    L_0013: dup 

    L_0014: stloc.0 

    L_0015: add 

    L_0016: stloc.1 

    L_0017: ldloc.0 

    L_0018: call void [mscorlib]System.Console::WriteLine(int32)

    L_001d: nop 

    L_001e: ldloc.1 

    L_001f: call void [mscorlib]System.Console::WriteLine(int32)

    L_0024: nop 

    L_0025: ret 

}

IL语言看得够明了了吧!这是要考验我的理解能力和基础能力还是要显示得你牛逼呢?所以啊,还是要自我修炼,才能无论别人怎么坑我,我都能有办法应付哇

2、举例,控制台输入一个成绩,如果 成绩>=90 :输出A,如果90>成绩>=80 输出B,如果80>成绩>=70 输出C ,如果70>成绩>=60 输出D,如果 成绩<60输出E

(考虑用if好还是用if-else好还是if else if好,思考,为什么好?)

以下是我在项目中看见的三种写法:

代码如下:

            Console.WriteLine("请输入你的考试成绩");

            int score = Convert.ToInt32(Console.ReadLine());

if else 的做法

            #region if -else的做法

            if (score >= 90)

            {

                Console.WriteLine("A");

            }

            else//<90

            {

                if (score >= 80)

                {

                    Console.WriteLine("B");

                }

                else//<80

                {

                    if (score >= 70)

                    {

                        Console.WriteLine("C");

                    }

                    else//<70

                    {

                        if (score >= 60)

                        {

                            Console.WriteLine("D");

                        }

                        else

                        {

                            Console.WriteLine("E");

                        }

                    }

                }

            } 

            #endregion

if的做法

  #region if的做法

            if (score >= 90)

            {

                Console.WriteLine("A");

            }

            if (score >= 80)

            {

                Console.WriteLine("B");

            }

            if (score >= 70)

            {

                Console.WriteLine("C");

            }

            if (score >= 60)

            {

                Console.WriteLine("D");

            }

            if (score < 60)

            {

                Console.WriteLine("E");

            } 

            #endregion

if else if的做法

   #region if else if的做法

            if (score >= 90)

            {

                Console.WriteLine("A");

            }

            else if (score >= 80)

            {

                Console.WriteLine("B");

            }

            else if (score >= 70)

            {

                Console.WriteLine("C");

            }

            else if (score >= 60)

            {

                Console.WriteLine("D");

            } 

            #endregion

请比对下这三种实现方式有什么不同,哪一种好,为什么?发现什么问题了么?我在工作中,经常会看到一些程序员乱用if else,明明可以有更好的实现方式。

分支结构:if结构  if-else结构

选择结构:if else-if

swich case的做法(这里只考虑100分制)

 public static string GetLevel(int score)

        {

            string level = "";

            switch (score / 10)

            {

                case 10:

                case 9: level = "A";

                    break;

                case 8: level = "B";

                    break;

                case 7: level = "C";

                    break;

                case 6: level = "D";

                    break;

                default: level = "E";

                    break;

            }

            return level;

        }

3、使用一个同步程序或者定时任务计划来往Ftp服务器上面拋送订单报文,然后获取Ftp服务器上面的回执报文

公司程序是这样写的:

1、从数据库中查询需要拋送的订单

2、在服务器上面生成订单报文并存在一个文件夹下面

3、然后使用Ftp上传这些订单报文,已经上传成功的就修改这个报文名称(后面+_1)

4、上传完报文后,然后根据这些报文名称就去Ftp服务器上面下载回执报文,又存到一个文件夹里面

5、再读取这个文件夹里面的回执文件,已经读取过的,就把回执文件改名

......

我真心说不下去了,第一次看到的时候,我就觉得写这代码的人应该抓去牢房里关着,不要放出来

结果程序跑了一个月,服务器奔溃了,我上服务器上面删报文文件删除了2个小时,几十个G,然后看下程序代码,一千万匹草泥马奔腾而过,然后安排人重写代码...

我到现在都想不明白,程序为什么要这样写?即便你要生成订单报文,你按年、月、日生成报文文件,处理完成之后,把报文文件移动到特定目录下面,总比你改名快吧,然后定期把一个月前的交易记录删除(不要就直接删除)或者压缩包(要的话就压缩备份),怎么就不想一下一个月,这个文件夹下面会有几十万个文件呢,这是典型要干死服务器的节奏啊。至于根据文件名去下载Ftp上面的回执文件,也不需要再从这个文件夹里面遍历出所有_1的文件名,再去下载,这多慢啊,何不记录在redis里面或者memory存储类型的mysql数据表中呢?还有生成交易报文,上传交易报文,取交易报文回执都是在同一个线程里面的,为什么不使用多个人定时任务计划,多个进程呢?我一直想不明白.........

不过海关的技术人人员也真是奇葩,订单报文居然用XML,还一条记录一个XML文件,也不让压缩上传再解压,回执目录里面还存放几家企业的回执报文.........只能说国企真心有钱,根本不用担心网络流量和我们这边抛单的性能,卡死我们这些傻不拉唧的电商企业,找一点存在感。

我以前在的一家公司是做城市一卡通的,现在深圳通啊XX通啊基本上都是那公司做的,很老了,但是做的程序很稳定,每天上传交易报文,一个客户端的报文是一天一个文件,里面的记录是16进制,压缩上传,再解压,分目录存储、分目录备份,为什么?不阔气啊,省钱啊,公交车的交易报文是用gprs无线传输的,如果按照海关这样搞,不被搞死?也用过Ftp传输,交易量那么大,没卡死过,很稳定,因为我发现他们解决交易报文的方式不错,而不是用了啥新技术啊什么的,现在的公司刚成立2年,一天几千单订单报文,居然把自己的Web服务器搞死了,哪里的移植?从网上copy了正确的代码但是用在了错误的解决问题的思维上面。海关报文弄XML,还一个文件一个,我呢个去,用FTP上传,那么多电商企业往你海关FTP服务器上面传,你还不让人家打包,作死了.....海关把自己服务器作死了,我们不能把自己的服务器也作死啊!

4、foreach或者for循环中,N次更新、插入、删除数据库操作

我很好奇,为什么在循环里面要进行一次数据库连接、然后执行数据库操作,然后关闭连接,为什么不把需要进行数据库操作的内容保存在一个集合里面,最后一次性去执行Sql操作,让循环里面只进行一次打开、执行、关闭操作呢?人家ORM框架里面都有了批处理操作....

我在项目中看见很多地方,批量导入,批量更新回执状态的时候,经常看见这样的C#代码,抓狂,想把写代码的人拉去吊着打...........

5、系统后台,是一个完整的界面,也就是说,无论你点击啥,整个界面刷新,对就是整个界面刷新,刷新还要每次从session里面读出html把菜单重新加载一遍,我第一次看到,眼睛就被刷瞎了.......然后左侧菜单栏,为了保持记录选中状态,然后各种js+cookies就出来了,几百行js穿插其中,我艹,当时想重构的心都没有,就想重写(可是谁给我时间?烂摊子一个...),还有那奇葩的资源角色权限管理,居然是半自动的,市面上面早就是全自动了,什么是半自动,就是后台界面配置了,代码里面还要去修改,而且还是硬编码,我真的是长见识了,随便从网上乱下载一个源码也不至于这样啊!至于为什么能做出这样的项目,我不想说话。最后客户终于忍受不了,要求改,我就安排一个同事改了,用最简单的方式弄个frameset,然后把那几百行户js删除,终于只刷新中间部分了,唉!记得当时美工给我的(我当时做财务系统和给这个项目开发者的是同一个界面,可做出来后就是天壤之别了....

省略一万字,这些都是最最基本的东西,为什么一些工作几年了的程序员会犯这样的错误,估计是懒得思考吧,程序员是要学会偷懒,但是要勤于思考.....

提示:细心的朋友,会发现本文if的实现方式有bug,少了return,项目中就有人是这样干的......


数据分析咨询请扫描二维码

客服在线
立即咨询