A股上市公司传智教育(股票代码 003032)旗下技术交流社区北京昌平校区

 找回密码
 加入黑马

QQ登录

只需一步,快速开始

© 侯丛政 中级黑马   /  2013-2-27 00:14  /  1003 人查看  /  3 人回复  /   0 人收藏 转载请遵从CC协议 禁止商业使用本文

        public Lamp blackOut(){
                this.lighted = false;
                if(opposite != null){
                        Lamp.valueOf(opposite).blackOut();
                }               
               
            //  中途出错的代码
                Lamp nextLamp= Lamp.valueOf(next);    // 这是张老师交通灯的一段代码, 为什么最开始使用这段代码运行时灯没有                       变呢?  nextLamp的值在if里面赋和在外面有什么不同吗 ?  
                if(next != null){       
                        nextLamp.light();
                }
                return nextLamp;
  
            //  最终改好的方法
        /*        Lamp nextLamp= null;
                if(next != null){
                        nextLamp = Lamp.valueOf(next);               
                        nextLamp.light();
                }
                return nextLamp;
        */
         }

评分

参与人数 1技术分 +1 收起 理由
陈丽莉 + 1

查看全部评分

3 个回复

倒序浏览
Lamp nextLamp= Lamp.valueOf(next);
假如在if外面定义,因为 if(next != null){   //条件不成立,无法执行里面的代码
                        Lamp nextLamp= Lamp.valueOf(next);  
                        nextLamp.light();
                }
                return nextLamp;
不知道是不是这样。

评分

参与人数 1技术分 +1 收起 理由
陈丽莉 + 1

查看全部评分

回复 使用道具 举报
本帖最后由 张学林 于 2013-3-5 12:05 编辑
  1. public class LampController {
  2.         private Lamp currentLamp;
  3.         
  4.         public LampController(){
  5.                 currentLamp = Lamp.S2N;//定义南向北为初始绿灯
  6.                 currentLamp.light();
  7.                
  8.                
  9.                 ScheduledExecutorService timer = Executors.newScheduledThreadPool(1);
  10.                 timer.scheduleAtFixedRate(
  11.                                 new Runnable(){
  12.                                         public void run(){//通过blackOut方法获取下一个线路的绿灯
  13.                                                 currentLamp = currentLamp.blackOut();
  14.                                         }
  15.                                 },
  16.                                 3,
  17.                                 3,
  18.                                 TimeUnit.SECONDS);
  19.         }
  20. }
复制代码
因为初始绿灯为S2N,当S2N.blackOut()时
  1.         public Lamp blackOut(){
  2.                 this.lighted = false;
  3.                 if(opposite != null){
  4.                         Lamp.valueOf(opposite).blackOut();
  5.                         
  6.                 }
  7. //               
  8. //                Lamp nextLamp = Lamp.valueOf(next);
  9. //
  10. //                if(next != null){        
  11. //                        System.out.println("绿灯从"+ name() + "----------->切换为"+ next);
  12. //                        nextLamp.light();
  13. //            }
  14.                
  15.                 Lamp nextLamp = null;
  16.                 if(next != null){
  17.                         System.out.println("绿灯从"+ name() + "----------->切换为"+ next);
  18.                         nextLamp = Lamp.valueOf(next);
  19.                         nextLamp.light();
  20.                 }
  21.                 return nextLamp;
  22.         }
复制代码
blackOut方法执行了S2N.lighted = false;由于(opposite=N2S)不等于空
又执行了N2S.blackOut()     N2S.lighted = false;   此时opposite和next都为null以至于后面的
Lamp nextLamp = Lamp.valueOf(next) ;会出空指针异常

评分

参与人数 1技术分 +1 收起 理由
陈丽莉 + 1

查看全部评分

回复 使用道具 举报
本帖最后由 张学林 于 2013-3-5 12:23 编辑

上传个图片
从网上搜到的下面这句话
特别提示通过ScheduledExecutorService执行的周期任务如果任务执行过程中抛出了异常那么过ScheduledExecutorService就会停止执行任务且也不会再周期地执行该任务了。所以你如果想保住任务都一直被周期执行那么catch一切可能的异常
可能就是因为这个原因才没有打印异常信息的吧

111212.jpg (111.47 KB, 下载次数: 17)

111212.jpg

评分

参与人数 1技术分 +1 收起 理由
陈丽莉 + 1

查看全部评分

回复 使用道具 举报
您需要登录后才可以回帖 登录 | 加入黑马