第一次见到这样的代码时，我的第一感觉是，它真复杂：
复制代码
List<Map<String, String>> configurations;
可只要理性稍一回归，便不难察觉，它少东西了。少什么了呢？
看看这段代码如何使用，下面是一个缩略的版本：
复制代码
for (Map<String, String> configuration : configurations) { for (Map.Entry<String, String> entry : configuration.entrySet()) { System.out.println(entry.getKey() + " " + entry.getValue()); } }
说白了很简单，其实就是要拿到存在 Map 里的键值对。在面向对象的程序语言中，有一种神奇的构造，叫做类，而它有一个很重要的特点叫做封装。是的，这段代码少了类，少了封装。闲言少叙，封装起来：
复制代码
public class ConfigurationItem { private String name; private String value; ... }
于是，那个容器嵌套容器的声明变成了
复制代码
List<ConfigurationItem> configurations;
有了类，有了封装，我们就可以再进一步进行封装，比如前面那段代码里的
复制代码
entry.getKey() + " " + entry.getValue()
实际上，可能只是为了得到这一项的字符串表示而已，那就不如直接提供一个方法：
复制代码
public class ConfigurationItem { ... @Override public String toString() { return name + " " + value; } }
于是，前面那个双重 for 循环就变化了：
复制代码
for (ConfigurationItem item : configurationItems) { System.out.println(item); }
单以丑陋而言，这段代码还算不上此类的极致，三五层嵌套也是有的，如果某些貌似负责的程序员再给每层取值都加上非空判定，那场面可是相当壮观的。
当容器开始嵌套容器，请考虑封装。
作者简介
郑晔，ThoughtWorks 公司首席咨询师，拥有十多年企业级软件开发经验，热衷于探索各种程序设计语言在真实软件开发中所能发挥的威力，致力于探寻合理的软件开发方式，加入 ThoughtWorks 公司后，投入到敏捷开发方法的实践之中，为其他公司提供敏捷开发方法方面的咨询服务。他的 blog 是梦想风暴，其微博是 @dreamhead 。
查看原文：代码之丑（十三）
感谢张凯峰对本文的审校。
给InfoQ 中文站投稿或者参与内容翻译工作，请邮件至 editors@cn.infoq.com 。也欢迎大家通过新浪微博（ @InfoQ ）或者腾讯微博（ @InfoQ ）关注我们，并与我们的编辑和其他读者朋友交流。
暂无签名
中国开发者画像洞察研究报告 2024
分析开发者的行为模式、工作价值、职业发展等内容，帮助整个行业生态更深入地理解开发者，为他们提供更精准...
拥抱 AI，我们需要什么样的存储系统？
打造属于自己的Copilot，就是现在！
数据密集型服务调度及稳定性建设实践
评论