Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • Users
  • Groups
  • Search
  • Get Qt Extensions
  • Unsolved
Collapse
Brand Logo
  1. Home
  2. General talk
  3. The Lounge
  4. Quiz Time! Prepending items to QVector
Forum Updated to NodeBB v4.3 + New Features

Quiz Time! Prepending items to QVector

Scheduled Pinned Locked Moved Solved The Lounge
22 Posts 12 Posters 4.9k Views 7 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • aha_1980A aha_1980

    Hi @VRonin,

    you first have to mention the Qt version you are using.

    Are you still on 5.15 or already on a 6.x series where lots of optimizations for that case have been done?

    Regards

    VRoninV Offline
    VRoninV Offline
    VRonin
    wrote on last edited by
    #3

    @aha_1980 great question! I ran the benchmark for both 5.15.2 and 6.0.3 so chose your poison

    "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
    ~Napoleon Bonaparte

    On a crusade to banish setIndexWidget() from the holy land of Qt

    JonBJ 1 Reply Last reply
    0
    • VRoninV VRonin

      @aha_1980 great question! I ran the benchmark for both 5.15.2 and 6.0.3 so chose your poison

      JonBJ Offline
      JonBJ Offline
      JonB
      wrote on last edited by
      #4

      @VRonin
      My guess is that append + reverse is more efficient than prepend.

      My reason is why would you even bother to ask this question if prepend is more efficient?

      :)

      VRoninV 1 Reply Last reply
      0
      • JonBJ JonB

        @VRonin
        My guess is that append + reverse is more efficient than prepend.

        My reason is why would you even bother to ask this question if prepend is more efficient?

        :)

        VRoninV Offline
        VRoninV Offline
        VRonin
        wrote on last edited by
        #5

        @JonB said in Quiz Time! Prepending items to QVector:

        My reason is why would you even bother to ask this question if prepend is more efficient?

        I asked myself this question before benchmarking, that’s why

        "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
        ~Napoleon Bonaparte

        On a crusade to banish setIndexWidget() from the holy land of Qt

        1 Reply Last reply
        0
        • Chris KawaC Online
          Chris KawaC Online
          Chris Kawa
          Lifetime Qt Champion
          wrote on last edited by
          #6

          How many elements? can I use reserve? Are we talking 32 or 64 bits?
          My cheat answer would by - it depends ;)

          VRoninV 1 Reply Last reply
          0
          • jeremy_kJ Offline
            jeremy_kJ Offline
            jeremy_k
            wrote on last edited by
            #7

            Am I misreading the problem, or does this also need to have the initial sequence reversed prior to the append + reverse operation?

            Asking a question about code? http://eel.is/iso-c++/testcase/

            Chris KawaC 1 Reply Last reply
            0
            • jeremy_kJ jeremy_k

              Am I misreading the problem, or does this also need to have the initial sequence reversed prior to the append + reverse operation?

              Chris KawaC Online
              Chris KawaC Online
              Chris Kawa
              Lifetime Qt Champion
              wrote on last edited by
              #8

              @jeremy_k My understanding was that we're starting from an empty container and only need it in correct order after everything is in, otherwise you'd need to do that double reversal on each addition. I could be wrong though.

              1 Reply Last reply
              1
              • fcarneyF Offline
                fcarneyF Offline
                fcarney
                wrote on last edited by
                #9

                It depends, I think.
                I don't know, but I am putting on depends in case the answer is shocking.

                C++ is a perfectly valid school of magic.

                1 Reply Last reply
                1
                • jeremy_kJ Offline
                  jeremy_kJ Offline
                  jeremy_k
                  wrote on last edited by jeremy_k
                  #10

                  If the container starts empty, I would expect using a reverse iterator for reading and append for writing would beat both strategies mentioned in the initial post. Unless QVector has become a single linked list (edit: or the consumer demands a QVector).

                  Asking a question about code? http://eel.is/iso-c++/testcase/

                  1 Reply Last reply
                  0
                  • Kent-DorfmanK Offline
                    Kent-DorfmanK Offline
                    Kent-Dorfman
                    wrote on last edited by Kent-Dorfman
                    #11

                    My knowledge base is more in line with stl containers, but making the asusmption that QVector works the same, there is implicit preallocation ie reserve(n) involved in append+reverse that probably doesn't exist with the prepend call, so it stands to reason that few memory reallocations may be necesary using append+reverse.

                    but the larger issue is whether O(1) random access is needed or guaranteeed continuous element allocation...if not, the QList would be a much more efficient container.

                    I light my way forward with the fires of all the bridges I've burned behind me.

                    1 Reply Last reply
                    0
                    • J.HilkJ Offline
                      J.HilkJ Offline
                      J.Hilk
                      Moderators
                      wrote on last edited by
                      #12

                      In case you do not reserve memory for the QVector beforehand, I would say append + std::reverse is faster

                      Because I think, that prepend does not reserve additional memory, compared to append(size / 2 more IIRC). So you end up with an allocation and memory move/copy on each prepend


                      Be aware of the Qt Code of Conduct, when posting : https://forum.qt.io/topic/113070/qt-code-of-conduct


                      Q: What's that?
                      A: It's blue light.
                      Q: What does it do?
                      A: It turns blue.

                      1 Reply Last reply
                      0
                      • Chris KawaC Chris Kawa

                        How many elements? can I use reserve? Are we talking 32 or 64 bits?
                        My cheat answer would by - it depends ;)

                        VRoninV Offline
                        VRoninV Offline
                        VRonin
                        wrote on last edited by
                        #13

                        How many elements?

                        The benchmark tested 10, 50, 100, 500, 1000, 5000 but turns out the answer doesn't change with size

                        can I use reserve?

                        Yes, in both cases I first called QVector::reserve

                        Are we talking 32 or 64 bits?

                        64

                        I would expect using a reverse iterator for reading and append for writing would beat both strategies mentioned in the initial post.

                        Yes, this is obvious but in my use case I can't read with reverse iterator

                        or does this also need to have the initial sequence reversed prior to the append + reverse operation?

                        To make it more explicit: If I have the sequence 1, 3, 2, 5 (that I can't reverse iterate) I want the vector to store those as 5, 2, 3, 1

                        "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
                        ~Napoleon Bonaparte

                        On a crusade to banish setIndexWidget() from the holy land of Qt

                        1 Reply Last reply
                        0
                        • VRoninV Offline
                          VRoninV Offline
                          VRonin
                          wrote on last edited by VRonin
                          #14

                          And the answer is:
                          In Qt5 append+std::reverse is vastly superior, In Qt6 prepend is the much faster one.

                          Qt5.PNG
                          Qt6.PNG

                          "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
                          ~Napoleon Bonaparte

                          On a crusade to banish setIndexWidget() from the holy land of Qt

                          1 Reply Last reply
                          6
                          • fcarneyF Offline
                            fcarneyF Offline
                            fcarney
                            wrote on last edited by
                            #15

                            What about just starting at the end of the data first and only appending?

                            C++ is a perfectly valid school of magic.

                            VRoninV 1 Reply Last reply
                            0
                            • fcarneyF fcarney

                              What about just starting at the end of the data first and only appending?

                              VRoninV Offline
                              VRoninV Offline
                              VRonin
                              wrote on last edited by
                              #16

                              @fcarney said in Quiz Time! Prepending items to QVector:

                              What about just starting at the end of the data first and only appending?

                              I guess you mean in Qt6. In that case append is slightly better but the difference is really tiny

                              "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
                              ~Napoleon Bonaparte

                              On a crusade to banish setIndexWidget() from the holy land of Qt

                              1 Reply Last reply
                              0
                              • S Offline
                                S Offline
                                SimonSchroeder
                                wrote on last edited by
                                #17

                                Is there any article about these specific changes in Qt 6 that I can read? What exactly has changed between Qt 5 and 6?

                                Christian EhrlicherC 1 Reply Last reply
                                0
                                • S SimonSchroeder

                                  Is there any article about these specific changes in Qt 6 that I can read? What exactly has changed between Qt 5 and 6?

                                  Christian EhrlicherC Offline
                                  Christian EhrlicherC Offline
                                  Christian Ehrlicher
                                  Lifetime Qt Champion
                                  wrote on last edited by
                                  #18

                                  @SimonSchroeder said in Quiz Time! Prepending items to QVector:

                                  What exactly has changed between Qt 5 and 6?

                                  QVector is no longer in Qt6 and an alias for QList. I would guess QList in Qt5 will also show the same fast results for prepend since QList has an optimization for this usecase since ages.

                                  Qt Online Installer direct download: https://download.qt.io/official_releases/online_installers/
                                  Visit the Qt Academy at https://academy.qt.io/catalog

                                  1 Reply Last reply
                                  2
                                  • VRoninV Offline
                                    VRoninV Offline
                                    VRonin
                                    wrote on last edited by
                                    #19

                                    From https://wiki.qt.io/New_Features_in_Qt_6.0

                                    QVector and QList are unified. QList is updated and should be used by default when array-like behaviour is needed
                                    QList, QString and QByteArray now have optimized complexity of insertion at the beginning (a.k.a. prepend)

                                    "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
                                    ~Napoleon Bonaparte

                                    On a crusade to banish setIndexWidget() from the holy land of Qt

                                    1 Reply Last reply
                                    0
                                    • VRoninV VRonin

                                      I had a case where I found myself with a loop that had to prepend items to a QVector<MyClass*> and I asked myself what would be more efficient:

                                      1. repeatedly calling QVector::prepend
                                      2. repeatedly calling QVector::append and then std::reverse at the end

                                      I actually benchmarked it but before sharing the results let's make it a challenge.
                                      The person who gets it right will be praised as holder of infinite wisdom forever.

                                      Before you ask, no this is still not peak nerd!

                                      kshegunovK Offline
                                      kshegunovK Offline
                                      kshegunov
                                      Moderators
                                      wrote on last edited by kshegunov
                                      #20

                                      @VRonin I'm late to the party, as usual ...
                                      ... and also as usual I probably sound like a broken record, but what your task sounds like is actually a stack, not a vector ... ;)

                                      Read and abide by the Qt Code of Conduct

                                      VRoninV 1 Reply Last reply
                                      2
                                      • kshegunovK kshegunov

                                        @VRonin I'm late to the party, as usual ...
                                        ... and also as usual I probably sound like a broken record, but what your task sounds like is actually a stack, not a vector ... ;)

                                        VRoninV Offline
                                        VRoninV Offline
                                        VRonin
                                        wrote on last edited by VRonin
                                        #21

                                        @kshegunov said in Quiz Time! Prepending items to QVector:

                                        but what your task sounds like is actually a stack, not a vector

                                        Indeed, this was an iper-semplification of my problem to solve the academic dilemma.
                                        I agree that a stack or, as @jeremy_k already pointed out, using reverse iterators would achieve the same

                                        "La mort n'est rien, mais vivre vaincu et sans gloire, c'est mourir tous les jours"
                                        ~Napoleon Bonaparte

                                        On a crusade to banish setIndexWidget() from the holy land of Qt

                                        1 Reply Last reply
                                        2
                                        • D Offline
                                          D Offline
                                          Dixon154
                                          Banned
                                          wrote on last edited by
                                          #22
                                          This post is deleted!
                                          1 Reply Last reply
                                          0

                                          • Login

                                          • Login or register to search.
                                          • First post
                                            Last post
                                          0
                                          • Categories
                                          • Recent
                                          • Tags
                                          • Popular
                                          • Users
                                          • Groups
                                          • Search
                                          • Get Qt Extensions
                                          • Unsolved