【Java】为什么 StackOverflow 上的代码片段会摧毁你的项目？

Z时代
2024-01-10
分类：技术分享

昨天公司里碰到一件令人哑然失笑的事情。帮朋友公司做的一个项目，做SIT测试的时候发现一些bug，仔细查了下原因，原来是因为当初觉得这个项目比较简单，交给了几个新入职的新丁，也算是给他们练练手，结果其中一位写了一段代码出现了问题结果导致bug的出现。

虽然问题不大很快就修复了，但是正所谓初生牛犊不怕虎，这位出问题的新丁不太服气，一直嚷嚷着他这段代码是从CSDN上找到的，别人都说好用他才用的。问了问他是否理解原文中那么写的原因，结果才支支吾吾了半天讲不清楚，最后也承认他只是看到那段代码的结果是他要的，并不是理解中间一些方法的运用到底是什么逻辑。

真是又好气又好笑，想想现在的年轻人真是有个性，自己当年遇到这种情况只敢虚心求教，哪敢都没明白就去用一段代码，别人指出问题还忿忿不平。

也许是自己老了吧。晚上和一个刚从国外归国的同学说起这个事情，他倒是深有体会，说到他们在国外留学时，遇到技术问题想到网上咨询用的最多的是StackOverflow,相比起国内的CSDN，他觉得StackOverflow上提问题的人和回答的人都更加严谨，不会有太多的废话和问题之外的讨论。按他的理解，CSDN上很多国内的技术新手纯粹是缺乏独立思考的能力，都只是来求代码完成目标，不去探究为什么这么写，本身CSDN上问题回答质量不高，这样很容易出事。

同学的看法，一方面可能是因为国内程序员需求极大，但整体浮躁、功利性的大环境导致很多培训机构没有培训到位，很多程序员都是一边开始工作一边学习；但另一方面，不排除他崇洋媚外的自满情绪。于是，我便去查阅了下StackOverflow上面的内容和相关资料。

【Java】为什么 StackOverflow 上的代码片段会摧毁你的项目？

仔细研究后发现，其实StackOverflow没有同学说的那么好，即使是该网站上的回答，也有很多有问题的代码。看来这还是一个世界性通用的难题~

对于国外的程序员来说，StackOverflow就是他们的CSDN，是他们遇到各种问题查询的第一选择，很多国外的程序员，会到StackOverflow上需求帮助，找到他们需要的解决问题的代码，然后不假思索的直接复制使用，和我公司出问题的那位年轻人一样，这个习惯很不好。

代码复制本身来说并不是什么坏事，我们学开发的时候，肯定老师都教过我们代码复用能有效提高软件的开发效率。已经被解决了的问题，实在没有必要再去重新写一遍代码。但是这个前提是，使用者，也就是开发人员，必须知道你要用的这段代码的来龙去脉真正含义，就是你要真正明白你用的代码。

简单举个例子：

几年前，很多程序员发现，在windows上使用Docker老是会碰到无法启动的问题，原因一直不明，很多人都是重装了系统什么的才解决。后来有一个大神发文解释原因后我们才知道了为什么

为什么会有这种情况呢？

因为有一段代码中，返回的 GUID 类型是 System.Reflection.RuntimeAssembly（系统层面的运行时程序集），而不是 Windows 中对应 Docker 程序集中定义的类型。Docker处理的逻辑就是在同一时间只允许运行一个实例，判断的方式就是通过判断 GUID 是否存在，但在获取GUID 时，用的是系统层面的而非自己独立程序集里面，从而导致了这个问题。问题代码如下：

var name = string.Format("Global\{0}", (object)Assembly.GetExecutingAssembly().GetType().GUID);

当然了，如果系统只有一个应用程序使用了上面的错误代码，那么这个问题还是不会出现，因为只有一个实例。但实际情况是很多应用程序都使用了这一段错误代码，结果就不允许两个实例同时运行。

那这些应用程序，想必你已经猜到，都用了这段代码，而且都是从StackOverflow上下载后不假思索的使用导致的。

Andreas Lundblad，国外一位大神，Palantir 的 Java 开发人员，同时也是 StackOverflow 上排名最高的贡献者之一。

他有一段代码，作用是将字节计数转换为更易于阅读的格式。打个比方，1024字节转换为1kB，1048576 字节转换为1MB。他最初在2010年将这一段代码放到了StackOverflow上，根据官方统计，他的这段代码是StackOverflow上被拷贝次数最多的Java代码，GitHub上使用这段代码的项目有6千多个。

当Andreas Lundblad知道此事后，出于负责，很小心谨慎的重新检查了下这段代码，随后发现这段代码里有一些问题，于是他马上更新了版本，并发博告诉大家：