こんにちは、林です。

フリーランスのサーバサイドエンジニアとして案件に携わり、幾度もソースコードのリファクタをする機会に恵まれました。
今回はその中の一つを紹介し、クラス継承をすることでの改善案を提案させていただきます。

・課題
ある会社では商品データを扱っているが、扱う商品データは提携する各会社からデータ（APIやファイル等）が送られてくる。
各会社のデータフォーマットは異なっているため、データを整えDBに格納する必要がある。

・解決案
提携先の会社は今後増える可能性もあり、それぞれの提携先ごとにファイルのフォーマットも変更したいので当然実装も変える必要がある。
しかし、扱う商品データやデータの取り込みフローなどは共通となることが多いので、抽象クラスを作成して処理を共通化することをおススメする。


public abstract class AbstractImportData {
abstract public void parseData();
public void importData();
}
public class ImportDataCompanyA extends AbstractImportData {
public void parseData() {
// 実装します
}
}
public class ImportDataCompanyA extends AbstractImportData {
public void parseData() {
// 実装します
}
}


このようにすることで、共通化された処理とそれぞれの処理が分かれ、影響する範囲も分かりやすくなる。
コードのわかりやすさはそのままバグの発生を減らすことにもつながるので、ぜひ読みやすいコードを意識してほしい。

楽しいプログラムライフを。

The following two tabs change content below.

• この記事を書いた人
• 最新の記事

林 和幸

大学院卒業後、ベンチャー企業で4年WEB開発を行った後フリーランスとして独立。 1児のパパです。

最新記事 by 林 和幸 (全て見る)

• 契約打ち切りの話 - 2022年10月28日
• 最近の円安について - 2022年9月26日
• 案件の獲得 - 2022年8月23日