Windows Embedded CE 6 msmqadm工具的一個bug

最近繼續使用C++做MSMQ的開發，最簡單的方法就是看微軟的原始碼，在做MSMQ管理部分，參考reg.cxx，發現了一個bug，其實這個bug沒什麼大不了，為了證明自己“眼光敏捷”，吹毛求疵的把這個bug報告一下。

 

這bug在reg.cxx檔案的273行，如果執行msmqadm register install 的時候，寫註冊表失敗就會進入這段代碼。我估計這個bug的出現是因為拷貝複製導致的，因為前後的代碼都非常類似，如果把這段代碼封裝起來，那麼就不會出現這個bug了。

dwValue = 1;
if (ERROR_SUCCESS != (hr = RegSetValueEx (hKey, L"Index", 0, REG_DWORD, (BYTE *)&dwValue, sizeof(DWORD)))) {
    RegCloseKey (hKey);
    rsc_wprintf (IDS_CANT_WRITE_KEY, L"Keep", hr);
return 1;
}

應該改為

dwValue = 1;
if (ERROR_SUCCESS != (hr = RegSetValueEx (hKey, L"Index", 0, REG_DWORD, (BYTE *)&dwValue, sizeof(DWORD)))) {
    RegCloseKey (hKey);
    rsc_wprintf (IDS_CANT_WRITE_KEY, L"Index", hr);
return 1;
}

 

我想這段代碼沒有走Unit Test，至少沒有做寫註冊表失敗的Unit Test。

其實Unit Test全面覆蓋真的很難，最近自己也寫很多Test cases，寫的代碼中，有一半為測試代碼。其實還遠遠不夠，因為要覆蓋全部異常情況，測試代碼應該是功能代碼的好幾倍。任重而道遠呀。