疑ってみることも必要

重複する部分のコードがあったので、リファクタリングをしていると、メソッドの使用方法がおかしかったり、if文の処理がネストしている部分があるなど、かなり修正する必要がある部分を見つけました。それ以外にもテストコードがない部分がありました。

初級のプログラマが作成したコードだったので、その場で注意をしました。

もくもくとマジメに作業をする子ですが、もう少し色々と考えて欲しい部分があるなと思いました。

常に読みやすいコードにするためには、どうするか？自分が書いている処理は違和感がないかなどを考えながら作業をしなくては、上記のようなコードになってしまいます。

中々難しいことですが、様々なコードに触れてよく考えることで、プログラミングスキルは上達します。

まずは自分のコードを疑ってみるのもいいのではないかと思います。