[Part 14] Item Collection Bug

[Tarha Apustaja]

Helpful?

Up

0

::

It sounds like you’re experiencing an issue where the tomatoes collected in your game sometimes overwrite the items in your hands. This could be due to a few different reasons in your code. To help you troubleshoot, here are some steps you can take:

1. Check Inventory Logic: Ensure that the logic handling the transfer of tomatoes from the ground to the inventory is consistent with the logic for cabbages. If the code for tomatoes is slightly different, it could lead to the overwriting issue.

2. Review Collection Conditions: Make sure the conditions under which the tomatoes are added to the inventory are correctly defined. There might be cases where the condition is not being met, leading to the item being overwritten.

3. Debugging Statements: Add debugging statements in your code to track when tomatoes are picked up and added to the inventory. This can help you see if there are specific cases where the logic is failing.

4. Check for Race Conditions: If your game has any asynchronous operations (like using timers or event listeners), race conditions might cause unexpected behavior. Ensure that the code that manages inventory is thread-safe.

5. Review the Overwrite Logic: If there’s specific code that controls when items overwrite the current items in hand, review that logic to confirm it’s not being triggered incorrectly.

6. Test Cases: Create specific test cases to simulate the collection of tomatoes under different scenarios (e.g., inventory full, partial inventory, etc.) to see if you can replicate the issue consistently.

If you can provide a snippet of the relevant code, I can offer more targeted suggestions!

[Jonathan Teo]

Helpful?

Up

0

::

Can you record youself checking the inventory state with each harvest? Or with the inspector showing the state of the inventory

[pwp]

Helpful?

Up

0

::

“I still don’t quite understand why this bug happens. The second video seems to show that I need a certain condition to collect one of the tomatoes from the tomato crop, and that is the tomato that can be sent to the inventory.

Situation 1: When I collect the crop 30 seconds ago, as we intended, it disappears after being harvested, but it ends up overwriting the hand slot.

Situation 2: 30 seconds later, when I collect the tomato, the first tomato I collect from the tomato crop doesn’t disappear, but it can successfully be sent to the inventory. After collecting again, it triggers situation 1 again.”

These two situations usually happen simultaneously, so I’m also not sure how to fix it.

[Tarha Apustaja]

Helpful?

Up

0

::

It sounds like you’re dealing with a bug related to the harvesting and inventory system in your game. Based on your description of the two situations, it seems like there might be an issue with how the game handles item collection and the state of the items in the inventory.

Here are a few steps you can take to debug and possibly resolve the issue:

1. Check Item State Management: Ensure that when an item is collected, its state is being updated correctly. In Situation 1, it sounds like the item is being marked as collected but still appears in the hand slot. Make sure that the item is properly removed from the game world and that its state is updated in the inventory system.

2. Review Collection Logic: Look at the logic that handles the collection of items. Ensure that there’s a clear distinction between when an item should be added to the inventory and when it should disappear from the game world. There might be a conditional statement that’s not firing correctly.

3. Debug Logging: Add debug logs to track the state of the item before and after collection. This will help you understand what happens to the item during the collection process. For example, log the item’s state when you try to collect it and when it’s added to the inventory.

4. Investigate Timing Issues: Since you mentioned a timing aspect (collecting the item 30 seconds later), consider whether there might be timing-related bugs, such as race conditions or delays in state updates. Ensure that any timers or delays are functioning as expected.

5. Check for Overlapping Events: Make sure there are no overlapping events that might be interfering with the collection process. For example, if the player collects the item while another event is taking place, it might lead to unexpected behavior.

6. Test with Different Conditions: Try to replicate the issue under various conditions. For instance, test the collection process with different crops or items to see if the issue persists. This can help narrow down whether it’s a specific problem with the tomato crop or a more general issue.

7. Review Game Logic: Double-check the game logic responsible for managing crops and inventory. Look for any flaws or inconsistencies that might lead to the behaviors you’re observing.

If you can provide more details about the code related to item collection and inventory management, I could help you troubleshoot further.

[Jonathan Teo]

Helpful?

Up

1

::

Ah ok I realise the problem. Apply the same fix on the Pickup function in RegrowableHarvestBehaviour and you should be good to go.

• Terence

has upvoted this post.

[Author]

Posts