Проблема проектирования класса C++

Question

У меня есть класс, который выполняет протокол MSNP15. Протокол требует от клиентов частое подключение/отключение к различным серверам, таким как сервер отправки, сервер входа и сервер коммутатора.

я решил хранить переменные протокола по связанных (например, жетонов билетов, Nonce и т.д.) в качестве статических переменных-членов в полезности класса, как показано ниже:

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; 
    }; 

Статические переменные выше инициализируется NULL при запуске, а затем когда токены становятся доступными по мере выполнения протокола.

Поскольку у меня нет доступа к стандартной библиотеке C++ (поскольку я разрабатываю платформу Symbian S60), я не могу использовать библиотеку строк. Выделите выделенные указатели символов, когда программа выйдет или есть какой-либо другой механизм, благодаря которому я мог бы обеспечить их освобождение.

Я также открыт для альтернативных предложений по дизайну.

Answer 1

Если вы выделите память, то вы,, должны освободить его. Поскольку члены статичны, они не принадлежат ни одному экземпляру класса. Поэтому вам нужно будет обеспечить освобождение памяти после последнего использования указателей на символы. Это часто очень сложно определить.

Я думаю, что лучшей идеей в этом случае было бы иметь одноэлементный класс со всеми необходимыми токенами. Сделайте этот класс доступным по всему миру, предоставите необходимые сеттеры \ getters для манипуляции с токеном. Затем, когда dtor одноэлементного класса вызывается при выходе из программы, вы можете де-alllocate память.

Что-то на следующих линиях: -

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

Зачем нужен класс, если все переменные являются статическими? Если вы хотите только один экземпляр класса, подумайте о том, чтобы сделать его одиночным. Затем вы можете использовать конструктор и деструктор для управления памятью. Я заново спроектировал класс как одноточечная:

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 проект, он еще находится на ранних стадиях, но у него есть библиотека богатого класса, которая также включает интерфейс.