Question

我刚刚完成了一个用C语言编写的小dll。它是第三方软件和占星术dll之间的接口。基本上，它给出了占星术的位置沿着黄道的经度位置，给出了出生日期，时间，经度和纬度。因为第三方软件需要返回字符串，然后将房屋经度转换为ascii并使用sprintf将它们与逗号串在一起。如果我在函数“housecusps”中声明了返回字符串“retrnString”，那么我的变量就会被破坏（主要是“indx”）。但是，只要“retrnString”被声明为全局，它就能完美地运行。有人可以解释为什么???

#include <stdio.h>
#include <windows.h>
#include "swephexp.h"
#include "Wave59_SDK.h"

typedef  int32 (*JULDAYPTR)(int32, int32, int32, int32, int32, double, int32, double*, char*);
typedef int (*HOUSECUSPSPTR)(double, double, double, int, double*, double*);
char retrnString[96];

BOOL APIENTRY DllMain (HINSTANCE hInst     /* Library instance handle. */ ,
                       DWORD reason        /* Reason this function is being called. */ ,
                       LPVOID reserved     /* Not used. */ );

char* __declspec(dllexport) housecusps(WAVE59_DATASTRUCT *price_ptr, int currentptr,
            int *int_args,int num_int_args,double *double_args,
            int num_double_args,char **string_args,int num_string_args)
{
    int32 iyear = int_args[0];
    int32 imnth = int_args[1];  
    int32 iday = int_args[2];
    int32 ihr = int_args[3];
    int32 imin = int_args[4];  
    int32 gregflag = SE_GREG_CAL;
    double dret[2], cuspArray[13], ascmc[10], julianDays;
    char serr[256];
    const double zeroSecs = 0;
    int hsys = 'P';
    int ctr, indx;
    JULDAYPTR JulDay;
    HOUSECUSPSPTR HouseCusps;

    HINSTANCE astrologyDLL = LoadLibrary("c:\\sweph\\bin\\swetrs32.dll");
    if (astrologyDLL == NULL)
       return "Error loading swedll32.dll";
    JulDay = (JULDAYPTR)GetProcAddress(astrologyDLL, "_swe_utc_to_jd@40");
    if (JulDay == NULL)
       return "Error loading swe_utc_to_jd";
    if (JulDay(iyear,imnth,iday,ihr,imin,zeroSecs,gregflag,dret,serr) == ERR)
       return serr;
    julianDays = dret[1];
    HouseCusps = (HOUSECUSPSPTR)GetProcAddress(astrologyDLL, "_swe_houses@36");
    if (HouseCusps == NULL)
       return "Error loading swe_houses";
    /*//Parms:- dret[1] = Julian day in UT, double_args[0] = Latitude, double_args[1] = Longitude.*/
    if (HouseCusps(julianDays,double_args[0],double_args[1],hsys,cuspArray,ascmc) == ERR)
        return "Error in swe_houses"; 
    indx = 0;
    for (ctr = 1;ctr < 13; ctr++)
    {
        indx += sprintf(retrnString + indx,"%.3f",cuspArray[ctr]);
        if (ctr != 12)
           indx += sprintf(retrnString + indx ,"%c",',');
    }
    FreeLibrary(astrologyDLL);
    return retrnString;
}

Answer 1

如果你把char retrnString[96];放在func housecusps(...)中，它是一个留在堆栈中的局部变量，当你从该函数返回时，这个堆栈空间将被清除，这就是为什么它变成了损坏。

有一些方法可以从函数“返回/获取”此类内容，

全局变量char retrnString[96];就像您所做的那样。
malloc一个内存来保存内容并将其返回，然后记得free以后

<强>通知：
更好的选择是让函数接受输出缓冲区作为参数。否则，如果调用者使用不同的分配器或不同的内存池，则此处的第二个选项（内部malloc）可能会出现问题。 ----来自 @Matt McNabb的建议

将全局变量用于返回值的主要缺点是您的代码变为不可重入。您不能有两个同时调用该函数的线程。此外，在调用函数一次之后，您必须在第二次调用函数之前完成第一个值，或者在创建第二个函数之前复制第一个调用的字符串。 ----来自 @Jonathan Leffler的评论

sprintf给出未定义的结果

1 个答案: