Hi!

Thank you very much for your help. I thought and I want to make little steps at first, so I changed my struct:

// person.h struct person { int id; QStandardItem* name; QStandardItem* family; }

Now I can directly use my members and append them.
I do not need write access, but to show my objects directly in QAbstractTableModel is indeed a sexy idea. I am trying around with QModelIndex, club_model->item(), club_model->itemData() and roles, and when I am done with that, maybe I'll try my own model.

For now, my question is solved!

Regards Gerhard

@Lasith
well std::bad_alloc often comes when out of memory.
My guess is that <List1[0].size() is always true since you add to it for each loop
and hence create an infinite loop.

Did you try VRonin code?

Update:
If you just want list1 in [0] and list2 in[1]
why all the loops then?

@kshegunov said in increasing QList<QStringList> contents twice!:

if it looks like a duck, swims like a duck, and quacks like a duck

Slight thread derail, but I would say the op's question general is answered .

@VRonin a suggest read is a SO post

As mentioned earlier aswel, I will use this iterator (first time :p)

Thanks for all the help!

I just had a typo in the group ^^
Sorry!

The easy way:
get rid of PortalMapItemInfo, store the values in 4 different QStringList and use a QStringListModel

The proper way:
in HomeController.h add Q_DECLARE_METATYPE(PortalMapItemInfo) http://doc.qt.io/qt-5/qmetatype.html#Q_DECLARE_METATYPE

Instead of a List store your data in a 1-column QStandardItemModel and expose it via a Q_PRPERTY

@Mathan-M Ok, then have a look at my code above; that works as expected, what's different to your code?

Thank you for the hints, I think I now have enough information to solve it.

I too was also thinking of full rewrite but i don't have much time for that right now :/ .

@kshegunov said:

You're missing the fact that QWidget and its descendants aren't reentrant.

Actually, I missed more than that... I completely miss the fact that, when dealing with QWidget or simillar + threads, that exactely when I should have used signal/slot mechanism... I feel dumb about it, but anyway, the Crawler object now crawls using Qt::Concurrent and send a signal with the text to add to the QListWidget, and then a MainWidget's slot is called to create and add the item.

Thanks for your help

I see. Thanks @raven-worx @SGaist!

@Chris-Kawa Sorry for getting you into a mistake and for leaving my problem without any further explanation. So, the variable intArray is just a simple Qlist, i forgot to change its name before posting so please, excuse me for that.

For index, it looks like index = w * 4, and after that, there is value = qRgba(intArray[index-3], intArray[index-2], intArray[index-1],intArray[index]);

I know that if the length of my text is not divisible, it's not gonna work. And here the Qlist come in handy. I can check if this number is divisible by 4 and if yes, the code goes as it is now, but when it's not I wanted to add as many items equal to 0 as I need to make it divisible.

Does my explanation makes any sense? I will try try to explain it like that :

if (number is divisible by 4) {
here comes my algorithm for making this image
}
else
{
add as many number as it is needed to make it divisable to qlist and then run the algorithm
}
But then those error appeared and I don't really know how can I get rid of them :/

But, how could I make it with vectors? Would it be faster or safer than by using Qlist?

Thanks a lot for help!

@Chris-Kawa

Got it ;)

@micland

WOOW!!! Thank you sooo much micland!! It works!!!
I tried a code similar to yours but some data were missing. Just needed to do a very tiny modification on your code:

qDebug() << key << value.variant().toString();

or directly

qDebug() << value.variant().toString();

Except it, it is perfect! Thank you again.
@micland said:

I have no clue why the qdbusviewer of Qt 5.6 is unable to call the DBus method but the code snippet above is working fine.

I hope it is going to change as I intend to use it or the 5.7 version very soon.

Cheers again.

@kshegunov
There is a reason why I can't use QT Property Binding (It only support predefined Property using Q_PROPERTY script).
Please take a look at my topic about Auto Binding 2 ways with Dynamic Property
But Binding is an other topic.

About "Object's pointers & Memory"
I also take an advise from a Pro in C++ & memory management.
His solution is implement a Factory Pattern to Create/Delete Object's pointers
(also free memory when an instance had no pointer to it.)
(I think I'll give it a try)

I'm such a newbie programmer :(
I study electronics engineer and I like programming but I don't receive a good foundation in my university and I had to learn a lots of things by myself. Also, I haven't done any OOP for years and I'm new in Qt.
Many many thanks!! I will pay more attention to my coding :)

@mrjj Thank you a lot, it works!!!

However I did certainly what you said that was only little part.
Thank for your answer, without it I would only find a way around this issue probably. (like 2 parallel QLists)

So what works?

I make additional QObject with parent "this (list of lists)". Then I make it parent of every QObject I want to delete (QObjects with list and QObjects within lists). I don't change CppOwnership. After that I clear() list and add new QObjects (obsolete QObjects stay accessible as children of that additional QObject). After final emit I delete that QObject with deleteLater(). QPointer isn't necessary here I think. I tried deleteLater in another way of doing things and it crashed, so that is the only way I found. My application deletes QObject and all children with it. With neither crash or leak. QObject * delQObject = new QObject (this); set_parents_list(delQObject ); m_list_of_listQobjects.clear(); for () m_list_of_listQobjects.append(new listQobject); emit m_list_of_listQobjectsChanged(m_list_of_listQobjects()); delQObject->deleteLater();

where:

listoflistsofQObjects :: set_parents_list(QObject * parent) { for (int i = m_list_of_listQobjects.count(); i>0; i--) { m_list_of_listQobjects[i-1]->setParent(parent); m_list_of_listQobjects[i-1]->set_parents_list(parent); } }

Following function isn't necessary if QObjects already have parent that is deleted (like listofQObjects or some QObject inside it). It's safer to make elements children, because then one doesn't need to call deletes in destructor.

listofQObjects :: set_parents_list(QObject * parent) { for (int i = m_list_ofQobjects.count(); i>0; i--) { m_list_ofQobjects[i-1]->setParent(parent); } }

I don't know if is it "right way", I even found here somebody uses my earlier version with additional list (but it's unsafe for sure).

The best explanation how to manage memory in qt I found here.

If there is better way to do so, please share.

Hi,

There's usually no need to allocate QList nor QStringList on the heap. Do you have any reasons to do that ?

When using a custom model, the usage is to add specialized methods to your model to add/remove your custom objects. So when you add/remove a Shape you know exactly where they are so you can use beginInsertRow/endInsertRow and the update will happen automatically for you.

Rather than going up-front with building your own GUI for a painting program, I'd recommend taking a look at what is currently existing like Krita or KolourPaint. Painting is a vast subject that can cover many aspects. Just take a look at Qt's examples on the subject.

You should also take the time to look at QtQuick for the GUI design part.

@Asperamanca
I am actually not sure anymore. I thought that there was an issue regarding the boundingRect() implementation but I have changed my design since then. I will give this another thought.

Well, your model should then return one column and the string list size as row count. Then data should return the appropriate string from the list

It's been solved. "this" pointer was null, thus it was crazy debugging.

Hi,

Do you mean move them at the right position ?

Hi,

Something's not clear with your code, you return a QQmlListProperty<MyObject> while mMyObjectsList is a QQmlListProperty<MyObject*>.

Also why are you calling (&mMyObjects)->values(); and not mMyObjects.values();?

super, thank you! that solved my litttle problem.

@Jeff-Andle

If you do not like that cleanup or if it is too hard to decide whether to free the memory you can use one oft these.

QSharedData QSharedDataPointer QSharedPointer

I have used QSharedDataPointer (but not the others so far), and it seems to work nice.

@Chris-Kawa
Thanks for the reply
I messed up as you mentioned above . I used your code to make it work. thank you ;)

QList<QColor> colors = { QColor::fromRgb(0,102,104,255), QColor::fromRgb(102,102,104,255), QColor::fromRgb(0,0,104,255) }; for(int i = 0; i < colors.size(); ++i) combo->setItemData(i, colors[i], Qt::BackgroundRole);

at funtion gerarRelatorio i swap the qDeleteAll on line down(original position). and it worked.

qDebug()<<"Documentos processados para a Tabela de Relatorio"; acessodb::setDataUltimoRelatorio(idEmpresa,idCliente); qDeleteAll(listaDocumentos); listaDocumentos.clear(); delete modelo; return true;

I now will upload to my test server, and see what happens.
i doesn't care any more about what is causing the problem, i just hope that is solved.

error: invalid application of 'sizeof' to incomplete type 'QStaticAssertFailure<false>' enum {Q_STATIC_ASSERT_PRIVATE_JOIN(q_static_assert_result, __COUNTER__) = sizeof(QStaticAssertFailure<!!(Condition)>)}

I have just read that: Values of a registered type can be constructed dynamically via QMetaType::construct()
There is an example in the link above. Thanks for your help! I will figure it out, still missing pure c++ knowledge ;)

I couldn't quite believe that this could be an actual bug, so I kept testing.
And the result... oh well... I'm truly ashamed... I have included QDataStream in every file... except the one I actually tested.
So the solution is: #include <QDataStream> Facepalm
Thanks for your tips, I'm now gonna implement what you told me.

@maximus

Sometimes the answer is right in front of you :)

@Afterwork said:

I recreate them every time. Are there better way ?

If they change then there's no way around it but if they're the same every frame then it's a waste of time. Generate the list once, store it as a class member and just expose them by reference e.g.

//split that to .h and .cpp of course class GameWidget : public QWidget { Q_OBJECT public: GameWidget(QWidget* parent = nullptr) : QWidget(parent) { generateMyListOfObjects(); } const QVector<Object>& objects() const { return objects_; } private: void generateMyListOfObjects() { //fill the objects vector here } QVector<Object> objects_; };

And how i can connect the slot of differents objects to built-in timer of QWidget ?

You don't. As I said - reimplement the timerEvent method. It will be called every time the timer times out. You can call some methods directly in it or emit a custom signal if you'd like.

Hi,

You can use the Animation Framework for that. But rather than painting on a widget since you seem to be writing a game, you should take a look at the GraphicsView Framework

My point is simply that blind adherence a principle or method can actually be worse than fully understanding the needs and goals of what you are trying to do.

Amen.

Hi,
Thank you both for your suggestions!

As this project is intended for industrial purposes I think XML is a great solution.
I followed some tutorials on how to use XML on Qt and managed to make it work (as well as understand) pretty well for what I intended to do.

I'm leaving here the tutorials I followed in case someone stumbles upon this thread with the same problem: (I apologize if this is not allowed)
https://www.youtube.com/watch?v=NXGE5XUrRSI
https://www.youtube.com/watch?v=NzQwJdcdRKE
I had to make some changes to my project but it's easy to understand.

Thank you, once again!