回答編集履歴
3
コードの修正
answer
CHANGED
|
@@ -7,7 +7,7 @@
|
|
|
7
7
|
const int ms[8] = [ 0, 30, 45, 0, 0, 30, 0, 30];
|
|
8
8
|
|
|
9
9
|
if (resistance <= 1023) {
|
|
10
|
-
int t = resitance / 128;
|
|
10
|
+
int t = (resitance - 1) / 128;
|
|
11
11
|
alarm_h = hs[t];
|
|
12
12
|
alarm_m = hs[m];
|
|
13
13
|
}
|
2
コードの整形
answer
CHANGED
|
@@ -3,8 +3,8 @@
|
|
|
3
3
|
書き換えるとしてもこんな感じでしょうか。
|
|
4
4
|
|
|
5
5
|
```c
|
|
6
|
-
const int hs[8] = [6,
|
|
6
|
+
const int hs[8] = [ 6, 6, 6, 7, 7, 7, 8, 8];
|
|
7
|
-
const int ms[8] = [0, 30, 45,
|
|
7
|
+
const int ms[8] = [ 0, 30, 45, 0, 0, 30, 0, 30];
|
|
8
8
|
|
|
9
9
|
if (resistance <= 1023) {
|
|
10
10
|
int t = resitance / 128;
|
1
案の追加
answer
CHANGED
|
@@ -1,8 +1,21 @@
|
|
|
1
1
|
条件ごとの代入値に一貫した共通点がないので、質問の状態が一番簡潔だと思います。
|
|
2
2
|
|
|
3
|
-
|
|
3
|
+
書き換えるとしてもこんな感じでしょうか。
|
|
4
4
|
|
|
5
5
|
```c
|
|
6
|
+
const int hs[8] = [6, 6, 6, 7, 7, 7, 8, 8];
|
|
7
|
+
const int ms[8] = [0, 30, 45, 0, 0, 30, 0, 30];
|
|
8
|
+
|
|
9
|
+
if (resistance <= 1023) {
|
|
10
|
+
int t = resitance / 128;
|
|
11
|
+
alarm_h = hs[t];
|
|
12
|
+
alarm_m = hs[m];
|
|
13
|
+
}
|
|
14
|
+
```
|
|
15
|
+
|
|
16
|
+
もしくは
|
|
17
|
+
|
|
18
|
+
```c
|
|
6
19
|
if (resistance <= 128) {
|
|
7
20
|
alarm_h = 6;
|
|
8
21
|
alarm_m = 0;
|
|
@@ -18,4 +31,4 @@
|
|
|
18
31
|
}
|
|
19
32
|
```
|
|
20
33
|
|
|
21
|
-
あまり、簡潔になったとは言い難いし、可読性も落ちましたね。
|
|
34
|
+
こんなふうに構成し直しても速度に影響がでるのかわかりませんが、あまり、簡潔になったとは言い難いし、可読性も落ちましたね。
|