回答編集履歴

4

不要な改善ポイントの削除

2020/07/04 13:38

投稿

d_tutuz
d_tutuz

スコア730

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

2020/07/04 13:38

投稿

d_tutuz
d_tutuz

スコア730

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

2020/07/04 13:34

投稿

d_tutuz
d_tutuz

スコア730

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())が気になったため、実装を修正しました

2020/07/04 13:25

投稿

d_tutuz
d_tutuz

スコア730

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 := fmt.Sprint(scanner.Text())
49
+ result := scanner.Text()
50
50
 
51
51
  //fmt.Println(logging)
52
52