Udemy講座「リーダブルコード：C#で読みやすいコードを書く50の方法」を修了する - Githubissues

ohshimatani / groupA_AcceLance

2 stars 0 forks source link

Udemy講座「リーダブルコード：C#で読みやすいコードを書く50の方法」を修了する #38

Closed Parasuatene closed 2 years ago

Parasuatene commented 2 years ago

講座URL

https://benesse.udemy.com/course/readable_code/

学習内容

C#のリーダブルコードについて解説した講座。特に3つの観点でコードの読みやすさを開設している。

BAD：良くない書き方
NOTBAD：別に悪くはない
GOOD：良い書き方、推奨

学習目的

「良いコードとは何か」の理解を深め、コーディング能力およびコードレビュー能力を高めることで、個人として、チームとしてのプロダクト品質を向上させる。

Parasuatene commented 2 years ago

7/22 Fri.

何行で書くかではなく、何秒で読めるかが重要
"."で繋ぐ数は隣のとなりまで

Parasuatene commented 2 years ago

7/25 Mon.

If-Else文は肯定形（True）の時に中に入るようにすること
- ただし、Ifの時はTrueである必要はない
比較するときは比較する値を左にすること
複数の比較を1回で行わないこと
- ただし、ネストが深くなるので考え物ではある気がする
```
// BADな例
if (valueA == 1 && valueB == 1) {
// 処理
}
```

// GOODな例 if (valueA == 1) { if (valueB == 1) { // 処理 } }


- 否定の否定は書かないこと
  - ```flag != false```のような書き方は特にダメ！
- メソッドにおいて、返却する値を無駄に変数に入れないこと
- privateな変数の頭文字にはアンダーバーを付けることもあるらしい
  - チームによるので、今回は使わない
- メソッド内の変数を最初に全部宣言しないこと

Parasuatene commented 2 years ago

メソッド内の変数は直前で定義する
ループの変数はループ内で宣言する
変数の使いまわしはNG
- 影響が及ばないからOK。ではなく、影響がないことを調査する必要があるのがNG。
bool型はどちらがTrueかを分かるようにする
- 例. IsExist() // Trueの場合は存在する
コメントアウトでTODO:を書いておくと、表示→タスク一覧から残作業が表示できる

Parasuatene commented 2 years ago

修了したのでCloseする