5-3作业运行之后只有读取天气数据，而且数据为0

100

101

102

103

104

105

public class Weather {
     
    private int temperature,humidity;
    boolean flag=true;
     
    public Weather(){
         
    }
    public int getTemperature() {
        return temperature;
    }
    public void setTemperature(int a) {
        this.temperature = a;
    }
    public int getHumidity() {
        return humidity;
    }
    public void setHumidity(int b) {
        this.humidity = b;
    }
     public synchronized void generate(){
         while(flag){
             try {
                 wait();
             } catch (InterruptedException e) {
                 // TODO Auto-generated catch block
                 e.printStackTrace();
             }
         }
           
         this.setTemperature((int)(Math.random()*40));
         this.setHumidity((int)(Math.random()*100));
         System.out.println("生成天气数据[温度："+this.getTemperature()+",湿度："+this.getHumidity()+"]");
         flag=true;
         notifyAll();
 }
     public synchronized void read(){
         while(!flag){
             try {
                 wait();
             } catch (InterruptedException e) {
                 // TODO Auto-generated catch block
                 e.printStackTrace();
             }
         }
         System.out.println("读取天气数据[温度："+this.getTemperature()+",湿度："+this.getHumidity()+"]");
         flag=false;
         notifyAll();
 }
 
    @Override
    public String toString() {
        return "天气数据 [温度=" + temperature + ", 湿度=" + humidity + "]";
    }
     
}
public class GenerateWeather implements Runnable {
    Weather weather;
 
    public GenerateWeather(Weather weather) {
        this.weather = weather;
    }
 
    @Override
    public void run() {
        for (int i = 0; i < 100; i++) {
            weather.generate();
            System.out.println("生成"+ weather);
            try {
                Thread.sleep(5000);
            } catch (InterruptedException e) {
                // TODO 自动生成的 catch 块
                e.printStackTrace();
            }
        }
    }
}
public class ReadWeather implements Runnable {
    Weather weather;
 
    public ReadWeather(Weather weather) {
        this.weather = weather;
    }
 
    @Override
    public void run() {
        for (int i = 0; i < 100; i++) {
            weather.read();
            try {
                Thread.sleep(500);
            } catch (InterruptedException e) {
                // TODO 自动生成的 catch 块
                e.printStackTrace();
            }
        }
    }
}
public class WeatherTest {
    public static void main(String[] args) {
        new Thread(new GenerateWeather(new Weather())).start();
        new Thread(new ReadWeather(new Weather())).start();
 
    }
}
运行之后只有读取天气数据，而且数据为0，这一章实在是听不太明白了，求老师耐心指导，谢谢。

半岛鸡盒 2017-05-26

源自：Java多线程 5-3 作业题

收起

3回答

好帮手慕珊回答被采纳获得+3积分 2017-05-26 18:40:42

你好，问题如下：

1、Weather类的generate和read方法不应该用while，而是if，即if(flag)和if(!flag)，如果flag为true或false后调用wait()方法，如果是循环就会一直循环下去，因为循环中flag的值没有变化，所以一直满足条件，程序就会一直执行。

2、为什么会输出一次数据0呢，因为flag初始值为true，满足generate方法中的条件，所以一开始生成数据的方法在等待，并且没有数据生成。而读取数据的方法没有等待，可以执行输出语句，因为温度和湿度都没有赋值，所以输出的是默认值0。然后flag的值变为false，read()方法再次执行时也会调用wait()方法进行等待，所以也就没有任何输出了。

3、GenerateWeather类中，run()方法中不要写输出语句，因为该方法并没有用synchronize关键字修饰，所以代码可能在执行一半的时候被打断，就会使输出结果不符合作业要求。

4、WeatherTest类，两个线程共享一个Weather类对象，所以是先创建一个Weather类的对象weather，GenerateWeather类和ReadWeather类创建对象时的参数都是weather

如果我的回答解决了你的疑惑，请采纳！祝学习愉快！

收起回答