Thread: How to get the index of a vector of tuples

It sounds like the problem is that you're not actually using the item choice, i.e., you're just looping through the whole inventory after the choice has been made. What I might do is this:

1. Check that the choice is valid, i.e., that the read was successful and that the value of choice maps to a valid index in the inventory vector. If not, you request that the user make a valid choice.
2. Access the item at the computed index (i.e., subtract 1) in the inventory vector.
3. Apply the item's effect. It is here that if the effect is healing, then you heal the player unless the player is already at full health.
4. If the item has been successfully used (e.g., the effect is healing and the player is not at full health), then you subtract 1 from the number of items.

Originally Posted by CH1156

But how does that look in code? thats where im confused. I cant seem to figure out how to get the index of the vector and the tuple, if it was a normal vector i could just do something like:

mInventory[choice]; without needing a for loop or iterator.

It's mInventory[choice - 1] because you numbered the choices starting from 1. So you would write get<0>(mInventory[choice - 1]) to get the Item itself.

I think you need to realise that range-based for loops are meant for those times when you want to iterate over the entire container (or range, etc). You seem to be in the habit of reaching for them at every opportunity, even when it doesn't make sense.

What you have is an inventory usage loop that allows the user to use one amount of an item at one time. This means that when the count for an item reaches zero, you know exactly which item it is by index. Therefore you only need to compute the iterator from the index in order to pass it to erase, and that's easy: begin() + index gives you that iterator (or more generically, there's std::advance).

Yeah I realized that mistake and removed the for loop from my first post and that works fine, but with this I want to remove every element where the tuple has a value of 0 for the second element of the tuple so i thought a for loop was best but i'll just not use a for loop for this.. I have tried these methods and none work:

Try 1

Code:

if (get<1>(i) == 0)
{
      mInventory.erase(mInventory.begin() + get<1>(mInventory[choice - 1]));
 }

Try 2:

Code:

if (get<1>(i) == 0)
{
    mInventory.erase(mInventory.begin() + get<1>(i));
}

Try 3

This works but i dont know if there is some hidden problems with it. Please let me know if there is.

The syntax is difficult to understand sometimes :/

Code:

if (get<1>(mInventory[choice - 1]) == 0)
{
   mInventory.erase(mInventory.begin() + choice - 1);
}

Originally Posted by CH1156

I want to remove every element where the tuple has a value of 0 for the second element of the tuple

For that, the typical approach is to #include <algorithm> and write:

Code:

mInventory.erase(
    std::remove_if(
        mInventory.begin(),
        mInventory.end(),
        [](const auto& item) { return std::get<1>(item) == 0; }
    ),
    mInventory.end()
);

remove_if is used because now you want to remove elements from the vector if they satisfy the condition specified in the lambda. However, remove_if "removes" by shifting the elements not removed to the front of the range, returning an iterator to one past the end of the new valid range. So we take this iterator and pass it to mInventory's range-based erase member function, so that it can erase those "removed" elements.

This approach is suitable if you want the player to be able to use multiple items and have their counts set to 0, and then later you perform clean up to erase all these zero-count items. It wouldn't make sense to use it for the scenario you described in post #6 ("if the player has 3 weak potions and they use alll of them, how would I remove the tuple element of that item") because it would be overkill, i.e., you'll be removing all zero-count items (and you don't know which ones they are) when there is exactly one zero-count item and you know exactly what it is.

Originally Posted by CH1156

This works but i dont know if there is some hidden problems with it. Please let me know if there is.

The "hidden" problem is that you're using the choice variable. In this scenario of removing every element, the choice variable conceptually does not exist: there is no choice because if you're zero, you're removed, no appeal granted. The player is not allowed to choose anything (those choices have already been made and the items used, so the time to choose has passed), hence what you're doing is conceptually wrong.

However, if you're trying to solve for the problem of "if the player has 3 weak potions and they use alll of them, how would I remove the tuple element of that item", then this is correct, because at this point, the player has chosen the weak potion, and you check its count after usage and find that it is zero, so you erase it. That's what I was talking about in my previous post.