回答編集履歴
4
不要な改善ポイントの削除
test
CHANGED
|
@@ -94,8 +94,6 @@
|
|
|
94
94
|
|
|
95
95
|
- `http.Get` のレスポンスのステータスコードが 200 なのかどうか確認する
|
|
96
96
|
|
|
97
|
-
- `scannner.Scan` でファイルの末尾まで読み込めたがどうかチェックするために `scanner.Err()` をチェックする
|
|
98
|
-
|
|
99
97
|
- for の中での defer を避ける(`readAge()` から return しないとクローズされないため)
|
|
100
98
|
|
|
101
99
|
- readAge() で呼び出し元に error を返すようにシグネチャを修正する
|
3
Typo
test
CHANGED
|
@@ -1,8 +1,8 @@
|
|
|
1
|
-
`scanner.Scan()` でファイルの末尾まで読み込んで false になるまでループがまわります。`Scanner.Text()` は読み込んだ文字列を取得しますが、空行を取り除くことはしません。デフォルトでは文字列の末尾に `\n` は含まれないため `strings.TrimRight(logging, "\n")` は不要です。
|
|
1
|
+
`scanner.Scan()` でファイルの末尾まで読み込んで false になるまでループがまわります。`Scanner.Text()` は読み込んだ文字列を取得しますが、空白行を取り除くことはしません。デフォルトでは文字列の末尾に `\n` は含まれないため `strings.TrimRight(logging, "\n")` は不要です。
|
|
2
2
|
|
|
3
3
|
|
|
4
4
|
|
|
5
|
-
さて、空白行の削除ですが、空行 `""` を除くには例えば以下のように文字列の長さをチェックして処理しないなどの実装が必要で
|
|
5
|
+
さて、空白行の削除ですが、空白行 `""` を除くには例えば以下のように文字列の長さをチェックして処理しないなどの実装が必要です。
|
|
6
6
|
|
|
7
7
|
|
|
8
8
|
|
2
Typo
test
CHANGED
|
@@ -18,7 +18,7 @@
|
|
|
18
18
|
|
|
19
19
|
|
|
20
20
|
|
|
21
|
-
少し改善した実装です。`fmt.Sprint(scanner.Text())` は `scanner.Text()` で問題
|
|
21
|
+
少し改善した実装です。`fmt.Sprint(scanner.Text())` は `scanner.Text()` で問題ありません。
|
|
22
22
|
|
|
23
23
|
|
|
24
24
|
|
1
fmt.Sprint(scanner.Text())が気になったため、実装を修正しました
test
CHANGED
|
@@ -18,7 +18,7 @@
|
|
|
18
18
|
|
|
19
19
|
|
|
20
20
|
|
|
21
|
-
少し改善した実装です。
|
|
21
|
+
少し改善した実装です。`fmt.Sprint(scanner.Text())` は `scanner.Text()` で問題ないありません。
|
|
22
22
|
|
|
23
23
|
|
|
24
24
|
|
|
@@ -46,7 +46,7 @@
|
|
|
46
46
|
|
|
47
47
|
for scanner.Scan() {
|
|
48
48
|
|
|
49
|
-
result :=
|
|
49
|
+
result := scanner.Text()
|
|
50
50
|
|
|
51
51
|
//fmt.Println(logging)
|
|
52
52
|
|