发帖 回复
返回列表
  • 3105阅读
  • 7回复

[提问]程序执行时间时长时短 [复制链接]

 上一主题 下一主题
离线介农酥
侠客
 
只看楼主 倒序阅读 楼主  发表于: 2019-06-24
回复本帖可获得10RMB金钱奖励！
每人最多可获奖1次，奖池剩余40RMB金钱 (中奖几率10%)
我的程序某块代码执行很慢。好的时候，十几毫秒，坏的时候几万毫秒，甚至更高。


这块代码主要目的是初始化PLC（倍福），同时写入数据也会很慢。如果好的话，写入和初始化都正常。


关键是程序跟神经一样，上午好好的，下午可能就不行，今天好，明天就不行。一会好，一会坏，重启电脑也没用。


计算机初学者，没有思路，跟CPU和内存使用率关系不大。今大家指点方向！！！。


PLc相关：

#include "C:\TwinCAT\AdsApi\TcAdsDll\Include\TcAdsDef.h"
#include "C:\TwinCAT\AdsApi\TcAdsDll\Include\TcAdsAPI.h"
LIBS += C:\TwinCAT\AdsApi\TcAdsDll\Lib\TcAdsDll.lib
C:/Program Files (x86)/Microsoft SDKs/Windows/v5.0/Include/WinDef.h（lib需要用到）

20190624 16:40贴上部分代码
复制代码
  1. #include <plc.h>
  2. #include <cstring>
  3. #include <QtDebug>
  4. #include <iostream>
  5. #include <QThread>
  6. #include <QTime>
  8. Plc::Plc()
  9. {
  11. }
  13. void Plc::initPlcSlot()
  14. {
  15.     //qDebug() << "初始化Plc开始..." << QThread::currentThread();
  16.     qDebug() <<__func__ <<__LINE__;
  17.     emit indicatorSignal("正在初始化Plc...");
  18.     qDebug() <<__func__ <<__LINE__;
  19.     //local
  20.     pAddr = &Addr;
  21.     qDebug() <<__func__ <<__LINE__;
  22.     lPort = AdsPortOpen();
  23.     qDebug() <<__func__ <<__LINE__;
  24.     lErr = AdsGetLocalAddress(pAddr);
  25.     qDebug() <<__func__ <<__LINE__;
  26.     //remote
  27. //    AmsNetId id = { 192, 168, 145, 129, 1, 1 };
  28.     qDebug() <<__func__ <<__LINE__;
  29.     AmsNetId id = { 192, 168, 0, 92, 1, 1 };
  30.     pAddr->netId = id;
  31.     qDebug() <<__func__ <<__LINE__;
  32.     pAddr->port = 801;
  33.     qDebug() <<__func__ <<__LINE__;
  35.     char carrierVarName[8][25] =
  36.     {
  37.         ".gbo_CarrierNumber[1]",
  38.         ".gbo_CarrierNumber[2]",
  39.         ".gbo_CarrierNumber[3]",
  40.         ".gbo_CarrierNumber[4]",
  41.         ".gbo_CarrierNumber[5]",
  42.         ".gbo_CarrierNumber[6]",
  43.         ".gbo_CarrierNumber[7]",
  44.         ".gbo_CarrierNumber[8]",
  45.     };
  46. qDebug() <<__func__ <<__LINE__;
  47.     char boxVarName[8][20] =
  48.     {
  49.         ".gbo_BoxNumber[1]",
  50.         ".gbo_BoxNumber[2]",
  51.         ".gbo_BoxNumber[3]",
  52.         ".gbo_BoxNumber[4]",
  53.         ".gbo_BoxNumber[5]",
  54.         ".gbo_BoxNumber[6]",
  55.         ".gbo_BoxNumber[7]",
  56.         ".gbo_BoxNumber[8]",
  57.     };
  58. qDebug() <<__func__ <<__LINE__;
  59.     char expressName[8][20] =
  60.     {
  61.         ".gbo_Identifer[1]",
  62.         ".gbo_Identifer[2]",
  63.         ".gbo_Identifer[3]",
  64.         ".gbo_Identifer[4]",
  65.         ".gbo_Identifer[5]",
  66.         ".gbo_Identifer[6]",
  67.         ".gbo_Identifer[7]",
  68.         ".gbo_Identifer[8]",
  69.     };
  70. qDebug() <<__func__ <<__LINE__;
  71.     char tableName[8][20] =
  72.     {
  73.         ".gbo_DbTable[1]",
  74.         ".gbo_DbTable[2]",
  75.         ".gbo_DbTable[3]",
  76.         ".gbo_DbTable[4]",
  77.         ".gbo_DbTable[5]",
  78.         ".gbo_DbTable[6]",
  79.         ".gbo_DbTable[7]",
  80.         ".gbo_DbTable[8]",
  81.     };
  82. qDebug() <<__func__ <<__LINE__;
  83.     //init _bHandle, _btHandle by variable
  84.     if (lErr == 0) {
  85.         for(int i = 0; i < 8; ++i) {
  86.             qDebug() <<__func__ <<__LINE__;
  87.             AdsSyncReadWriteReq(
  88.                       pAddr, ADSIGRP_SYM_HNDBYNAME, //ADS-indexGroup-handle-by-name
  89.                       0x0, sizeof(carrierHandle[i]), &carrierHandle[i],
  90.                       sizeof(carrierVarName[i]), carrierVarName[i]);
  91.             qDebug() <<__func__ <<__LINE__;
  92.             AdsSyncReadWriteReq(
  93.                       pAddr, ADSIGRP_SYM_HNDBYNAME,
  94.                       0x0, sizeof(boxHandle[i]), &boxHandle[i],
  95.                       sizeof(boxVarName[i]), boxVarName[i]);
  96.             qDebug() <<__func__ <<__LINE__;
  97.             AdsSyncReadWriteReq(
  98.                       pAddr, ADSIGRP_SYM_HNDBYNAME,
  99.                       0x0, sizeof(expressHandle[i]), &expressHandle[i],
  100.                       sizeof(expressName[i]), expressName[i]);
  101.             qDebug() <<__func__ <<__LINE__;
  102.             AdsSyncReadWriteReq(
  103.                       pAddr, ADSIGRP_SYM_HNDBYNAME,
  104.                       0x0, sizeof(tableHandle[i]), &tableHandle[i],
  105.                       sizeof(tableName[i]), tableName[i]);
  106.             qDebug() <<__func__ <<__LINE__;
  107.         }
  108.     }
  109.     //qDebug("Plc初始化完成!");
  110.     emit indicatorSignal("Plc初始化完成!");
  111. }
  113. void Plc::plcWrite(QString carrierId, QString expressId, QString boxId, int pos, QTime start)
  114. {
  115. //    qDebug() << __func__ << carrierId << expressId << boxId << pos;
  116. //    QTime slottime = QTime::currentTime();
  117. //    qDebug() << signaltime.msecsTo(slottime) << __func__ << __LINE__;
  118.     short carrierIdShort = carrierId.toShort();
  119.     char *expressIdChar = expressId.toLatin1().data();
  120.     short boxIdShort = boxId.toShort();
  121.     qDebug() << __func__ << carrierIdShort << expressIdChar << boxIdShort << pos;
  122.     AdsSyncWriteReq(
  123.                 pAddr, ADSIGRP_SYM_VALBYHND,
  124.                 carrierHandle[pos], sizeof(carrierIdShort), &carrierIdShort);
  125.     AdsSyncWriteReq(
  126.                 pAddr, ADSIGRP_SYM_VALBYHND,
  127.                 boxHandle[pos], sizeof(boxIdShort), &boxIdShort);
  128.     AdsSyncWriteReq(
  129.                 pAddr, ADSIGRP_SYM_VALBYHND,
  130.                 expressHandle[pos], strlen(expressIdChar) + 1, expressIdChar);
  132.     QTime end = QTime::currentTime();
  133.     int delay = start.msecsTo(end);
  134. //    qDebug() << delay << "ms" <<__func__ << __LINE__;
  135.     emit delaySignal(delay);
  136. }
  138. void Plc::plcRead(int index)
  139. {
  140.     short carrierId, boxId;
  141.     char express[31] = { 0 };
  142.     char table[81] = { 0 };
  144.     AdsSyncReadReq(pAddr,
  145.         ADSIGRP_SYM_VALBYHND,
  146.         carrierHandle[index],
  147.         sizeof(short),
  148.         &carrierId);
  149.     qDebug() << "vehicleNum[" +
  150.                 QString::number(index) +"] " +
  151.                 QString::number(carrierId);
  152.     //qDebug() << QString("vehicleNum[%1]: %2").arg(pos).arg(vehicleNum);
  154.     AdsSyncReadReq(pAddr,
  155.         ADSIGRP_SYM_VALBYHND,
  156.         expressHandle[index],
  157.         sizeof(express),
  158.         express);
  159.     qDebug() << "express[" +
  160.                 QString::number(index) +"] " +
  161.                 express;
  163.     AdsSyncReadReq(pAddr,
  164.         ADSIGRP_SYM_VALBYHND,
  165.         boxHandle[index],
  166.         sizeof(short),
  167.         &boxId);
  168.     qDebug() << "boxNum[" +
  169.                 QString::number(index) +"] " +
  170.                 QString::number(boxId);
  172.     AdsSyncReadReq(pAddr,
  173.         ADSIGRP_SYM_VALBYHND,
  174.         tableHandle[index],
  175.         sizeof(table),
  176.         table);
  177.     qDebug() << QObject::tr("table: %1").arg(table);
  178. }
  180. void Plc::plcWriteTable(QString table, int pos)
  181. {
  182.     char *tableChar = table.toLatin1().data();
  183.     if (pos) { //1 deputy 申通
  184.         AdsSyncWriteReq(
  185.                     pAddr, ADSIGRP_SYM_VALBYHND,
  186.                     tableHandle[2], strlen(tableChar) + 1, tableChar);
  187.         AdsSyncWriteReq(
  188.                     pAddr, ADSIGRP_SYM_VALBYHND,
  189.                     tableHandle[1], strlen(tableChar) + 1, tableChar);
  190.         AdsSyncWriteReq(
  191.                     pAddr, ADSIGRP_SYM_VALBYHND,
  192.                     tableHandle[0], strlen(tableChar) + 1, tableChar);
  193.         AdsSyncWriteReq(
  194.                     pAddr, ADSIGRP_SYM_VALBYHND,
  195.                     tableHandle[3], strlen(tableChar) + 1, tableChar);
  196.     } else {
  197.         AdsSyncWriteReq(
  198.                     pAddr, ADSIGRP_SYM_VALBYHND,
  199.                     tableHandle[4], strlen(tableChar) + 1, tableChar);
  200.         AdsSyncWriteReq(
  201.                     pAddr, ADSIGRP_SYM_VALBYHND,
  202.                     tableHandle[5], strlen(tableChar) + 1, tableChar);
  203.         AdsSyncWriteReq(
  204.                     pAddr, ADSIGRP_SYM_VALBYHND,
  205.                     tableHandle[6], strlen(tableChar) + 1, tableChar);
  206.         AdsSyncWriteReq(
  207.                     pAddr, ADSIGRP_SYM_VALBYHND,
  208.                     tableHandle[7], strlen(tableChar) + 1, tableChar);
  209.     }
  211. //    qDebug() << __func__ << table << pos;
  212. }
  214. Plc::~Plc()
  215. {
  216.     for (int i = 0; i < 8; i++) {
  217.         AdsSyncWriteReq(
  218.                     pAddr, ADSIGRP_SYM_RELEASEHND, carrierHandle[i], 0x0, nullptr);
  219.         AdsSyncWriteReq(
  220.                     pAddr, ADSIGRP_SYM_RELEASEHND, boxHandle[i], 0x0, nullptr);
  221.         AdsSyncWriteReq(
  222.                     pAddr, ADSIGRP_SYM_RELEASEHND, expressHandle[i], 0x0, nullptr);
  223.         AdsSyncWriteReq(
  224.                     pAddr, ADSIGRP_SYM_RELEASEHND, tableHandle[i], 0x0, nullptr);
  225.     }
  226.     AdsPortClose();
  227. }

复制代码
  1. ######################################################################
  2. # Automatically generated by qmake (3.1) Wed Jun 19 20:46:06 2019
  3. ######################################################################
  5. QT += widgets sql network core
  6. TEMPLATE = app
  7. TARGET = flowOutExpress
  8. INCLUDEPATH += .
  9. #LIBS += C:\TwinCAT\AdsApi\TcAdsDll\Lib\TcAdsDll.lib
  10. UI_DIR = ./build-flowOutPortCopy3_db-Qt_5_12_1_MinGW_32_bit-Debug
  12. # The following define makes your compiler warn you if you use any
  13. # feature of Qt which has been marked as deprecated (the exact warnings
  14. # depend on your compiler). Please consult the documentation of the
  15. # deprecated API in order to know how to port your code away from it.
  16. DEFINES += QT_DEPRECATED_WARNINGS
  18. # You can also make your code fail to compile if you use deprecated APIs.
  19. # In order to do so, uncomment the following line.
  20. # You can also select to disable deprecated APIs only up to a certain version of Qt.
  21. #DEFINES += QT_DISABLE_DEPRECATED_BEFORE=0x060000    # disables all the APIs deprecated before Qt 6.0.0
  23. # Input
  24. HEADERS += configureflowoutport.h \
  25.            mainwindow.h \
  26.            plc.h \
  27.            socket.h \
  28.            C:/TwinCAT/AdsApi/TcAdsDll/Include/TcAdsDef.h \
  29.            "C:/Program Files (x86)/Microsoft SDKs/Windows/v5.0/Include/WinDef.h" \
  30.            C:/TwinCAT/AdsApi/TcAdsDll/Include/TcAdsAPI.h
  31. FORMS += configure.ui flowoutport.ui
  32. SOURCES += configureflowoutport.cpp \
  33.            main.cpp \
  34.            mainwindow.cpp \
  35.            plc.cpp \
  36.            socket.cpp \
  40. win32: LIBS += -L$$PWD/../build-flowOutPortThreadSeparate-Desktop_Qt_5_12_2_MinGW_32_bit-Debug/debug/Lib/ -lTcAdsDll
  42. INCLUDEPATH += $$PWD/../build-flowOutPortThreadSeparate-Desktop_Qt_5_12_2_MinGW_32_bit-Debug/debug/Lib
  43. DEPENDPATH += $$PWD/../build-flowOutPortThreadSeparate-Desktop_Qt_5_12_2_MinGW_32_bit-Debug/debug/Lib
本帖提到的人: @angelus
回复
举报
分享到 淘江湖 新浪 QQ微博 QQ空间 开心 人人 豆瓣 网易微博 百度 鲜果 白社会 飞信
在线20091001753
论坛版主
只看该作者 1楼 发表于: 2019-06-24
几万毫秒？！
1000毫秒 = 1秒
几万毫秒 就是 几十秒啊，都快赶上 1分钟了。
(づ￣ 3￣)づ
回复
举报
离线九重水
圣骑士
只看该作者 2楼 发表于: 2019-06-24
没代码无真相。。。。。。
请救要贴代码呀。
回复
举报
离线cuisy
新手上路
只看该作者 3楼 发表于: 2019-06-24
回帖奖励+ 10
回复
举报
离线介农酥
侠客
只看该作者 4楼 发表于: 2019-06-24
回 九重水 的帖子
九重水:没代码无真相。。。。。。
请救要贴代码呀。[表情]  (2019-06-24 13:53) 

代码我贴上了，不过估计也不好看
回复
举报
离线介农酥
侠客
只看该作者 5楼 发表于: 2019-06-24
回 20091001753 的帖子
20091001753:几万毫秒？！
1000毫秒 = 1秒
几万毫秒 就是 几十秒啊，都快赶上 1分钟了。 (2019-06-24 13:27) 

对，实际已经几分钟了，因为越积越多
回复
举报
离线九重水
圣骑士
只看该作者 6楼 发表于: 2019-06-25
首先，看你初始化和读写的时候，调用的函数名，都是同步函数比如AdsSyncReadWriteReq，如果双方通信线路或有一方机器出了些许问题，一直在校验丢包的话，可能要重发很多次才能成功。

或者下位机PLC很忙碌，没空理上位机，上位机同样需要等待很久很久。

1、如果你确认它们没故障，可以调用库提供的【异步函数】；
2、或者你在开【一两个线程】初始化和读写，明确告诉用户正在初始化和读写，让用户等待；
3、换一台好的PLC再试试，确认你的电脑和PLC都没问题。
回复
举报
离线zxwmail
新手上路
只看该作者 7楼 发表于: 2019-06-25
楼上说的正确。
回复
举报
发帖 回复
返回列表
快速回复
限100 字节
 
上一个 下一个