有遇到一個程式流程中 某些步驟可能要檢查或不檢查的問題 想請問該
怎麼寫比較好. 語言是使用C++
首先我有一個computing class, 裡面有個member function,
專門負責做計算的動作 而裡面有七個步驟:
class Engine::compute(...arguments...)
1. compute1
2. compute2
3. if checking fails this return pre-condition is not satisfied
4. compute3
5. foreach cofig in allowed_user_configurations
6 if checking succeed then store this configuration
7. return allowed_configuration
步驟3)跟步驟6)的檢查是由一個complianceChecker的class來做的:
bool compliance.check(....arguments....)
現在呢 為了一些原因 提供了一個選項 可以讓步驟3.)跟步驟6.)的檢查不做-也就是
永遠檢查成功, 請問要怎麼在不動到主流程的狀況下, 來達成這件事情呢?
因為這邊是簡化版本 但其實檢查的地方大概十多處....
全部都加if-else好像不是太好 如果考慮到未來在其他部分需要加檢查 或是未來
流程異動 這樣if-else很容易漏掉 或是忘記要加if-else
目前我的想法是 加一個wrapper class在compliance checker上 ctor吃一個要不要
做檢查的boolean, 然後有一個perfect forwarding的 member function取代掉
原本的 compliance checker的check function(轉call):
class wrapper {
public:
wrapper(bool doCheckIn): doCheck(doCheckIn) {}
template <typename ... Args>
bool check(Args&& ... args){
return !doCheck || checker.check(std::forward<Args>(args) ... );
}
private:
compliance checker
bool doCheck;
}
然後在第一行之前生成這個wrapper, 用這個wrapper取代掉步驟3.)跟步驟6.)的
compliance class:
class Engine::compute(...arguments...)
==> 0. wrapper newChecker(doCheckOrNot);
1. compute1
2. compute2
=> 3. if (!newChecker.check(...)) return the pre-condition is not satisfied
......................
5. foreach cofig in allowed_user_configurations
==> 6. if (newChecker.check(....)) then store this configuration
7. return allowed_configuration
目前這個做法有一些好處:
1. 不動到compliance checker, 而且也不用怕compliance checker的interfance of
check function會改變(用template+perfect forwarding處理掉了)
2. 整個流程不變 邏輯上也很分明 就是有檢查跟永遠成功這兩種
但是有些不順的地方:
1.這個wrapper class定位很尷尬, 不知道要當作一個放出來給大家用的class 或是該
跟誰綁在一起 綁在一起是指邏輯關係上.......現在wrapper class九成像syntax sugar
我直接放在CPP file的 anonymous的namespace裡 XDDDDD
未來 若別的地方遇到同一種狀況 是不是又要寫一次? 要不然就要把這個有template的
wrapper class放在header, 但這樣會引進compliance class的dependency.....
2. 要改寫成virtual function的形式嗎? 這樣可繼承之類的
但因為有用到template所以 沒法用virtual function, 而且只有兩種狀況
用virtual是否有效益呢?
3. wrapper class生成後 就決定他的行為了(由ctor的參數決定) 要動態變化就得再多
一組setter/getter. 想要動態變化的理由是 這樣wrapper可當作一個member variable..
不知道要當作local variable還是member variable比較好......
其實覺得這種狀況應該是一個非常標準且常見的狀況(二選一) 只是不曉得一般是怎麼
處理的 覺得應該是繼承+virtual function處理居多吧? 但個人比較傾向少用繼承
先嘗試template + 組合的方式, 或是virtual function是做在別的地方的方式
P.S. compliance checker是我們動不了的一個class.......
在此 想請各位給一些想法跟建議~~~謝謝~~~