MyException - 我的异常网
当前位置:我的异常网» Java Web开发 » 求教牛人两段代码为什么不合理?解决方法

求教牛人两段代码为什么不合理?解决方法

www.MyException.Cn  网友分享于:2013-03-05  浏览:2次
求教牛人两段代码为什么不合理?

      近日完成了一个项目,然后把代码交给总公司进行code   review时,总公司同事指出下面两段代码存在问题,并给出了修改意见,是用英文描述的,小弟基本上能看懂其意思,但不知如何来修改,特地请教各位大虾.

        第一段代码:
                  public   Category   getCategory()   {
return   category;
}
public   Category   getCategoryRank()   {
return   categoryRank;
}
            同事给出的Error   Description:
              Directly   return   the   object   reference   of   a   mutable   object   from   getter   method   may   violates   the   loose   coupling   concept   and   casue   thread   interference   problem   unless   it   is   well   thought   and   written   in   documents.

                  Suggestion:
                        Return   a   duplicated   object


        第二段代码:

            public   synchronized   boolean   add()   {
    boolean   result   =   false;

try   {
                              .........
                          }catch(Exception   ){.....
                        }finally{
                          ....  
                          }
                    return   result
                            }

                同事给出的Error   Description:
                      Where   2   users   add   a   new   category   at   the   same   time,   the   2   new         categories   can   be   with   the   same   name   which   is   not   desired.
             
                          Suggestion:
                    Intrinsic   locks   of   class   but   not   object.


------解决方案--------------------
第二个是把方法改为static的 第一个好像是说返回的对象的clone,
英文不好不知道说的对不对
------解决方案--------------------
第二个控制synchronized 粒度在细些!
------解决方案--------------------
在第二个类里成名个static 类型的属性,用这个属性做锁旗标,用来锁类(个人理解)
------解决方案--------------------
import java.lang.Runnable;

public class TestFrame{
public static void main(String[] args){
synchronized (name){
TestRunnable test1 = new TestRunnable();

文章评论

“肮脏的”IT工作排行榜
“肮脏的”IT工作排行榜
为什么程序员都是夜猫子
为什么程序员都是夜猫子
10个帮程序员减压放松的网站
10个帮程序员减压放松的网站
程序员最害怕的5件事 你中招了吗?
程序员最害怕的5件事 你中招了吗?
一个程序员的时间管理
一个程序员的时间管理
程序员必看的十大电影
程序员必看的十大电影
Web开发者需具备的8个好习惯
Web开发者需具备的8个好习惯
Java 与 .NET 的平台发展之争
Java 与 .NET 的平台发展之争
要嫁就嫁程序猿—钱多话少死的早
要嫁就嫁程序猿—钱多话少死的早
我跳槽是因为他们的显示器更大
我跳槽是因为他们的显示器更大
 程序员的样子
程序员的样子
写给自己也写给你 自己到底该何去何从
写给自己也写给你 自己到底该何去何从
程序员眼里IE浏览器是什么样的
程序员眼里IE浏览器是什么样的
Web开发人员为什么越来越懒了?
Web开发人员为什么越来越懒了?
我的丈夫是个程序员
我的丈夫是个程序员
漫画:程序员的工作
漫画:程序员的工作
程序员的一天:一寸光阴一寸金
程序员的一天:一寸光阴一寸金
代码女神横空出世
代码女神横空出世
Java程序员必看电影
Java程序员必看电影
做程序猿的老婆应该注意的一些事情
做程序猿的老婆应该注意的一些事情
程序员的鄙视链
程序员的鄙视链
5款最佳正则表达式编辑调试器
5款最佳正则表达式编辑调试器
程序员都该阅读的书
程序员都该阅读的书
总结2014中国互联网十大段子
总结2014中国互联网十大段子
程序员和编码员之间的区别
程序员和编码员之间的区别
那些争议最大的编程观点
那些争议最大的编程观点
鲜为人知的编程真相
鲜为人知的编程真相
程序员应该关注的一些事儿
程序员应该关注的一些事儿
科技史上最臭名昭著的13大罪犯
科技史上最臭名昭著的13大罪犯
旅行,写作,编程
旅行,写作,编程
中美印日四国程序员比较
中美印日四国程序员比较
老程序员的下场
老程序员的下场
聊聊HTTPS和SSL/TLS协议
聊聊HTTPS和SSL/TLS协议
亲爱的项目经理,我恨你
亲爱的项目经理,我恨你
初级 vs 高级开发者 哪个性价比更高?
初级 vs 高级开发者 哪个性价比更高?
每天工作4小时的程序员
每天工作4小时的程序员
老美怎么看待阿里赴美上市
老美怎么看待阿里赴美上市
看13位CEO、创始人和高管如何提高工作效率
看13位CEO、创始人和高管如何提高工作效率
团队中“技术大拿”并非越多越好
团队中“技术大拿”并非越多越好
10个调试和排错的小建议
10个调试和排错的小建议
如何成为一名黑客
如何成为一名黑客
“懒”出效率是程序员的美德
“懒”出效率是程序员的美德
我是如何打败拖延症的
我是如何打败拖延症的
编程语言是女人
编程语言是女人
什么才是优秀的用户界面设计
什么才是优秀的用户界面设计
软件开发程序错误异常ExceptionCopyright © 2009-2015 MyException 版权所有