我刚刚用C语言写了一个小的动态链接库,它是第三方软件和占星术动态链接库之间的接口。基本上它得到了星占宫沿黄道的位置经度,给出了出生日期、时间、经度和纬度。因为第三方软件需要一个返回字符串,所以我将房子的经度转换为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;
}

最佳答案

如果将char retrnString[96];放入funchousecusps(...)中,则它是一个留在堆栈中的局部变量,当您从该函数返回时,该堆栈空间将被清除,这就是它被损坏的原因。
有一些方法可以从函数“返回/获取”这些内容,
一个全局变量char retrnString[96];就像你所做的一样。
malloc保存内容并返回的内存,然后记住以后再返回
告示
更好的选择是让函数接受输出缓冲区作为参数。否则,如果调用者使用不同的分配器或不同的内存池,这里的第二个选项(free内部)可能会有问题。----来自@Matt McNabb的建议
使用全局变量作为返回值的主要缺点是代码变得不可重入。不能有两个线程同时调用函数。此外,在调用函数一次之后,您必须在第二次调用函数之前完成第一个值,或者在进行第二次调用之前复制第一次调用的字符串。----来自@Jonathan Leffler的评论

关于c - sprintf提供不确定的结果,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/31307040/

10-12 19:55