C API

Question

için bir iş parçacığı güvenli C++ sarmalayıcısı yazmak için çaba harcamamıştım. API, kendiliğinden iş parçacığı için güvenli DEĞİL ise, C API 'thread-safe' C++ sarmalayıcı yazıyorum. RAII'yi kullanmaya çalıştım.

Bilmek istiyorum, uygulamam doğru mu? ve iş parçacığı güvenli olsun. Kodumdaki tüm yorumları takdir ediyorum. Şimdiden teşekkürler! aşağıdaki gibi

C API'sıdır sarılacak olan aşağıdaki şekilde sargının

/* an data structure which represents a connection proxy to the logger: */ 
struct cLog_Logger; 

/* connect the logger, and returns a handle to it: */ 
cLog_Logger* cLog_connect(); 

/* appends a zero terminated string to the log: */ 
void cLog_write(cLog_Logger* logger, const char* message); 

/* closes the connection with the logger: */ 
void cLog_close(cLog_Logger* logger); 

Benim uygulamasıdır:

class LoggerWrapper{ 

    public: 
     LoggerWrapper(){    //constructor 
      cLog= cLog_connect(); 
     } 

     void log(const std::string &message){ //entry point 
      cLog_write(cLog, message); 
      cLog_close(cLog); 
      } 

     ~LoggerWrapper(){  //destructor 
      delete cLog; 
     } 
    protected: 
     cLog_Logger *cLog; 
} 

Teşekkür! Bu veriyor

class LoggerWrapper{ 

public: 
    LoggerWrapper(){    //constructor 
     cLog= cLog_connect(); 
    } 

    void log(const std::string &message){ //entry point 
     cLog_write(cLog, message); 
     } 

    ~LoggerWrapper(){  //destructor 
     cLog_close(cLog); 
     delete cLog; 
    } 
protected: 
    cLog_Logger *cLog; 
} ; 

böyle kod yazmak:

Answer 1

Ben böyle uygulamasını değiştirmeye gerek

LoggerWrapper logger ; 
logger.log("Something") ; 
logger.log("Something else) ; 

nedenle aynı nesne ile birden fazla günlüğü yapmak; aksi halde ilk çağrı kaydediciyi kapatır ve nesne işe yaramaz. İstediğin bu mu? O halde ikinci soru var: İplik güvenliği ile ne demek istiyorsun? Farklı nesnelerden aynı nesnede günlükler yapmak ister misiniz?

Sonra böyle bir mutex ve günlük işlevi içinde bir kilit bekçi ekleyebilirsiniz:

class LoggerWrapper{ 

public: 
    LoggerWrapper(){    //constructor 
     cLog= cLog_connect(); 
    } 

    void log(const std::string &message){ //entry point 
     std::lock_guard<std::mutex> guard(mutex); 
     cLog_write(cLog, message); 
     } 

    ~LoggerWrapper(){  //destructor 
     cLog_close(cLog); 
     delete cLog; 
    } 
protected: 
    cLog_Logger *cLog; 
    std::mutex mutex ; 
} ; 

Answer 2

Kısa cevap: hayır değil. Öncelikle, delete yerine free() olmalıdır, çünkü c api. Yaptığınız şey, kaynak sızıntısız programınızı verebilir, ancak iş parçacığı güvenli değildir. RAII, kaynak sızıntısından kaçınmak içindir. RAII sınıfınıza statik muteks eklemek için API'nizi iş parçacığı için güvenli bir şekilde sarma etmenin basit ama etkili bir yolu yoktur.

#include <mutex> 

class LoggerWrapper{ 

public: 
    LoggerWrapper() : l(globalLock); 
    {    //constructor 
     cLog= cLog_connect(); 
    } 

    void log(const std::string &message){ //entry point 
     cLog_write(cLog, message); 
     cLog_close(cLog); 
     } 

    ~LoggerWrapper(){  //destructor 
     free(cLog); // I think here must be free(), but not sure 
    } 
protected: 
    cLog_Logger *cLog; 
    static std::mutex globalLock; 
    std::lock_guard<std::mutex> l; 
} 

std::mutex LoggerWrapper::globalLock;