Обнуляется Label

@4elus · Регистрация: 30.06.2017

Студворк — интернет-сервис помощи студентам

Здравствуйте, программа показывает температуру в градусах цельсия, когда добавляю новую температуру в фаренгйтах первый label обнуляется. Не могли бы Вы сказать как это исправить ?

@4elus · 24.10.2018, 01:26 **[ТС]**

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
public class Runner {
 
    /**
     * @param args
     */
    public static void main(String[] args) {
        new HeaterProgram().run();
 
    }
 
}
 
 
public class HeaterView {
 
    GraphicalDisplay display = new GraphicalDisplay("Температура", "Температура2");
 
    public void updateView(Heater h) {
        String temp = Integer.toString(h.getTemperature());
        display.setText1(temp + "C");
    }
}
 
 
 
public class HeaterFahrenheitView {
 
    GraphicalDisplay display = new GraphicalDisplay("Температура", "Температура2");
 
    public void updateDisplay(Heater view){
        double temp = (view.getTemperature() * 1.8) + 32;
        String t = Double.toString(temp);
        display.setText2(t + "F");
    }
}
 
 
import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
 
public class HeaterProgram {
 
    private BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
    private Heater heater;
    private HeaterView hView;
    private HeaterFahrenheitView heaterFahrenheitView;
 
    public void run() {
        heater = new Heater(20, 5);
        hView = new HeaterView();
        heaterFahrenheitView = new HeaterFahrenheitView();
        heater.setView(hView);
        heater.setFahrenheitView(heaterFahrenheitView);
        int choice;
        try {
            do {
                choice = menu();
                if (choice == 1)
                    heater.warmer();
                if (choice == 2)
                    heater.colder();
            } while (choice != 0);
        } catch (IOException e) {
            e.printStackTrace();
        }
        System.exit(0);
    }
 
    public int menu() throws IOException {
        System.out.println("Выберите операцию");
        System.out.println("'1' - увеличить температуру");
        System.out.println("'2' - уменьшить температуру");
        System.out.println("'0' - завершить работу");
        String choice = br.readLine();
        return Integer.parseInt(choice);
    }
 
}
 
 
public class Heater {
 
    private int temperature;
    private int increment;
    private HeaterView view;
    private HeaterFahrenheitView fahrenheitView;
 
    public Heater(int temperature, int increment) {
        super();
        this.temperature = temperature;
        this.increment = increment;
    }
 
    public int getTemperature() {
        return temperature;
    }
 
    public int getIncrement() {
        return increment;
    }
 
    public void colder() {
        temperature -= increment;
        if (view != null && fahrenheitView != null) {
            view.updateView(this);
            fahrenheitView.updateDisplay(this);
        }
    }
 
    public void warmer() {
        temperature += increment;
        if (view != null && fahrenheitView != null) {
            view.updateView(this);
            fahrenheitView.updateDisplay(this);
        }
    }
 
    public HeaterView getView() {
        return view;
    }
 
    public void setView(HeaterView view) {
        this.view = view;
    }
 
    public HeaterFahrenheitView getFahrenheitView() {
        return fahrenheitView;
    }
 
    public void setFahrenheitView(HeaterFahrenheitView fahrenheitView) {
        this.fahrenheitView = fahrenheitView;
    }
}
 
 
 
import java.awt.Dimension;
import java.awt.GridLayout;
 
import javax.swing.JFrame;
import javax.swing.JLabel;
 
public class GraphicalDisplay {
 
    private JFrame frame;
    private JLabel titleLabel1;
    private JLabel titleLabel2;
    private JLabel titleLabel3;
    private JLabel label1;
    private JLabel label2;
    private JLabel label3;
 
    public GraphicalDisplay(String title1) {
        buildFrame();
        titleLabel1.setText(title1);
    }
 
    public GraphicalDisplay(String title1, String title2) {
        buildFrame();
        titleLabel1.setText(title1);
        titleLabel2.setText(title2);
    }
 
    public GraphicalDisplay(String title1, String title2, String title3) {
        buildFrame();
        titleLabel1.setText(title1);
        titleLabel2.setText(title2);
        titleLabel3.setText(title3);
    }
 
    private void buildFrame() {
        frame = new JFrame("Графический дисплей");
        frame.setPreferredSize(new Dimension(300, 100));
        frame.setLayout(new GridLayout(3, 2));
        titleLabel1 = new JLabel();
        frame.add(titleLabel1);
        label1 = new JLabel("");
        frame.add(label1);
 
        titleLabel2 = new JLabel();
        frame.add(titleLabel2);
        label2 = new JLabel("");
        frame.add(label2);
 
        titleLabel3 = new JLabel();
        frame.add(titleLabel3);
 
        label3 = new JLabel("");
        frame.add(label3);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setAlwaysOnTop(true);
        frame.setVisible(true);
        frame.pack();
    }
 
    public void setText1(String text) {
        label1.setText(text);
    }
 
    public void setText2(String text) {
        label2.setText(text);
    }
 
    public void setText3(String text) {
        label3.setText(text);
    }
 
 
}

@vvm28 · 24.10.2018, 21:33

4elus, Почему у вас 2 класса View ? один класс "HeaterFahrenheitView" и другой "HeaterView".
Создается два окна. Так и должно быть? В одном обновляется температура по Цельсию, в другом по Фаренгейту.
PS. Дайте осмысленные имена переменным класса и вам будет легче разобраться в своем коде.
Иначе путаница какая-то получается.

Добавлено через 1 час 17 минут
Переделал вашу программу по быстрому.
Убрал лишний класс HeaterView. Осталось 5 классов.
Программа работает. Но вы продолжите над ней работать.
Вычисления не должны быть в классе View. В нем оставьте только обновление полей. Перенесите все вычисления в нужный класс.

1. class Runner остался таким же: