首页 > 嗟来之食 > 程序员的基础和解决问题的思维很重要 – 邹琼俊 –
2016
07-08

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

为什么今天突然想说这个话题,那是因为在工作当中,被队友坑过。同样的一个功能,你让不同的人去实现,可能会有不同的实现方式,同样,就会得到不一样的结果。有些人做出来性能好,代码精简,可读性好,易于维护等,有些人写的代码不独代码凌乱堪,更能让服务器崩溃。程序员的基础和思维很重要,解决问题的思维,要多思考,而不能只考虑完成功能,其它就啥都不管了,说得严重点,给人条活路好么?人都是爹妈生的!
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;注意优先顺序,从左至右,一元运算符高于二元运算符答案是:827一眼看上去,精简是精简了,可是可读性呢?这是给人看的么?如果你口算不出来,请复习下这一块的基础知识,补一补。

算运运算符 ++ —
++;分为前++和后++,不管是前++还是后++,最终的结果都是这个变量的值自身加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删除,终于只刷新中间部分了,唉!记得当时美工给我的(我当时做财务系统http://www.cnblogs.com/jiekzou/p/4508392.html)和给这个项目开发者的是同一个界面,可做出来后就是天壤之别了….
省略一万字,这些都是最最基本的东西,为什么一些工作几年了的程序员会犯这样的错误,估计是懒得思考吧,程序员是要学会偷懒,但是要勤于思考…..
提示:细心的朋友,会发现本文if的实现方式有bug,少了return,项目中就有人是这样干的……

最后编辑:
作者:
这个作者貌似有点懒,什么都没有留下。

留下一个回复