对于程序员来说，上手一门计算机语言是很轻松的一件事情，但是能把这门计算机语言运用的炉火纯青就需要下一番功夫了，比如多读优秀代码，从中了解一些比较好的代码设计思想，多琢磨代码的写法并多去实践可以让自己的代码更高效，更简练，这中间有很多很多的道理，只有在实践中不断的去悟，才能悟出其中的奥妙所在，因此没事多与同行切磋切磋一些代码问题也是比较有益的，但是保持友好的争论，动了情绪就不是很好了，近期一名码农朋友就有一个写法问题与他的同事争了起来。
据这名码农朋友说，他的同事是一名有3年工作经验的程序员了，比他的资历会老一点，今天他吐糟了这名同事if判断赋值的写法问题，可是他这名同事反过来喷他不懂代码，然后这名码农朋友更是不服气了，虽然嘴上没再说什么，但是这名网友回头就把他同事的这段代码晒到网上让大家评理，并且还说了句：“大家看看这是几年程序员写出来的？”，哎，这名码农这么做也不太地道，如果让他的同事看到了多不好啊，他晒出的代码如下图，针对目前的这些情况，让我们一起看看大家都是怎么认为的吧！
网友一：看到vue里面的$我就不想看了，什么乱七八糟的代码和架构，写代码的人绝对不超过10k。 正常人的洁癖在vue里宁愿写死原生也不会再引入jquery那么大的dom库。
上世是朵花：这波吐糟我个人并不赞同，怎么不超过10k都扯出来了。
网友二：if判断，常量应该放在==前面
上世是朵花：没错，这是一个比较好的习惯，为了避免误把==写成=的错误，通常把常量放前面，因为如果写错，编译就会报错，也算是一个提示。
网友三：vue啊。。。我也来喷，es6是基本操作啊。不用路由控制跳转么？好吧，喷完发现，你这个是script方式引入的。告辞。。。
上世是朵花：这名网友逗了，来一个自问自答式的吐糟，然后又否定了自己。
网友四：为啥作为后端来讲，不觉得这段代码有问题。我觉得很清晰的表达了意思。
上世是朵花：每个人的挑剔程度不一样，只能这么说吧，所以有时候review代码的时候，一些朋友总会以为别人提出的问题总是故意挑刺的。
网友五：parseInt 只用一个参数，这个面试就算实习生都不给过
上世是朵花：不应该吧，这么说有点严重了。
网友六：后端，这代码虽然if可以减少嵌套，也没有扩展性，整体来说还算干净
上世是朵花：这名网友表明自己是后端开发的身份，在他看来代码整体还是可以的。
网友七：逻辑算清晰，但是细节很差劲，像是不太熟悉js的人写的
上世是朵花：谁的代码晒出来，也都能被大家造出毛病。
网友八：表示没见到有啥明显不妥之处，有的话估计也就是一些仁者见仁智者见智的问题
上世是朵花：比较认同这名网友的观点
网友九：看不懂部分喷楼主秀优越的，都是打工干活的，代码也没硬伤，有必要恶语相向？都什么玩意儿
上世是朵花：这名网友应该没有好好审题，楼主晒的并不是他自己的代码。
其实，针对这个事情，我个人是这么看待的，楼主晒出别人的代码这个行为的确不是很妥，其实关于评论代码，评论代码的具体细节还是可以的，比如具体判断条件，常量在前，这类问题还是可以讨论的，如果根据一段代码就说人家是新手或者说月薪不超过多少多少的话，这么做就比较不够友好了，其实站在技术的角度来讲，写法严谨一点当然是没错了，本例中的是前端代码，个人认为对于前端代码更需要注意的是各个浏览器，各个设备客户端之间的兼容性问题，只有这个能得到很好的解决才是比较靠谱的代码，前端代码的好坏直接影响到用户的体验，特别是界面的流畅度，相比于前端代码，后端代码一般涉及到兼容性的地方相对少一点，后端代码的侧重点主要放在安全性（是否Hold所有的情况，是否有漏洞），性能（是否耗内存，是否耗cpu,是否耗资源），是否有并发瓶颈，会不会死锁，会不会内存泄露等。如果只是看代码写的够不够个性，够不够花样的话，那只是程序员之间的自娱自乐，并没有多少必要，代码只是一个工具，代码的终极作用是为生活服务，为人民服务，说代码是艺术品的话，你见过那个艺术品可以随便复制了，如果非要说艺术，我个人感觉代码中所蕴含的一些思想倒是可以称得上是艺术，这种思想是灵魂深处的，那是不能被复制的。
以上所有图片均来之互联网
大家好，我是“上世是朵花”。如果你有什么好的看法或者观点可以在评论区展现你的才华，互动交流，如果想进一步了解我，那就关注我吧！(微信公众号：sssdhua )
扫一扫
538
被折叠的 条评论
为什么被折叠?
到【灌水乐园】发言
