看高手代码--从小case学大道理

今天看sun的HttpMessages.java文件，虽然文件很小，但是对我的启发很大。
前面定义了大量的HTTP常量：

    ...
    ...
    private static final String STATUS_305 = "Use Proxy";
    private static final String STATUS_307 = "Temporary Redirect";
    private static final String STATUS_400 = "Bad Request";
    private static final String STATUS_401 = "Unauthorized";
    private static final String STATUS_402 = "Payment Required";
    private static final String STATUS_403 = "Forbidden";
    private static final String STATUS_404 = "Not Found";
    private static final String STATUS_405 = "Method Not Allowed";
    private static final String STATUS_406 = "Not Acceptable";
    private static final String STATUS_407 = "Proxy Authentication Required";
    private static final String STATUS_408 = "Request Timeout";
    ...
    ...

也可以趁机从这里查看HTTP返回值的意义^_^。然后定义了一个java.util.concurrent.ConcurrentHashMap的变量httpStatusCodeMappings，再往里面添加刚才定义的常量（这个操作放在了一个静态块里），如：

static {
        httpStatusCodeMappings.put("sc.100", STATUS_100);
        httpStatusCodeMappings.put("sc.101", STATUS_101);
        httpStatusCodeMappings.put("sc.200", STATUS_200);
        httpStatusCodeMappings.put("sc.201", STATUS_201);
        httpStatusCodeMappings.put("sc.202", STATUS_202);
        httpStatusCodeMappings.put("sc.203", STATUS_203);
  ...
  ...

最重要的就是这个方法了，先看代码：

public static String getMessage(int status) {

        // Return the status message for the most frequently used status
        // codes directly, without any lookup
        switch (status) {
            case 200: return STATUS_200;
            case 302: return STATUS_302;
            case 400: return STATUS_400;
            case 404: return STATUS_404;
	}

        return httpStatusCodeMappings.get("sc."+ status);
    }

这段代码的用途很简单，就是要返回状态码对应的Message，而这些这些消息都已经存放在httpStatusCodeMappings变量里了，为了提高访问常用的几个返回码的Message，它直接用了一个静态块，而不去lookup那个map了，这就是高手！

评论

16 楼 diferent 2010-06-01  

iaimstar 写道

skydream 写道

sw1982 写道

...lookup 一下hashmap真的那么低效吗？ 建议复习下数据结构哦，你这些总结是没错，可是很表面

典型的没有写过高并发程序的思维方式，明明可以节约的地方，仅仅几行代码就可以优化，偏偏不做。

hashmap再快，也比case 一个 整型满上1w倍。

性能，是一点一点挤牙膏挤出来的，哪能到处浪费啊。

一个团队好多个人

指望所有人都挤牙膏，开玩笑

顶多就是核心代码慎重一些

整个项目再整体测试一下性能瓶颈，有针对性的调整就好了

每句代码都想写的句句珠玑，既不可能，也不现实。

在需求允许的范围内，尽量的提高开发效率，才能适时的交出合适的作业

做国内项目的吧 没有人在乎你的性能
但是还是有许多场合要求的 十分严格
我觉得 楼主找到的例子的价值 远比JE上讨论Linux与Windows Firefox与IE C与Java这种问题有意义的多.
"每句代码都想写的句句珠玑，既不可能，也不现实。"
要知道作家写出的每句话都是细细斟酌而出的
只有网上写连载小说的 才不会珍惜自己写出的文字

15 楼 skyuck 2010-06-01  

嗯   可以拿来为我所用.

14 楼 抛出异常的爱 2010-06-01  

mathfox 写道

又一个可以在工作中装x的方法出现了。

少装X 装X遭雷劈

13 楼 shuiguozheng 2010-06-01  

skydream 写道

sw1982 写道

...lookup 一下hashmap真的那么低效吗？ 建议复习下数据结构哦，你这些总结是没错，可是很表面

典型的没有写过高并发程序的思维方式，明明可以节约的地方，仅仅几行代码就可以优化，偏偏不做。

hashmap再快，也比case 一个 整型满上1w倍。

性能，是一点一点挤牙膏挤出来的，哪能到处浪费啊。

     lz分享的精神值得学习！

12 楼 mathfox 2010-06-01  

又一个可以在工作中装x的方法出现了。

11 楼 stoneskin 2010-06-01  

anyway...学习到了提高效率的一种方式。。

10 楼 iaimstar 2010-06-01  

skydream 写道

sw1982 写道

...lookup 一下hashmap真的那么低效吗？ 建议复习下数据结构哦，你这些总结是没错，可是很表面

典型的没有写过高并发程序的思维方式，明明可以节约的地方，仅仅几行代码就可以优化，偏偏不做。

hashmap再快，也比case 一个 整型满上1w倍。

性能，是一点一点挤牙膏挤出来的，哪能到处浪费啊。

一个团队好多个人

指望所有人都挤牙膏，开玩笑

顶多就是核心代码慎重一些

整个项目再整体测试一下性能瓶颈，有针对性的调整就好了

每句代码都想写的句句珠玑，既不可能，也不现实。

在需求允许的范围内，尽量的提高开发效率，才能适时的交出合适的作业

9 楼 skydream 2010-05-31  

switch (status) { 
            case 200: return STATUS_200; 
            case 302: return STATUS_302; 
            case 400: return STATUS_400; 
            case 404: return STATUS_404; 

正常情况下，一个正常工作的系统，返回状态码为上述几个的概率应该是99.99%以上，这个时候的节约就很可观的。

8 楼 skydream 2010-05-31  

sw1982 写道

...lookup 一下hashmap真的那么低效吗？ 建议复习下数据结构哦，你这些总结是没错，可是很表面

典型的没有写过高并发程序的思维方式，明明可以节约的地方，仅仅几行代码就可以优化，偏偏不做。

hashmap再快，也比case 一个 整型满上1w倍。

性能，是一点一点挤牙膏挤出来的，哪能到处浪费啊。

7 楼 srdrm 2010-05-31  

hashmap 效率再怎么O(1)，也得调用hashCode，调用equals，进行 % 的运算，肯定不及将最常用的几个状态码作为条件判断语句来得快

考虑真实场景，大并发用户下，绝大多数是 200，如果这时候还hashmap 里查找，量大起来，效率肯定是有差距的，而这个节省效率的成本，也就是写几个 switch的case 而已，何乐不为

6 楼 181054867 2010-05-31  

这些我也经常用，很常见的技术

5 楼 sw1982 2010-05-31  

ConcurrentHashMap是分块治理的思路，插入的时候会差一点，因为设计到块的拷贝，查询效率还是O(1)

swanky_yao 写道

sw1982 写道

...lookup 一下hashmap真的那么低效吗？ 建议复习下数据结构哦，你这些总结是没错，可是很表面

多谢教导！！

这里的hashmap是一个java.util.concurrent.ConcurrentHashMap，应该会慢一些吧？

4 楼 godtiger 2010-05-31  

道行太浅,没明白啥意思

3 楼 swanky_yao 2010-05-31  

sw1982 写道

...lookup 一下hashmap真的那么低效吗？ 建议复习下数据结构哦，你这些总结是没错，可是很表面

多谢教导！！

这里的hashmap是一个java.util.concurrent.ConcurrentHashMap，应该会慢一些吧？

2 楼 jenlp520 2010-05-31  

     
case 200: return STATUS_200;  
            case 302: return STATUS_302;  
            case 400: return STATUS_400;  
            case 404: return STATUS_404;  

这在编译成class后这里都是常量了 不是什么静态块

1 楼 sw1982 2010-05-31  

...lookup 一下hashmap真的那么低效吗？ 建议复习下数据结构哦，你这些总结是没错，可是很表面