はじめに タブやスペースの数(インデント)とか変数名のセンスだけを指摘してコードレビューしたぜ!となっていいのは小学生までと言われています[要出典]。このエントリでは主にScalaをターゲットにして、どういうところに気をつけると効果的にレビューできるのか?について僕の個人的な経験に基づきつつ紹介しようと思います。 こういうコードがでてきたら、これは慎重に見るべきポイントだな!というのを紹介していきます。コードの全てのを見ていくのは非常に大変なので、こういう気になるところだけ見るような形にしています。また、このエントリではサンプルコードとしてScalaを利用してます。 型Aをとって型Bを返す関数 こう言われるとやや謎かもしれないですが、たとえば次のような関数です。 def toInt(str: String): Int = ??? // 実装があるとする こういう関数があったら、僕は脳死で次
![コードレビューで気をつけているいくつかのこと | Recruit Tech Blog](https://cdn-ak-scissors.b.st-hatena.com/image/square/256e9fb4171137bc8f12735b0bdca05a6d680644/height=288;version=1;width=512/https%3A%2F%2Ftechblog.recruit.co.jp%2Fassets%2Frmp%2Fthumbnail%2F19281.jpg)