1. package cn.itcast.day1;
   
2. 
   
3. public class EnumTest {
   
4.            public static void main(String[] args){
   
5. 
   
6.                   new Thread(new Runnable(){
   
7.                          public void run(){
   
8.                                   display(TrafficLamp.RED);
   
9.                          }
   
10.                   }).start();
    
11.                   new Thread(new Runnable(){
    
12.                          public void run(){
    
13.                                   display(TrafficLamp.GREEN);
    
14.                          }
    
15.                   }).start();
    
16. 
    
17. 
    
18.            }
    
19. 
    
20.            public enum TrafficLamp{
    
21.            //当抽象方法存在，必须覆盖枚举类中的抽象方法
    
22.                      RED(25){
    
23.                            public TrafficLamp nextLamp(){
    
24.                                  return GREEN;
    
25.                            }
    
26.                      },GREEN(20){
    
27.                            public TrafficLamp nextLamp(){
    
28.                                  return YELLOW;
    
29.                            }
    
30.                     },YELLOW(5){
    
31.                            public TrafficLamp nextLamp(){
    
32.                                  return RED;
    
33.                            }
    
34.                     };
    
35. 
    
36.                     public abstract TrafficLamp nextLamp();
    
37. 
    
38.                     private int time;
    
39.                     private TrafficLamp(int time){
    
40.                              this.time = time;
    
41.                     }
    
42.                     public int getTime(){
    
43.                              return time;
    
44.                     }
    
45.                    public void setTime(int time){
    
46.                              this.time = time;
    
47.                    }
    
48.            }
    
49. 
    
50.            public static void display(TrafficLamp state){
    
51.                      while (true) {
    
52.                                   switch (state) {
    
53.                                          case GREEN:
    
54.                                                      if (state.getTime() != 0) {
    
55.                                                            state.setTime(lampSleep(state, state.getTime()));
    
56.                                                      } else {
    
57.                                                            state = state.nextLamp();
    
58.                                                      }
    
59.                                         case RED:
    
60.                                                     if (state.getTime() != 0) {
    
61.                                                            state.setTime(lampSleep(state, state.getTime()));
    
62.                                                    } else {
    
63.                                                            state = state.nextLamp();
    
64.                                                    }
    
65.                                        case YELLOW:
    
66.                                                    if (state.getTime() != 0) {
    
67.                                                            state.setTime(lampSleep(state, state.getTime()));
    
68.                                                    } else {
    
69.                                                            state = state.nextLamp();
    
70.                                                    }
    
71.                                }
    
72.                     }
    
73.            }
    
74. 
    
75.            public static int lampSleep(TrafficLamp state,int count){
    
76.                     try {
    
77.                              Thread.sleep(1000);
    
78.                     } catch (InterruptedException e) {
    
79.                              e.printStackTrace();
    
80.                     }
    
81.                    System.out.println(state.toString()+count);
    
82.                    count--;
    
83.                    return count;
    
84.           }
    
85. 
    
86. }
    

复制代码

有没有人可以用eclipse运行一下以上代码，代码的目标是：
运行一个模拟交通灯，进行倒计时，红-->黄-->绿-->红，每秒都会输出“灯状态+剩余秒数”。

我遇到的问题是：
当交通灯走完一个循环以后，似乎阻塞住不动了，但是主线程还在运行中，因为运行并没有终止，这是怎么回事呢？
为了验证是不是主线程问题，我还自己在main中加了两个线程来运行，貌似仍然会出现同样的问题，

求帮助，求解答

• 我觉得楼主的代码适合写成单线程，因为有state.nextLamp();，直接单线程互相转换就行。我运行了单线程，很正常~！
  
  主函数直接用：
  public static void main(String[] args){
  

                  new Thread(new Runnable(){

                         public void run(){

                                  display(TrafficLamp.RED);

                         }

                  }).start();


楼主写的代码非常好哇~！

不用state.nextLamp();的话可以用3线程的相互唤醒，不过不如楼主的简单~

贾联国 发表于 2012-5-3 00:41

我觉得楼主的代码适合写成单线程，因为有state.nextLamp();，直接单线程互相转换就行。我运行了单线程 ...

谢谢夸奖，本人新人一枚。

嗯，其实用单线程的话就和用main直接执行应该是一样的。
我用两个线程是假设模拟东西方向 和 南北方向的灯转换。严谨一些应该加个同步什么的。为了单纯演示我就偷懒了。

我试了一下，其实按理来说是只要不手动停止程序的话，是会一直循环打印的。但是我尝试了一次还是不行。比如打印 red--->green-->yellow 然后就停止打印了，但是Main函数还是在运行的。
所以，不知道是我eclipse的问题，还是其他。

不过还是感谢帮忙，因为帖子没什么人回。
估计黑马9期招满了以后，论坛的活跃性就稍微减弱了吧。

云惟桉 发表于 2012-5-3 01:04
谢谢夸奖，本人新人一枚。

嗯，其实用单线程的话就和用main直接执行应该是一样的。

南北走向也是可以的，我的eclipse也是同样问题，但是控制台就没有问题。

lz你的nextLamp实现貌似不是红-黄-绿-红的顺序 把返回值改一下吧、
另外就是switch/case的地方，每一段之后就要break，否则每次循环三个case都执行，相当于，循环一次，同一个灯的time减减三次（time大于0的时候），比如起步是红灯，第一次循环之后，红灯的time减了3，其他灯没变。
还有就是，在切换下一个灯之后，把下一个灯的时间重置，三个灯都走完一次不动就是因为他们的时间都归零了。这样我用单线程走了一次看起来是正常的。你自己再看一下吧，很晚了 先睡了明天还要上班{:soso_e136:}

刘少伟 发表于 2012-5-3 02:20
lz你的nextLamp实现貌似不是红-黄-绿-红的顺序 把返回值改一下吧、
另外就是switch/case的地方，每一段之后 ...

十分感谢帮忙！但是灯的状态是没错的，绿灯之后才是黄灯，生活里的顺序。
另外，我查阅了一下java核心技术，才发现我的错，原文是这样的：

【警告】：switch语句有可能触发多个case分支。如果在case分支语句的末尾没有break语句，那么就会接着执行下一个case分支语句。这种情况相当危险，常常会引发错误。为此，我们在程序中从不使用switch语句。
                                                                         ——P70 《Java核心技术：卷1：基础知识》

细节真的使人挫败啊，楼主真是有心人。正确的解决方法就像你所说的，应该是考虑两点：
1.运行过程使用set方法修改了time的值，因此在状态转换时一定需要重置time；
2.没有break语句，无论满不满足case判断，每一个case都会执行（可以使用system.out输出检验）

希望我的错误能给看帖的人带来收获。
正确代码如下：

1. public static void display(TrafficLamp state){
   
2.           while (true) {
   
3.                    switch (state) {
   
4.                    case GREEN:
   
5.                            if (state.getTime() != 0) {
   
6.                                  state.setTime(lampSleep(state, state.getTime()));
   
7. 
   
8.                            } else {
   
9.                                  state = state.nextLamp();
   
10.                                  state.setTime(5);
    
11.                            }
    
12.                            break;
    
13.                    case RED:
    
14.                            if (state.getTime() != 0) {
    
15.                                  state.setTime(lampSleep(state, state.getTime()));
    
16.                            } else {
    
17.                                  state = state.nextLamp();
    
18.                                  state.setTime(20);
    
19.                            }
    
20.                            break;
    
21.                    case YELLOW:
    
22.                            if (state.getTime() != 0) {
    
23.                                  state.setTime(lampSleep(state, state.getTime()));
    
24.                            } else {
    
25.                                  state = state.nextLamp();
    
26.                                  state.setTime(25);
    
27.                            }
    
28.                            break;
    
29.                   }
    
30.          }
    
31. }

复制代码

贾联国 发表于 2012-5-3 01:37
南北走向也是可以的，我的eclipse也是同样问题，但是控制台就没有问题。
...

经过有心人提点，终于找到错误了。
你可以看我的回复，希望你也有收获，呵呵。

收获很大！