QVector / QList проблем при използването им със потребителски класове
-
@Stoyan said:
Не мога да разбера връзката между Task, Operation и Trigger. Изглежда, че всяка задача има по няколко операции, а всяка операция съдържа по няколко тригера, но последното не виждам да си го дефинирал никъде. Къде трябва да се четат и задават тригерите?
Всяка задача и операция има вектор от тригери, които могат да я стартират впоследствие. Задават се във всеки обект при зареждането му, като трябва да могат и да се променят във всяко едно време от клиентската част. През определен период от време се проверяват тригерите и ако условието е изпълнено връща ок за изпълнение на задачата/операцията.
Най-вероятно ще ми трябва Scheduler да бъде наследник на QOBject за да мога да ползвам connect, но нещо пак не иска.scheduler.cpp:-1: error: undefined reference to `vtable for Scheduler'Добавил съм Q_OBJECT Макрото и промених конструктора.
Програмата се стартира като конзолно приложение с добавяне на
-e
в Run arguments .
Каква е разликата с нормална конзолна програма и сървис с -е ?@pecuna91
Тази грешка се появява когато си добавил макроса Q_OBJECT. При компилирането не се е изпълнила командата qmake, както би трябвало. Трябва да я пуснеш ръчно като цъкнеш с десен бутон на проекта и избереш "Run qmake". При мен така се компилира.По принцип не би трябвало да има разлика, ако се пусне и като service с аргумент "-e", но при мен така даде грешка:
The service could not be executed. ()
can't find linker symbol for virtual table for `Service' value
Когато преместих съдържанието на функцията start() в main(), както ти писах, тръгна и даде такъв резултат:
SSL is supported
Server Listening on port: 1234
"stopped"
Task is disabled
"stopped"
Task is disabled
....Предполагам, че има някакъв проблем при свързването с външната библиотека за поддръжка на service.
Преди време правих една програма, която трябваше да работи като service и използвах ето този компонент:
https://github.com/qtproject/qt-solutions/tree/master/qtservice
Ако го добавиш като подпроект няма да има нужда от външни библиотеки, защото се включва в изпълнимия файл. -
@pecuna91
Тази грешка се появява когато си добавил макроса Q_OBJECT. При компилирането не се е изпълнила командата qmake, както би трябвало. Трябва да я пуснеш ръчно като цъкнеш с десен бутон на проекта и избереш "Run qmake". При мен така се компилира.По принцип не би трябвало да има разлика, ако се пусне и като service с аргумент "-e", но при мен така даде грешка:
The service could not be executed. ()
can't find linker symbol for virtual table for `Service' value
Когато преместих съдържанието на функцията start() в main(), както ти писах, тръгна и даде такъв резултат:
SSL is supported
Server Listening on port: 1234
"stopped"
Task is disabled
"stopped"
Task is disabled
....Предполагам, че има някакъв проблем при свързването с външната библиотека за поддръжка на service.
Преди време правих една програма, която трябваше да работи като service и използвах ето този компонент:
https://github.com/qtproject/qt-solutions/tree/master/qtservice
Ако го добавиш като подпроект няма да има нужда от външни библиотеки, защото се включва в изпълнимия файл.@Stoyan
АБсолютния същия проект ползвам. Можеш да видиш QtService папката във сорсовете. Библиотеката съм я добавил за да нямам допълнителни cpp файлове и да се компилира по-бързо програмата. Добавил ли си
QtSolutions_Service-head.dll файла в release директорията?Благодаря за "run qmake", досега каквото съм добавял и правил наследник на QObject винаги е тръгвало без ръчно изпълнение на тази команда.
Успеях да преместя таймера при сървис класа.
Остава въпросът как точно ще се пушват новите тригери във вектора на обекта.
Успях да го направя , но искам да е с копи конструктор или оператор = , a не с функция setAll(const Trigger &obj), което е същото де. -
@Stoyan
АБсолютния същия проект ползвам. Можеш да видиш QtService папката във сорсовете. Библиотеката съм я добавил за да нямам допълнителни cpp файлове и да се компилира по-бързо програмата. Добавил ли си
QtSolutions_Service-head.dll файла в release директорията?Благодаря за "run qmake", досега каквото съм добавял и правил наследник на QObject винаги е тръгвало без ръчно изпълнение на тази команда.
Успеях да преместя таймера при сървис класа.
Остава въпросът как точно ще се пушват новите тригери във вектора на обекта.
Успях да го направя , но искам да е с копи конструктор или оператор = , a не с функция setAll(const Trigger &obj), което е същото де.@pecuna91
Даваше грешка, само когато се компилира в Debug режим. В Release работи. Предполагам, защото във файла на проекта си задал винаги да се зарежда библиотеката QtSolutions_Service-head.dll, която е за Release. При компилиране за Debug трябва да се зарежда библиотеката QtSolutions_Service-headd.dll.Доколкото виждам зареждането на тригерите става с функциите loadOperationTrigger() и loadTaskTrigger(), а добавянето в съответната задача или операция - с BaseObject::actionTriggers(QString Action, const Trigger &obj).
Ако се използва списък с указатели, аз предлагам последната функция да стане:
int BaseObject::actionTriggers(QString Action, Trigger *obj)
а още по-добре е да я разделиш на две функции:
{
addTrigger(Trigger * trigger);
delTrigger(Trigger * trigger);
В тях се включва:
triggers.append(trigger); // в addTrigger() - добавя указания тригер в списъка на текущия обект (Task или Operation)
triggers.removeAll(trigger); // в delTrigger() - премахва указания тригер от списъка на обекта.
После ги извикваш така:
Trigger * trigger;
trigger = new Trigger();
trigger->id = ...;
trigger->type = ...;
...
task.addTrigger(trigger); // или operation.addTrigger(trigger); -
@pecuna91
Даваше грешка, само когато се компилира в Debug режим. В Release работи. Предполагам, защото във файла на проекта си задал винаги да се зарежда библиотеката QtSolutions_Service-head.dll, която е за Release. При компилиране за Debug трябва да се зарежда библиотеката QtSolutions_Service-headd.dll.Доколкото виждам зареждането на тригерите става с функциите loadOperationTrigger() и loadTaskTrigger(), а добавянето в съответната задача или операция - с BaseObject::actionTriggers(QString Action, const Trigger &obj).
Ако се използва списък с указатели, аз предлагам последната функция да стане:
int BaseObject::actionTriggers(QString Action, Trigger *obj)
а още по-добре е да я разделиш на две функции:
{
addTrigger(Trigger * trigger);
delTrigger(Trigger * trigger);
В тях се включва:
triggers.append(trigger); // в addTrigger() - добавя указания тригер в списъка на текущия обект (Task или Operation)
triggers.removeAll(trigger); // в delTrigger() - премахва указания тригер от списъка на обекта.
После ги извикваш така:
Trigger * trigger;
trigger = new Trigger();
trigger->id = ...;
trigger->type = ...;
...
task.addTrigger(trigger); // или operation.addTrigger(trigger);@Stoyan
По този начин сетвам тригерите да.
Функцията искам да е една, еднотипна за всичките обекти.
Проблемът сега се явява отново Segmentation fault при извикване на get някой атрибут на тригер извън globalvars класа. Например в servicе или Scheduler. Понеже понякога /в редки случаи/ не го дава предполагам че е отново реда на създаване на обектите и сетването им. Можеби ще трябва да въведа някакви паузи докато се инициализират нещата,
Интересното е че не излизат атрибутите на тригерите когато ги викам с гетерите в друг клас извън GlobalVars и гърми с грешката. -
@Stoyan
По този начин сетвам тригерите да.
Функцията искам да е една, еднотипна за всичките обекти.
Проблемът сега се явява отново Segmentation fault при извикване на get някой атрибут на тригер извън globalvars класа. Например в servicе или Scheduler. Понеже понякога /в редки случаи/ не го дава предполагам че е отново реда на създаване на обектите и сетването им. Можеби ще трябва да въведа някакви паузи докато се инициализират нещата,
Интересното е че не излизат атрибутите на тригерите когато ги викам с гетерите в друг клас извън GlobalVars и гърми с грешката.BaseObject::actionTriggers в момента изпълнява две съвсем различни неща и ще бъде по-ясно ако ги разделиш в отделни функции, но естествено не е задължително. А функциите, които предложих addTrigger() и delTrigger() пак са в основния клас и са едни и същи и за задачите и за операциите.
А за грешките - ще трябва да качиш промените, за да видя какви са и как се получават.
-
BaseObject::actionTriggers в момента изпълнява две съвсем различни неща и ще бъде по-ясно ако ги разделиш в отделни функции, но естествено не е задължително. А функциите, които предложих addTrigger() и delTrigger() пак са в основния клас и са едни и същи и за задачите и за операциите.
А за грешките - ще трябва да качиш промените, за да видя какви са и как се получават.
-
Качих промените. Намалил съм дебъг логването.
Както ще видиш проблемът се появява когато се вика triggers.at(n).getНещоси от Scheduler класа 10 секунди след като се е стартирало приложението.@pecuna91
Трябва да махнеш от BaseObject::actionTriggersnewtrigger->deleteLater();
С него първо добавяш тригера към вектора и веднага след това го изтриваш.
Ето какво се получи без него:Service started!
SSL is supported
Server Listening on port: 1234
"stopped"
Task is disabled
activating task: "Task 2"
Trigger: "time"
"stopped"
Task is disabled
activating task: "Task 2"
Trigger: "time"
....Освен това, защо ти е това Trigger::operator =()? Не можеш ли командите в него да ги изпълниш директно във функцията Trigger::setAll()?
При начина, по който ги използваш в момента, всичките "operator =()" са излишни. И за Trigger, и за Task, и за Operation. -
@pecuna91
Трябва да махнеш от BaseObject::actionTriggersnewtrigger->deleteLater();
С него първо добавяш тригера към вектора и веднага след това го изтриваш.
Ето какво се получи без него:Service started!
SSL is supported
Server Listening on port: 1234
"stopped"
Task is disabled
activating task: "Task 2"
Trigger: "time"
"stopped"
Task is disabled
activating task: "Task 2"
Trigger: "time"
....Освен това, защо ти е това Trigger::operator =()? Не можеш ли командите в него да ги изпълниш директно във функцията Trigger::setAll()?
При начина, по който ги използваш в момента, всичките "operator =()" са излишни. И за Trigger, и за Task, и за Operation.@Stoyan
Хмм, хем ми направи впечетление за тоя Delete later още на сокетите че го казаха в туториала че прави проблем и го сложих...
Махнах оператор= за тригера, но ми направи впечетление че Copy constructor-а трябва да си стои като дефиниция поне иначе гърми с грешки, но защо изобщо трябва да е дефиниран , но не и реално имплентиран? -
@Stoyan
Хмм, хем ми направи впечетление за тоя Delete later още на сокетите че го казаха в туториала че прави проблем и го сложих...
Махнах оператор= за тригера, но ми направи впечетление че Copy constructor-а трябва да си стои като дефиниция поне иначе гърми с грешки, но защо изобщо трябва да е дефиниран , но не и реално имплентиран?@pecuna91
Не бих казал, че deleteLater() прави проблеми. Понякога дори не може без него. Просто в този случай не прави това, което си си мислел, че прави.Trigger *newtrigger = new Trigger(); // създава се нов обект Trigger като newtrigger е указател към него
newtrigger->setAll(obj); // задават се всички параметри за тригера
newtrigger->deleteLater(); // изтрива се обекта, към който сочи указателя
triggers.push_back(newtrigger); // добавя се указателя към тригера във вектораКогато по-късно използваш triggers.at(i) всъщност взимаш указателя от вектора, но той вече сочи към случаен адрес в паметта. Затова и гърми.
Но когато изтриваш тригер от вектора ще трябва да използваш delete() или deleteLater().Copy constructor-а за Trigger всъщност се използва неявно във функцията GlobalVars::loadTrigger(). Затова и не можеш да го махнеш.
-
@pecuna91
Не бих казал, че deleteLater() прави проблеми. Понякога дори не може без него. Просто в този случай не прави това, което си си мислел, че прави.Trigger *newtrigger = new Trigger(); // създава се нов обект Trigger като newtrigger е указател към него
newtrigger->setAll(obj); // задават се всички параметри за тригера
newtrigger->deleteLater(); // изтрива се обекта, към който сочи указателя
triggers.push_back(newtrigger); // добавя се указателя към тригера във вектораКогато по-късно използваш triggers.at(i) всъщност взимаш указателя от вектора, но той вече сочи към случаен адрес в паметта. Затова и гърми.
Но когато изтриваш тригер от вектора ще трябва да използваш delete() или deleteLater().Copy constructor-а за Trigger всъщност се използва неявно във функцията GlobalVars::loadTrigger(). Затова и не можеш да го махнеш.
@Stoyan
error: passing 'const Operation' as 'this' argument of 'int Operation::setAttribute(QString, QString)' discards qualifiers [-fpermissive]
operations.at(i).setAttribute(Aname,Value);Не мога да разбера защо се получава това съобщение в Task класа, опитвайки се да извикам метода SetAttribute за операция във вектора.
Const Operation не знам откъде идва вътре в тази функция конкретно, след като подавам само id-то на операцията.
Подобна е грешката и при Task сетване на атрибут:
scheduledtopservice\scheduler.cpp:31: error: passing 'const Task' as 'this' argument of 'int Task::setAttribute(QString, QString)' discards qualifiers [-fpermissive]
tasks.at(k).setAttribute("state","running");ПП:
Успях да го реша проблема. Разбрах че .at() връща const обект, затова използвах [i] за индекс.task.cpp:17: warning: base class 'class BaseObject' should be explicitly initialized in the copy constructor [-Wextra]
Task::Task(const Task &obj)
Това как мога да го оправя, четох че създавало още 1 копие на BaseObject така. Но не е като експлицитния конструктор сякаш : BaseObject ( const BaseObject &obj) ако долепя до Copy конструктора на Task. Не съм сигурен, но май просто като сложих : BaseObject() и се изчисти предупреждението. Така ли трябвa да бъде ?