吐槽一下 Java JRE 的源码可读性有点差

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 1556 天前的主题，其中的信息可能已经有所发展或是发生改变。

想了解一下 java.util.concurrent.ConcurrentHashMap，发现可读性太差了。

吐槽点如下：

大部分局部变量都是单个字母命名（ WTF ）。
大量的位运算（不明觉厉）。
大量的魔法值（ WTF ）。
很多层嵌套循环在一个方法中（头晕目眩）。
一个 if 判断条件中，含大量的与、或子句，且含许多赋值语句。

以上或许都是为了追求极致性能，但是可读性真的太差了。

WTF

可读性

嵌套

吐槽

23 条回复 • 2020-08-21 22:27:56 +08:00

wsy2220

2020-08-21 09:30:00 +08:00 via Android

底层数据结构实现是可以为了性能牺牲可读性的

yangbonis

2020-08-21 09:36:22 +08:00 via iPhone

单个字母命名屏蔽垃圾信息，都是通用的逻辑，又不是业务，取名字反而是干扰。

yangbonis

2020-08-21 09:37:23 +08:00 via iPhone

你只是站在自己角度觉得可读性差，站在作者角度，说不定人家觉得很好。

40EaE5uJO3Xt1VVa

2020-08-21 09:41:48 +08:00

看到 jre 我就过来了 www.injdk.cn

taogen

2020-08-21 09:43:30 +08:00

@yangbonis #2 当有多个单字母变量在一起时，很难记住每个变量的含义，除非对代码非常熟悉。

lamy

2020-08-21 09:49:12 +08:00 via Android

和 c++ stl 比一下

Still4

2020-08-21 09:49:38 +08:00

你才搞了几年，写底层的搞了多少年，不是说资历就是能力，而是人家这么做一定有道理，底层必然优先性能，可读性靠边站，参考这个

https://www.v2ex.com/t/691522

flippydoo

2020-08-21 09:56:16 +08:00 via iPhone

这可读性都能差？但凡你好好看了 chm 类上面的注释和属性注释都不会这么讲，douglee 差不多用注释写了篇论文了

ChanKc

2020-08-21 09:57:23 +08:00 via Android

函数里面单个字母还挺正常的
位运算看习惯就好，很多和 2 相关的运算，用位运算都比乘除要快。而且我看位运算的基本都有注释。

sockpuppet9527

2020-08-21 10:33:18 +08:00

1. 局部变量单个字母命名：个人习惯问题，一般为单次首字母，看多了也习惯了，kernel 里面长词也会这样，打个比方，super block = sb , inode = i or n , Group Descriptors = gdt, Multiple mount protection = mmp
2. 位运算：位运算很常见也很方便，如果不明觉厉我个人会认为这是你的问题，举个简单的例子：掩码，再举个简单的例子：两个 32 位变量支持 64 位变量。
如果不用位运算，去写一大堆逻辑来支撑某个功能，那样做反而更复杂。
3. 魔法值：不知道楼主说的是不是 Magic number 。如果是的话，那我想说，如果你是 jre 的维护者，你不考虑版本兼容问题？

其他逻辑上的，大项目的基础功能都是这样的，面向的不是业务，考虑的东西以及开发的经验都是日积月累的。

sockpuppet9527

2020-08-21 10:38:59 +08:00

@sockpuppet9527 #10 补充一下，JRE 最少还有注释，还不断重构。
一些维护十年、二十年的开源项目注释都很少，patch 打的多了也不重构（开发者无精力），兼容多代的版本，兼容多个平台（指 OSX/WIN32/SUN/UNIX/LINUX)，甚至因为 GCC 版本问题而弄得一些兼容(特指不通过 CONTAINER_OF 转型，直接强转)。这类看起来才是头大。
开源就是这样的，如果面对单一业务线，精简很多，也不需要考虑那么多。