Results 1 to 7 of 7

Thread: QAbstractItemModel::reset() obsolete?

  1. #1
    Join Date
    Jul 2012
    Posts
    244
    Thanks
    27
    Thanked 15 Times in 14 Posts
    Qt products
    Qt4
    Platforms
    Windows

    Unhappy QAbstractItemModel::reset() obsolete?

    Hi,


    I have a model whose backingstore can change or outright die without me knowing what exactly changed, just that something changed. So I need to reset the entire model. I figured the method reset() is the perfect match here:

    https://doc.qt.io/qt-5/qabstractitem...ete.html#reset

    But it's obsolete, and Qt doc suggest I replace it with a beginResetModel() + endResetModel().


    Problem is obviously that I dont know when my model is about to be reset, so I cant call beginResetModel before.



    What's the protocol here? Should I just ignore the fact that the data is already reset and do

    Qt Code:
    1. beginResetModel()
    2. endResetModel()
    To copy to clipboard, switch view to plain text mode 

    Documentation explicitly says:

    You must call this function before resetting any internal data structures in your model or proxy model.

    or should I just emit a lone "endResetModel()" call? That feels wrong.

  2. #2
    Join Date
    Jan 2008
    Location
    Alameda, CA, USA
    Posts
    5,229
    Thanks
    302
    Thanked 864 Times in 851 Posts
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: QAbstractItemModel::reset() obsolete?

    The call to beginResetModel() ensures that any view or proxy that is connected to your model will not attempt to do any updates until the endResetModel() method is called. While the model is being reset, pointers, model indexes, and other things used by views or proxies could become invalid and thus cause a crash.

    Problem is obviously that I dont know when my model is about to be reset, so I cant call beginResetModel before.
    That seems really unlikely. It's your code after all. Somewhere you have a method that must rebuild the internal data structure used by your model, and that's the place where you would call the begin / endResetModel() methods. Somewhere else, you must have a method that gets called to trigger that rebuild. It can't just decide on its own...

    If you don't have such a place (eg. you are simply replacing a pointer to an internal data structure inside your model wrapper on a reset), then you have to ensure that you call beginResetModel() before messing around with the object the pointer references or create a new, duplicate instance of the data structure so that the old one continues to remain valid until it is replaced.
    <=== The Great Pumpkin says ===>
    Please use CODE tags when posting source code so it is more readable. Click "Go Advanced" and then the "#" icon to insert the tags. Paste your code between them.

  3. The following user says thank you to d_stranz for this useful post:

    tuli (15th January 2019)

  4. #3
    Join Date
    Jul 2012
    Posts
    244
    Thanks
    27
    Thanked 15 Times in 14 Posts
    Qt products
    Qt4
    Platforms
    Windows

    Default Re: QAbstractItemModel::reset() obsolete?

    My model is read-only, does that change anything?


    The model is grabbing data from the network / from a networkshare. That share might go down without notice, and i have to handle it gracefully. Surely there must be some way to handle that? As far as I can tell the reset() method that is now labeled "obsolete" did exactly that...

    SqlModels should have the exact same problem with remote databases, so I find it difficult to imagine this problem hasnt come up so far.

  5. #4
    Join Date
    Jan 2008
    Location
    Alameda, CA, USA
    Posts
    5,229
    Thanks
    302
    Thanked 864 Times in 851 Posts
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: QAbstractItemModel::reset() obsolete?

    My model is read-only, does that change anything?
    I presume you mean read-only with respect to views or proxies, right? No, it doesn't make any difference - if a view or proxy can call data() on your model and get non-valid results, then it's a problem.

    The model is grabbing data from the network / from a networkshare. That share might go down without notice, and i have to handle it gracefully. Surely there must be some way to handle that? ... SqlModels should have the exact same problem with remote databases
    Is your app checking to see that the network connection is still alive? (Just like I presume SQL drivers also have some sort of "heartbeat" monitor to see that their connection to a remote database is still alive). Do you observe network connection timeouts when you get no response to a request for more data? In any case, you should make your app robust enough that it can detect that the remote share has gone down and "fail" gracefully. Failure then becomes a matter of either freezing in place the last set of valid data or deleting the data if having stale data is inappropriate, and having your app tell the user that the connection was lost.

    And even when the connection is still alive and well, either you are polling the share to see if anything has changed or you are waiting for the share to send you a "data changed" signal of some sort - it is at the point where you handle that change that you need to call the begin / endResetModel() pair.

    The only way you might be able to get away with not doing it is if your app (and everything it depends on) is single-threaded and you can guarantee that there will be no event processing from the time you begin your update until the end - that is, no signals emitted or slots invoked where your model data is involved. Then you can probably call begin / end immediately in sequence and things will be fine. But network communications are inherently asychronous and almost certainly multithreaded, so I would be cautious in assuming you could get away with it.
    Last edited by d_stranz; 15th January 2019 at 02:09.
    <=== The Great Pumpkin says ===>
    Please use CODE tags when posting source code so it is more readable. Click "Go Advanced" and then the "#" icon to insert the tags. Paste your code between them.

  6. The following user says thank you to d_stranz for this useful post:

    tuli (15th January 2019)

  7. #5
    Join Date
    Jul 2012
    Posts
    244
    Thanks
    27
    Thanked 15 Times in 14 Posts
    Qt products
    Qt4
    Platforms
    Windows

    Default Re: QAbstractItemModel::reset() obsolete?

    Sadly it's a multithreaded up, but yes I check and know when the share goes down. At that point I used to call reset(). But I dont know how I could replace that reset() call at this point with the new beg..end..() syntax, since the data is gone at that point.

    Failure then becomes a matter of either freezing in place the last set of valid data or deleting the data if having stale data is inappropriate, and having your app tell the user that the connection was lost.
    I want to delete the data, but I dont understand how I can do that? (There is no cache of my own between the model and the network share or something like that.)

  8. #6
    Join Date
    Jan 2008
    Location
    Alameda, CA, USA
    Posts
    5,229
    Thanks
    302
    Thanked 864 Times in 851 Posts
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: QAbstractItemModel::reset() obsolete?

    There is no cache of my own between the model and the network share or something like that.
    Then I guess I don't understand how you have implemented your model. In most cases of a custom model, the model is implemented as a wrapper around some other custom data structure. The app updates the custom data structure and tells the model to notify its listeners (views or proxies) that the data is changing / has changed, through the various begin... / end... method pairs. Even the specialized models (QSqlQuery, QFileSystemModel, etc.) are wrappers around data structures that map external data into the Model / View architecture and take care of these notifications internally.
    <=== The Great Pumpkin says ===>
    Please use CODE tags when posting source code so it is more readable. Click "Go Advanced" and then the "#" icon to insert the tags. Paste your code between them.

  9. #7
    Join Date
    Jan 2006
    Location
    Graz, Austria
    Posts
    8,416
    Thanks
    37
    Thanked 1,544 Times in 1,494 Posts
    Qt products
    Qt3 Qt4 Qt5
    Platforms
    Unix/X11 Windows

    Default Re: QAbstractItemModel::reset() obsolete?

    If you have no local data then simply do the begin/end calls.

    Cheers,
    _

  10. The following user says thank you to anda_skoa for this useful post:

    tuli (17th January 2019)

Similar Threads

  1. setHandlesChildEvents() obsolete?
    By SvenA in forum Qt Programming
    Replies: 1
    Last Post: 12th September 2016, 17:29
  2. QAbstractItemModel reset() method
    By lotek in forum Qt Programming
    Replies: 1
    Last Post: 20th September 2011, 15:55
  3. QAbstractItemModel reset Persistent Index
    By larcho in forum Qt Programming
    Replies: 8
    Last Post: 5th April 2010, 06:01
  4. Replies: 1
    Last Post: 9th February 2010, 13:11
  5. Is QWSServer obsolete now?
    By hybrid_snyper in forum Qt for Embedded and Mobile
    Replies: 2
    Last Post: 20th October 2009, 16:05

Tags for this Thread

Bookmarks

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  
Digia, Qt and their respective logos are trademarks of Digia Plc in Finland and/or other countries worldwide.