回答編集履歴
2
修正
test
CHANGED
|
@@ -3,9 +3,9 @@
|
|
|
3
3
|
- function(string) {
|
|
4
4
|
- return ` ${string.toLowerCase()}`;
|
|
5
5
|
- }
|
|
6
|
-
+ m => ` ${m.toLowerCase()}`
|
|
6
|
+
+ m => ` ${m.toLowerCase()}`
|
|
7
7
|
```
|
|
8
|
-
とするとかですかね。で、コードレビューで「コールバックの第1引数名が `m` だと分かりにくい（手を抜きすぎ）」みたいな指摘をされたら `match` に修正すると。少なくとも`string` よりも `match` のほうがよいかと思います。(MDNの[replaceの説明](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/String/replace#%E5%BC%95%E6%95%B0%E3%81%A8%E3%81%97%E3%81%A6%E3%81%AE%E9%96%A2%E6%95%B0%E3%81%AE%E6%8C%87%E5%AE%9A)でも第1引数名は `match` になってます）
|
|
8
|
+
とするとかですかね。で、コードレビューで「コールバックの第1引数名が `m` だと分かりにくい（手を抜きすぎ）」みたいな指摘をされたら `match` に修正すると。少なくとも`string` よりも `match` のほうがよいかと思います。(MDNの[replaceの説明](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/String/replace#%E5%BC%95%E6%95%B0%E3%81%A8%E3%81%97%E3%81%A6%E3%81%AE%E9%96%A2%E6%95%B0%E3%81%AE%E6%8C%87%E5%AE%9A)でも第1引数名は `match` になっています。）
|
|
9
9
|
|
|
10
10
|
|
|
11
11
|
|
1
修正
test
CHANGED
|
@@ -5,7 +5,7 @@
|
|
|
5
5
|
- }
|
|
6
6
|
+ m => ` ${m.toLowerCase()}`
|
|
7
7
|
```
|
|
8
|
-
とするとかですかね。で、コードレビューで「コールバックの第1引数名が `m` だと分かりにくい（手を抜きすぎ）」みたいな指摘をされたら `match` に修正すると。少なくとも`string` よりも `match` のほうがよいかと思います。(MDNの[replaceの説明](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/String/replace)でも第
|
|
8
|
+
とするとかですかね。で、コードレビューで「コールバックの第1引数名が `m` だと分かりにくい（手を抜きすぎ）」みたいな指摘をされたら `match` に修正すると。少なくとも`string` よりも `match` のほうがよいかと思います。(MDNの[replaceの説明](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/String/replace#%E5%BC%95%E6%95%B0%E3%81%A8%E3%81%97%E3%81%A6%E3%81%AE%E9%96%A2%E6%95%B0%E3%81%AE%E6%8C%87%E5%AE%9A)でも第1引数名は `match` になってます）
|
|
9
9
|
|
|
10
10
|
|
|
11
11
|
|