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

V2EX = way to explore

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

现在注册

已注册用户请登录

• 请不要在回答技术问题时复制粘贴 AI 生成的内容

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

想了解一下 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 转型，直接强转)。这类看起来才是头大。
开源就是这样的，如果面对单一业务线，精简很多，也不需要考虑那么多。