ライブラリ内のグローバル変数を取り除く

Question

私はいくつかのクラス（プッシュボタン用、ポテンショメータ用、ロータリーエンコーダ用など）を持つMIDIコントローラライブラリで作業しています。これらすべてのクラスには、sendMIDI関数を使用するメソッドがあります。また、プログラムの最初に一度呼び出されるsetupMIDI関数もあります。 setupMIDIには、sendMIDI関数でアクセスする必要がある3つのパラメータがあります。クラスの

定義

class Analog { 
public: 
    Analog (...) { ... } 
    void refresh() { 
    ... 
    sendMIDI(x, y, z, ...); 
    ... 
    } 
} 

class Digital { ... (also uses sendMIDI) ... } 
etc. 

setupMIDIとsendMIDI機能

byte pin; 
int delayTime; 
bool debug; 

void setupMIDI (byte p, int d, bool db = false) { 
    ... 
    pin = p; delayTime = d; debug = db; 
    ... 
} 
void sendMIDI (...) { 
    ... 
    if(debug) ... 
    digitalWrite(pin, 1); 
    delay(delayTime); 
    ... 
} 

実際のプログラム：あなたが見ることができるように

Analog a1(...); 
Analog a2(...); 
Digital d1(...); 

setupMIDI(13, 10, true); 

while(true) { 
    a1.refresh(); // calls sendMIDI 
    a2.refresh(); 
    d1.refresh(); 
} 

、アナログおよびデジタルのクラスのすべてのインスタンスを使用します同じsendMIDI関数であり、これはsetupMIDI関数に入力された値に依存します。問題は、これらの値がグローバル変数に格納されていることです。これは理想的ではありません。

私はMidiSenderクラスの使用について考えましたが、そのインスタンスをすべてのアナログまたはデジタルコンストラクタに渡す必要があります。

これを回避するより良い方法はありますか？

ありがとうございます！代わりに、いくつかのグローバル変数を設定setupMIDI機能を有するの
ピーター

Answer 1

は、それらの値を格納し、そのコンストラクタを介してそれらを初期化しMIDIContextクラスを持っています。

pin

のユーザ、delayTime、及びdebugいずれか方法MIDIContextのであるか、または値にアクセスするMIDIContext&（const&適切な）を取ります。

例：

class MIDIContext 
{ 
private: 
    byte pin; 
    int delayTime; 
    bool debug; 

public: 
    MIDIContext(byte x_pin, int x_delayTime, bool x_debug) 
     : pin{x_pin}, delayTime{x_delayTime}, debug{x_debug} 
    { 
    } 

    void sendMIDI(); 
}; 

class Analog 
{ 
    void refresh (MIDIContext& ctx); 
}; 

可能な用法：

Analog a1(...); 
Analog a2(...); 
Digital d1(...); 

MIDIContext context(13, 10, true); 

while(true) { 
    a1.refresh(context); 
    a2.refresh(context); 
    d1.refresh(context); 
}