Skip to content

Commit 77f1867

Browse files
author
Misaki Shibata
authored
Update README.md
1 parent 7830fee commit 77f1867

File tree

1 file changed

+6
-19
lines changed

1 file changed

+6
-19
lines changed

README.md

Lines changed: 6 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,8 @@ https://github.com/misak1/clean-code-javascript
3333
Clean Codeにヒントを得て、有志によってつくられた
3434
JavaScriptに対するコーディング・ガイドになります。
3535
<a href="https://www.amazon.co.jp/dp/B001GSTOAM/ref=dp-kindle-redirect?_encoding=UTF8&amp;btkr=1" taget="_blank">Clean Code</a>は著者のRobert C. Martinらが長年の経験をもとにまとめたものです。
36-
こちらの本は言語は限定されていません。
37-
36+
こちらの本は言語は限定されていません。
37+
3838
キレイなコードは3R(Readable、Reusable、Refactorable)になっていることを指し
3939
読みやすい、再利用可能、理解や修正がしやすいように整理されている
4040
これらを目指すためのガイドになります。
@@ -228,7 +228,6 @@ function createMicrobrewery(breweryName = 'Hipster Brew Co.') {
228228
ただ、IEとかモバイルサファリで使えない場合があるので、生のjsを扱う場合は注意です。
229229

230230
## **Functions**
231-
### Function arguments (2 or fewer ideally)
232231
### 関数の引数(2つ以下が理想的)
233232

234233
関数の引数の数を制限することは、テストを簡単に行えるという点において非常に重要なことです。
@@ -262,9 +261,6 @@ createMenu({
262261
});
263262
```
264263

265-
266-
267-
### Functions should do one thing
268264
### 関数は1つのことを行うこと
269265

270266
これはとても重要なルールです。
@@ -297,7 +293,6 @@ function isActiveClient(client) {
297293
```
298294

299295

300-
### Function names should say what they do
301296
### 関数名は何をするかを表すこと
302297

303298
**Bad:**
@@ -323,8 +318,6 @@ const date = new Date();
323318
addMonthToDate(1, date);
324319
```
325320

326-
327-
### Remove duplicate code
328321
### 重複したコードを削除すること
329322

330323
重複したコードを避けるために絶対にベストを尽くしてください。
@@ -391,12 +384,9 @@ function showEmployeeList(employees) {
391384
}
392385
```
393386

394-
サンプルはリファクタリングとしてはいい例なの、
395-
最悪なのはコピペコードが複数箇所に点在していることです
387+
<img src="https://github.com/misak1/clean-code-javascript/blob/master/point-50.png?raw=true" width="20"/> サンプルはリファクタリングとしてはいい例だと思います、
388+
最悪なのはコピペされたコードが複数箇所に点在していることです
396389

397-
398-
399-
### Set default objects with Object.assign
400390
### Object.assignでデフォルトオブジェクトを設定すること
401391

402392
**Bad:**
@@ -444,14 +434,11 @@ function createMenu(config) {
444434
createMenu(menuConfig);
445435
```
446436

447-
似ているObject.assignと分割代入構文
448-
Object.assignはオブジェクトのテンプレートを定義すること、
437+
<img src="https://github.com/misak1/clean-code-javascript/blob/master/point-50.png?raw=true" width="20"/> 似ているObject.assignと分割代入構文
438+
Object.assignはオブジェクトのテンプレートを定義することに対し
449439
分割代入構文はその場でオブジェクトを作ること
450440

451441

452-
453-
454-
### Don't use flags as function parameters
455442
### フラグを関数の引数のように利用しない
456443

457444
フラグは、この関数が複数のことを行うことを利用者に伝えます。

0 commit comments

Comments
 (0)