前回書きました通り、リーダブルコードを書く上で大切だなと思ったことを紹介します。―より良いコードを書くためのシンプルで実践的なテクニック今回はList.Add()のパフォーマンスについても少し出ていますので、List.Add()といえばあれか?という考えがある前提とします。無駄なものを削除する無駄なコードはいわばプログラムのぜい肉です。例をあげるとキリがないとは思いますが幾つか・・・。見かけるものとしては下記ですかね。パフォーマンスを無駄に喰っていたり、ソースコードの行数が無駄に多かったりするところに潜んでいます。いうコードはただただ見通しが悪くなるだけなのでどんどん排除します。あと、バージョン管理しておけば遠慮なく削除できますね。定義位置もう、言わずもがなな事かもしれませんが、変数などの定義位置は必要になる箇所の直前で定義すると良いです。すべきだと私は思っていますよ?全てを最初に定義するのはもう古いですね。私自身、学生の時にまれに見かけました。(読みにくいことこの上なかったです。昔々のC言語では最初に定義しないとエラーになる言語仕様があり、その時の時代背景が影響しているのだとか。現代のコンパイラではどこで定義してもコンパイルが通るように進化してくれたので昔ながらの「定義位置の枷」が外れたわけですね。(正確には今でもエラーになるコンパイラは存在するようですが。)最初に全てを定義すれば同時に記憶することも増えるので、精神衛生上よくないです。書きましたが、いわゆるメソッド抽出です。役割に関係のない汎用的に使えるだろうコードブロックはユーティリティとして抽出してあげれば処理が分離できますし、見通しもよくなります。汎用的なものとしては役割とは別の処理を3回コピーしていれば該当すると思っています。いえ私自身なかなかリファクタリングしてあげられていないので心境複雑です・・・。いった汎用コードの抽出は後々、別の開発でも使えるし、かつ、開発効率の上昇にも繋がります。まとめ今回は無駄の排除、変数やメソッドの定義位置、実践的な問題抽出を紹介しました。自分自身この辺りの内容は、普段から気をつけるようにしています。この手の内容で陥りやすいのが、自分には読みやすいという点ではないかと思います。有効な改善手段はコードレビューだと思っていますので、不安なコードはいろんな人に見てもらいましょう。それでは今回はここまで。採用情報クラウドクリエイティブスタジオではエンジニアの方を絶賛募集中です。