Boolean に潰された「状態」

Wed, 04 Mar 2026 12:00:00 +0900

Boolean に潰された「成功」 ― リファクタリングで気づいた抽象化の罠

Boolean に潰された「成功」 ― リファクタリングで気づいた抽象化の罠

はじめに

ViewModel の保存処理をリファクタリングしていたとき、私は一度コードを壊しました。

原因は単純な「うっかりミス」に見えました。

しかし振り返ってみると、問題はもっと深いところにありました。

それは、

「成功」という概念を Boolean に潰してしまったこと

でした。

この記事では、その過程と学びを書きます。

元のコード

保存処理は、新規作成と更新で分岐していました。

private fun saveData() {
 viewModelScope.launch {
 val isExistingItem: Boolean = (取得)

 if (destId == 0) {
 // 新規作成
 val resultSuccess = directDebitDefRepo.createDestination(...)

 if (resultSuccess) {
 // フォームの初期化
 _formInputState.update { FormInputState() }
 showSuccess()
 } else {
 showFailure()
 }

 } else {
 // 更新
 val resultSuccess = directDebitDefRepo.updateDestination(...)

 if (resultSuccess) {
 showSuccess()
 } else {
 showFailure()
 }
 }
 }
}

ポイントはここです。

State on 設計で、迷わなくなるために | 奥田智紘

Boolean に潰された「状態」

Boolean に潰された「成功」 ― リファクタリングで気づいた抽象化の罠

はじめに

元のコード