看着项目的代码，感觉要黄，准备跑路了

[george404]

小弟在一个创业型小公司 10 个人搞 golang 项目。最近模块连调的时候发现一个 bug 追到另一个人写的代码里去了，直接崩溃了。

1.那个人一共撸 4 万行代码（包括注释换行），其中 3 万行在一个文件夹下，一个文件夹下直接 30 个.go 源文件。。。。没有任何逻辑和层次，甚至分类。

2.很多函数复制粘贴多次，后面修改代码，得跟着所有函数改一遍。

3.几乎没有封装，要用另外一个 package 的函数，直接大写，导成全局函数。

4.当看到 package 里一个 go 文件全局变量是这么定义的（ types 为另外一个包）

var(
a=types.Function1
b=types.Function2
......
......
n=types.Functionn
）

一共定义了 100 多个全局变量。。。。。。

这个人自称 10 年开发经验，我怀疑他是故意这么写代码的。我当时就觉得项目上线 1 个星期内肯定黄，这种代码如何维护更新，我看我要准备跑路了。。。。。 当你想好好做事的时候，碰到这种货，，，哎。。。。。。。。重头再来了。

[awolfly9]

项目黄不黄其实和代码没有一点关系

[Muninn]

如果是别的语言还好……

如果是 golang，你没法说啥，因为最开始官方就是推荐这么写的。

铺 100 个文件的著名项目都多得是。

所以 1 不是问题。 2，算是半个问题，社区有很多过度封装的讨论，有些情况复制是更好的。

3，是 1 和 2 衍生的问题，也不是问题。

4，确实有点问题


总的说来 创业公司刚开始主要是跑通业务 很多公司刚开始用 php 或者啥别的 cms 改改后面再重来。

[u823tg]

10 年这样肯定是故意这么写的，不用怀疑。

[zhuangzhuang1988]

还好是 go
要是 js, 或者 python, ruby
那才是要黄

[luoqeng]

Golang 就是当代的 PHP

[liujialongstar]

之前疯传过一篇如何写出无法维护的代码的文章, 这位深得其中真意, 或许这样, 这位就变得无可替代了

[xiangyuecn]

一言不合就重构😆 又可以多混半年工钱

[george404]

@Muninn
1. 要是著名项目就算了，这种 low b 项目根本没到那档次。再说好的项目至少不同版本不会全部塞一个文件夹下吧？整个项目基本上就 2-3 个 package 。


不是过度封装，是几乎没有封装，一个文件夹下 aa_handler.go, bb_handerler.go,cc_handler.go ,然后 aa_handler_v2.go, bb_handler_v2.go 全是这种东西，恶心不？公共函数和逻辑都不提出来。直接 copy-past,大概看一眼，全是每个源文件里面的函数名都基本上相似。

好像我在哪看到，go 中尽量避免使用全局可变的变量，容易被误修改。我是比较认同这种说法

[george404]

@u823tg 嗯，我也是觉得，这样做快，不用考虑接口，老板看功能好的快，开心，同时，自己又不可替代了，呵呵。。。。。没有人会接那烂摊子。

[george404]

@liujialongstar 同意！🤝

[luojian666]

跑路跑路

[xuanbg]

还好吧，才 4 万行代码，里面有效代码估计不足 1 万行。实在不行重写一遍也费不了多少时间。总之，这个正在向不可替代发展的先干下去再说，不然还真的要变成不可替代的了。

[qinfensky]

大道至简

[DJQTDJ]

这么说吧，公司可以没有你，但是不能没有他。
他的代码只有他知道怎么改。
你一改你就得懵圈

[lepig]

@DJQTDJ 所以说楼主还是太年轻啊。

[fy1993]

不可或缺的人

[murmur]

我记得 lua 年代就有把库里的函数再定义出来一次加快运行速度的例子
比如 local cosine = Math.cosine

[roundgis]

系統能用嗎？

[iwh718]

第一版的话，能跑通就可以了。

[DelayNoMay]

改需求的话，自己也得累死啊