C++：重複自己的方法; DRY優化需要

Question

這個類的三種初始化方法非常非常非常相似。我想看看是否有方法將調用鏈接在一起，可能會涉及需要兩個參數的方法。謝謝。

AudioHandler.h

class AudioHandler { 

public: 
    static bool Initialize(const SoundLibrary& sl); 
    static bool Initialize(const Soundtrack& st); 
    static bool Initialize(const SoundLibrary& sl, const Soundtrack& st); 
    static void Release(); 
private: 
    static const SoundLibrary* _sl; 
    static const Soundtrack* _st; 
}; 

AudioHandler.cpp

bool AudioHandler::Initialize(const SoundLibrary& sl) { 
    if(_sl != NULL || _st != NULL) return false; 

    unsigned long numVoices = 0; 

    //If allegro is unable to initialize the sound drivers then return false. 
    if((numVoices = detect_digi_driver(DIGI_AUTODETECT)) == 0) return false; 
    if(install_sound(DIGI_AUTODETECT, MIDI_NONE, NULL) == -1) return false; 
    _sl = &sl; 

    return true; 
} 

bool AudioHandler::Initialize(const Soundtrack& st) { 
    if(_sl != NULL || _st != NULL) return false; 

    if(detect_midi_driver(MIDI_AUTODETECT) == 0) return false; 
    if(install_sound(DIGI_NONE, MIDI_AUTODETECT, NULL) == -1) return false; 
    _st = &st; 

    return true; 
} 

bool AudioHandler::Initialize(const SoundLibrary& sl, const Soundtrack& st) { 
    if(_sl != NULL || _st != NULL) return false; 

    unsigned long numVoices = 0; 

    if((numVoices = detect_digi_driver(DIGI_AUTODETECT)) == 0) return false; 
    if(detect_midi_driver(MIDI_AUTODETECT) == 0) return false; 
    if(install_sound(DIGI_AUTODETECT, MIDI_AUTODETECT, NULL) == -1) return false; 

    _sl = &sl; 
    _st = &st; 

    return true; 
} 

void AudioHandler::Release() { 
    _sl = NULL; 
    _st = NULL; 
    remove_sound(); 
} 

Answer 1

1. 僅限第3個變體。
2. 更改對指針的引用。
3. 發送NULL您不需要的參數。
4. 在做特定代碼之前檢查NULL。

PS：在您當前的版本中，在對象中存儲傳遞引用的地址不是一個好主意。

Answer 2

這些都是很短的功能，所以我懷疑這將是值得的時間和精力來修改代碼。但我可能也想這樣做。您可以創建另一個私有函數，其他三個函數調用某些參數來執行它們共同執行的所有操作。這樣常見的操作就在一個地方。這取決於你是否值得。如果你擔心將來會維護，那就去做吧。