保守性

はじめに こんにちは！コインチェック株式会社（以下、コインチェック）ディーリングインフラグループの梅下です。 ディーリングインフラグループでは、コインチェック自身が保有する暗号資産の残高管理やそのための外部取引所への注文管理のシステム（ディーリングシステム）を開発しています。 暗号資産業界は通貨の多様化など変化が激しく、また暗号資産のETF承認や法整備が進んでいることもあり、より一層金融的側面が強まっています。コインチェックとしてもNASDAQ市場への上場を行い、よりスピーディな成長を狙っていきます。そこで、システムの堅牢性や変化に素早く適応できるものを作ることが重要となっています。 私のグル…

概要 コードの保守性を良くするための内部設計というか、コードの書き方のテクニックを一つ紹介します。ここで言う保守性は、変更の容易さや影響の少なさです。 そういう話については、体系的かつ汎用的な法則としてまとめてある本は良著がいくつもあります。ここでは違ったアプローチとして、実務の中でよく見かけるケースについて、対象を絞って、できるだけ具体例を付けて紹介しようと思います。分厚い設計の本を読むのは辛いが、エッセンスだけでも1つずつ読めたら頭に入ってくるかも、という人などへ役に立ったら良いなと思います。 最初にポイントまとめ クラスのメソッドが「引数で指示されないと何もできないAPI」になっていて、…

エンジニアとして仕事をしていると、「処理を共通化しなさい」という言葉をよく耳にすることがあると思います。 確かに、共通化することで保守性や再利用性が向上するなど、多くのメリットがあるのは間違いありません。 しかし、なんでもかんでも闇雲に「共通化」することが必ずしも良いとは限らないのではないかと考えています。そこで、今回はその点について少し考えを整理してみました。 過度な共通化の問題点 全く同じ処理を共通化すること自体には、特に問題はないと思います。 問題が生じるのは、似ている処理を無理に共通化しようとする場合です。 似ている処理を共通化する際、どうしても異なる箇所を含めなければならなくなります…

ソースコードの複雑性を表す基準として、循環的複雑度という指標があります。この循環的複雑度はどれくらいの設定値にするのが適切なのかをまとめておこうと思います。 循環的複雑度とは 循環的複雑度はThomas J. McCabe氏によって1976年に考案された指標です。この値はソースコードの複雑性を示すものであり、この値を測定し、低い値を保つことで、ソースコードの保守性を高い状態で維持することができます。 循環的複雑度の目安 循環的複雑度の基準は、対象としているソフトウェアの複雑性や、ドメインの複雑性などに左右されます。そのため、絶対の数値というものはありませんが、書籍などで紹介されているものの一部…

「火星の自律走行車」は、かくして完成した：NASAが打ち上げる火星探査機「パーサヴィアランス」ができるまで @wired_jp https://wired.jp/2020/07/26/how-nasa-built-a-self-driving-car-for-its-next-mars-mission/… 「パーサヴィアランスのメインコンピューターは、1997年製の高性能PCとほぼ同じくらいの性能にとどまる。」というよりもとどめてある。 それは、絶対に動かなければならない部分には、何十年も、様々な場面で使われて、バグが無くなっていると期待できるものを使うという判断によるものだ。枯れた技術をバカ…