XLL功能的线程安全

Thread-safety of xll functions

本文关键字:安全 线程 功能 XLL      更新时间:2023-10-16

我有这个愚蠢的xll函数:

__declspec(dllexport) long F(long arg1, long arg2)
{
    return arg1 + arg2;
}

我猜这没有问题,由此产生的Excel功能将是线程安全。

但是,一个功能数组和返回数组呢?例如,例如该功能:

__declspec(dllexport) LPXLOPER12 WINAPI G(LPXLOPER12 arrayin1, LPXLOPER12 arrayin2)
{
    static XLOPER12 xlArray; // the reference to this one is going to be returned
    // treating arrayin1 and arrayin2 and outputting something in xlArray
    return static_cast<LPXLOPER12>(&xlArray);
}

显然,static不适合螺纹安全和Excel中的同时调用。我应该如何修改我的功能以确保线程安全?

编辑

使用句柄的代码示例:

__declspec(dllexport) LPXLOPER12 WINAPI CreateComplex(LPXLOPER12 arrayin1, LPXLOPER12 arrayin2)
{
    static XLOPER12 xlArray;
    xlArray.xltype = xltypeStr;
    if (arrayin1->xltype != xltypeNum)
    {
        xlArray.val.str = L"blah";
        return static_cast<LPXLOPER12>(&xlArray);
    }
    if (arrayin2->xltype != xltypeNum)
    {
        xlArray.val.str = L"blahblah";
        return static_cast<LPXLOPER12>(&xlArray);
    }
    double real = arrayin1->val.num;
    double imag = arrayin2->val.num;
    Complex * z = new Complex(real, imag);
    char * handle = StoreObject("Complex", z);
    xlArray.xltype = xltypeStr;
    FromCharPtrToWChartPtr(handle, &xlArray.val.str);
    return static_cast<LPXLOPER12>(&xlArray);
}
__declspec(dllexport) double WINAPI GetComplexNorm(LPXLOPER12 arrayin)
{
    char *handle = nullptr;
    FromWChartPtrToWharPtr(arrayin->val.str, &handle);
    Complex * z = (Complex*)RetrieveObject(handle);
    double res = z->getNorm();
    return res;
}

Complex是一个经典的复数类别,"内存"功能如下(当心它是旧的C代码(:

#include "stdafx.h"
#include <string.h>
#include "memory.h"
#include <stdio.h>
#include "XLCALL.H"
#include <cstdlib>
void FromCharPtrToWChartPtr(char * from, XCHAR **to)
{
    size_t len = strlen(from);
    *to = new XCHAR[len + 1];
    *to[0] = static_cast<XCHAR>(len);
    if (len > 0)
    {
        mbstowcs(*to + 1, from, len + 1);
    }
}
void FromWChartPtrToWharPtr(XCHAR * from, char **to)
{
    size_t len = from[0];
    *to = new char[len + 1];
    wcstombs(*to, from + 1, len);
}
typedef struct _TObject
{
    char *name;
    int   version; /* increments by 1 for every new store operation */
    void *data;
    void *next;
} TObject;

static TObject *cache = NULL;
#define SEPARATOR '#'
TObject* FindNode(char* name)
{
    TObject *node = cache;
    while (node)
    {
        if (_stricmp(node->name, name) == 0)
            break;
        node = (TObject*)node->next;
    }
    return node;
}
#define FAILURE -1
#define SUCCESS 0
char* StoreObject(char* name, void* data)
{
    static char *routine = "StoreObject";
    int          status = FAILURE;
    char        *handle = NULL;
    TObject     *node;
    static char buffer[255];
    if (data == NULL)
    {
        // error to handle later
        goto done;
    }
    if (name == NULL)
    {
        // error to handle later
        goto done;
    }
    if (strlen(name) > 200)
    {
        // error to handle later
        goto done;
    }
    node = FindNode(name);
    if (node == NULL)
    {
        node = new TObject();
        if (node == NULL)
            goto done;
        node->name = _strdup(name);
        node->version = 1;
        node->data = data;
        node->next = cache;
        cache = node;
    }
    else
    {
        node->version += 1;
        delete node->data; // should I template taylor the object destruction diffenrently ?
        node->data = data;
    }
    sprintf(buffer, "%s%c%d", node->name, SEPARATOR, node->version);
    handle = _strdup(buffer);
    if (handle == NULL)
        goto done;
    strcpy(handle, buffer);
    status = SUCCESS;
done:
    if (status != SUCCESS)
    {
        // error to handle later
        delete handle;
        handle = NULL;
    }
    return handle;
}
void* RetrieveObject(char* handle)
{
    static char *routine = "RetrieveObject";
    int          status = FAILURE;
    void        *data = NULL;
    char        *name = NULL;
    TObject     *node;
    char        *sep;
    if (handle == NULL)
    {
        // error to handle later
        goto done;
    }
    name = _strdup(handle);
    if (name == NULL)
        goto done;
    /* remove version number from lookup string */
    sep = strchr(name, SEPARATOR);
    if (sep != NULL)
        *sep = '';
    node = FindNode(name);
    if (node == NULL)
    {
        // error to handle later
        goto done;
    }
    data = node->data;
    status = SUCCESS;
done:
    if (status != SUCCESS)
    {
        // error to handle later
        data = NULL;
    }
    delete name;
    return data;
}
void FreeObjects()
{
    TObject *next = cache;
    TObject *node;
    while (next)
    {
        node = next;
        next = (TObject*)node->next;
        delete node->name;
        delete node->data;
        delete node;
    }
}

在此MSN网页上有一个很好的演示,确实不是线程安全,您需要将新变量分配给线程局部安全存储器。您可以在上述页面中查看函数get_thread_local_xloper12