来源:邹琼俊 - 博客园
链接:http://www.cnblogs.com/jiekzou/p/5294492.html
为什么今天突然想说这个话题,那是因为在工作当中,被队友坑过。同样的一个功能,你让不同的人去实现,可能会有不同的实现方式,同样,就会得到不一样的结果。有些人做出来性能好,代码精简,可读性好,易于维护等,有些人写的代码不独代码凌乱堪,更能让服务器崩溃。程序员的基础和思维很重要,解决问题的思维,要多思考,而不能只考虑完成功能,其它就啥都不管了,说得严重点,给人条活路好么?人都是爹妈生的!
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,如果 成绩<>
(考虑用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//<>
{
if (score >= 80)
{
Console.WriteLine('B');
}
else//<>
{
if (score >= 70)
{
Console.WriteLine('C');
}
else//<>
{
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 <>
{
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,项目中就有人是这样干的......
联系客服