C++クラスの設計上の問題

Question

私はMSNP15プロトコルを実行するクラスを持っています。このプロトコルでは、クライアントは、ディスパッチサーバー、ログインサーバー、およびスイッチボードサーバーなどのさまざまなサーバーへの頻繁な接続/切断を実行する必要があります。

Iは、以下のようなユーティリティクラスの静的メンバ変数として（チケットトークンなど、ナンスなど）プロトコルに関連する変数を格納することを決定した：次いで、上記の静的変数は、起動時にNULLに初期化され

class MsnUtility 
    { 
    public: 
     static void SetChallengeStringL (const char *string); 

     static const char* GetChallengeString (); 

     static void SetContactTicketL (const char *ticket); 

     static const char* GetContactTicket (); 

    private: 

     MsnUtility(); 

    static char *iChallengeString; 

     static char *iContactTicket; 
    }; 

とプロトコルが実行されるときにトークンが利用可能になったときに新しくされました。

私はC++標準ライブラリ（Symbian S60プラットフォームで開発中）にアクセスできないため、文字列ライブラリは使用できません。割り当てられた文字ポインタは、プログラムが終了したときに解放されるか、解放されることを確実にするための他のメカニズムがありますか？

代替デザインの提案もあります。

Answer 1

あなたがメモリを割り当てる場合、あなたはそれを解放する必要があります一つです。メンバは静的なので、クラスのインスタンスには属しません。したがって、文字ポインタの最後の使用後にメモリが解放されるようにする必要があります。これはしばしば決定することが非常に困難です。

このケースでは、必要なすべてのトークンを持つシングルトンクラスを持つ方がよいと思います。このクラスをグローバルにアクセス可能にし、トークン操作に必要なsetters \ getterを提供します。次に、プログラム終了時にシングルトンクラスのdtorが呼び出されると、メモリの割り当てを解除することができます。次の行に

何か： - それは、基本的にすべてのインスタンスがします -

class TokenDict { 
public: 
    static TokenDict& instance() { 
    static TokenDict instance; 
    return instance; 
    } 
    // getters \ setters for tokens 
    void setToken(char* tptr) { 
    if(token1) 
     delete[] token1; 
    // allocate memory for new token here 
    token1 = new char[strlen(tptr) + 1]; 
    // copy tptr into token1 
    } 
    char const* getToken() const { return token1; } 
private: 
    ~TokenDict() 
    { 
    delete[] token1; 
    } 
    TokenDict() : token1(0) // ctor hidden 
    { } 
    TokenDict(TokenDict const&); // copy ctor hidden 
    TokenDict& operator=(TokenDict const&); // assign op. hidden 
    char* token1; 
}; 

Answer 2

終了後にリークを心配する必要はないと思います。 また、プログラムのライフタイム中にリークを考えている場合は、参照カウントでシングルトンを実行してください。

Answer 3

あなたはクラスMsnUtilityの複数のインスタンスを使用するつもりなら、それの一部だとしてフィールドが静的であることを、予めご了承ください同じである。これはシングルトンと呼ばれ、おそらくあなたが行っていたものではありません。

Answer 4

すべての変数が静的である場合、なぜクラスが必要ですか？クラスのインスタンスを1つしか必要としない場合は、それをシングルトンにすることを検討してください。次に、コンストラクタとデストラクタを使用してメモリを管理することができます。私はシングルトンとして、クラス設計し直す：

class MsnUtility 
{ 
public: 
    static MsnUtility& Instance() 
    { 
    return instance; 
    } 
public: 
    void SetChallengeStringL(const char* cstring) 
    { 
    if (iChallengeString) delete[] iChallengeString; 
    iChallengeString = new char[strlen(cstring) + 1]; 
    strcpy(iChallengeString, cstring); 
    } 
    const char* GetChallengeString() const 
    { 
    return iChallengeString; 
    } 
    void SetContactTicketL(const char* ticket) 
    { 
    if (iContactTicket) delete[] iContactTicket; 
    iContactTicket = new char[strlen(ticket) + 1]; 
    strcpy(iContactTicket, ticket); 
    } 
    const char* GetContactTicket() const 
    { 
    return iContactTicket; 
    } 
private: 
    MsnUtility() : iChallengeString(0), iContactTicket(0) { } 
    ~MsnUtility() 
    { 
    if (iChallengeString) delete[] iChallengeString; 
    if (iContactTicket) delete[] iContactTicket; 
    } 
    char* iChallengeString; 
    char* iContactTicket; 
    static MsnUtility instance; 
}; 

あなたはこのようなクラスを使用することができます：

// test.cpp 

#include <iostream> // or whatever is available on your development platform. 
#include "MsnUtility.h" 

MsnUtility MsnUtility::instance; 

int 
main() 
{ 
    MsnUtility& u = MsnUtility::Instance(); 
    u.SetContactTicketL("ticket"); 
    u.SetChallengeStringL("ch"); 
    std::cout << u.GetContactTicket() << ", " << u.GetChallengeString() << '\n'; 
    return 0; 
} 

Answer 5

私は、あなたがQt Towerプロジェクトに見ることができると思い、それはまだ初期段階にあるが、それは持っていますUIを含む豊富なクラスライブラリ。