Question

我的手上有一个很大的问题，我认为它不能正常运作的原因在于我的C语言知识，让我们面对它最好的新手......

void * KT_Indirect_CNS(int NumCNS, char *CNSDescr[], char *CNSTag[], int *CNSIsBit[], char* lpszPictureName){

#include "apdefap.h"

char  TagNameEnable[255],
    TagNameTrue[255],
    TagNameFalse[255],
    TagNameString[255],
    TagNameCNS[255],
    TagNameCNSString[255],
    TagNameTemp[255];

int   i,
        count;

typedef struct 
{ 
    LONG x; 
    LONG y; 
} tPOINT;

#pragma code ("user32.dll");
BOOL GetCursorPos( tPOINT* lpPoint);
#pragma code();

tPOINT pt;

GetCursorPos(&pt);

for (i=1; i<=NumCNS; i++)
{
    sprintf(TagNameEnable, "CNS%d.Enabled", i);
    SetTagBit(TagNameEnable, 1);

    sprintf(TagNameCNS, CNSTag[i-1]);
    sprintf(TagNameTrue, "CNS%d.True", i);
    sprintf(TagNameFalse, "CNS%d.False", i);
    if (CNSIsBit[i-1] == 17)
    {
              BOOL temp = GetTagBit(TagNameCNS);
        if (temp==1)
        {
            SetTagBit(TagNameTrue, 1);
            SetTagBit(TagNameFalse, 0);
                    count++;
        }
        else
        {
            SetTagBit(TagNameTrue, 0);
            SetTagBit(TagNameFalse, 1);
        }
    }
    if ((CNSIsBit[i-1] >= 0) && (CNSIsBit[i-1] <=16))
    {
        int *bits = GetBitsFromWord(GetTagWord(TagNameCNS));
        int index = (int)CNSIsBit[i-1];
        if (bits[index]==1)
        {
            SetTagBit(TagNameTrue, 1);
            SetTagBit(TagNameFalse, 0);
                    count++;
        }
        if (bits[index]==0)
        {
            SetTagBit(TagNameTrue, 0);
            SetTagBit(TagNameFalse, 1);
        }
    }
    sprintf(TagNameString, "CNS%d.String", i);
    SetTagChar(TagNameString, CNSDescr[i-1]);
}

if (NumCNS <16)
{
    for (i=NumCNS+1; i<=16; i++)
    {
        sprintf(TagNameTrue, "CNS%d.True", i);
        sprintf(TagNameFalse, "CNS%d.False", i);
        sprintf(TagNameEnable, "CNS%d.Enabled", i);
        SetTagBit(TagNameTrue, 0);
        SetTagBit(TagNameFalse, 0);
        SetTagBit(TagNameEnable, 0);
    }
}

if (count != NumCNS)
{
    sprintf(TagNameTemp, "CNS_Page_AllGood");
    SetTagBit(TagNameTemp, 0);
        SetVisible(lpszPictureName,"KT_CNS",TRUE);  
    sprintf(TagNameTemp, "CNS_PAGE_MOUSE_X");
    SetTagSWord(TagNameTemp, pt.x-5);
    sprintf(TagNameTemp, "CNS_PAGE_MOUSE_Y");
    SetTagSWord(TagNameTemp, pt.y-150);
    sprintf(TagNameTemp, "CNS_PAGE_HEIGHT");
    SetTagSWord(TagNameTemp, 30*NumCNS);
}
if (count == NumCNS)
{
    sprintf(TagNameTemp, "CNS_Page_AllGood");
    SetTagBit(TagNameTemp, 1);
}

return ("Controller Initilized");}


int *GetBitsFromWord(WORD _word){

int bits[16];

bits[0] = (_word & 0x0001)?1:0;
bits[1] = (_word & 0x0002)?1:0;
bits[2] = (_word & 0x0004)?1:0;
bits[3] = (_word & 0x0008)?1:0;
bits[4] = (_word & 0x0010)?1:0;
bits[5] = (_word & 0x0020)?1:0;
bits[6] = (_word & 0x0040)?1:0;
bits[7] = (_word & 0x0080)?1:0;
bits[8] = (_word & 0x0100)?1:0;
bits[9] = (_word & 0x0200)?1:0;
bits[10] = (_word & 0x0400)?1:0;
bits[11] = (_word & 0x0800)?1:0;
bits[12] = (_word & 0x1000)?1:0;
bits[13] = (_word & 0x2000)?1:0;
bits[14] = (_word & 0x4000)?1:0;
bits[15] = (_word & 0x8000)?1:0;

return bits;}

这些脚本用于收集工厂中设备的一般未运行原因...我认为它在我测试时有用，愚蠢的我会添加... 我认为我的指针使用知识足够低，以至于我犯了一个会让我被杀的错误... 基本上我需要从我的PLC（s7-400）收集WORD类型数据（16位），我需要从所述单词中添加位来确定WinCC中屏幕上对象的状态。事情是我没有得到我期望的价值......它编译没有问题，但...... 有人有一些建议吗？如何改进我的代码？

Answer 1

哦，小伙子，对代码改进和可能解决问题的一些评论：

你在函数中做了一个include。这是不好的做法。然后在函数内部定义typedef并使用pragma＆a和原型。强烈建议在启动函数之前执行此操作，所以：

#include "apdefap.h" typedef struct { LONG x; LONG y; } tPOINT; #pragma code ("user32.dll"); BOOL GetCursorPos( tPOINT* lpPoint); #pragma code(); void * KT_Indirect_CNS(int NumCNS, char *CNSDescr[], char *CNSTag[], int *CNSIsBit[], char* lpszPictureName) { char TagNameEnable[255], TagNameTrue[255], TagNameFalse[255], TagNameString[255], TagNameCNS[255], TagNameCNSString[255], TagNameTemp[255]; int i, count; //... }

在循环for (i=1; i<=NumCNS; i++)中，从1开始。在C中，所有索引都从0开始。在代码的其余部分，我看不出从1开始的令人信服的理由，所以从零开始：{{1}但是，你所做的并没有错。

但你的for (i=0; i<NumCNS; i++)确实存在问题。在此函数内部，您声明一个数组GetBitsFromWord，并在函数的末尾返回此数组。但是aray是一个所谓的自动（本地）变量，当函数返回时它不再存在（它在堆栈上分配，并且该部分在返回时被释放并且可能被覆盖）通过跟随函数调用）。

相反，将其声明为int bits[16]并将bits数组作为参数传递给调用者。

WinCC c脚本从单词读取位 - 一般C建议

1 个答案: