Fixing bug despawning pickable after collected

Author: moreirayokoyama

src/character.rs

@@ -11,9 +11,10 @@ use bevy::{
 use bevy_rapier2d::prelude::*;
 
 use crate::{
-    control::CharacterControlInput, pickables::PlacedPickable, GameWorld, BLOCK_SIZE,
-    CHARACTER_JUMP_SPEED, CHARACTER_MOVEMENT_SPEED, CHARACTER_SIZE, GRAVITY, PIXEL_PERFECT_LAYERS,
-    WORLD_BOTTOM_OFFSET_IN_PIXELS, WORLD_CENTER_COL,
+    control::CharacterControlInput,
+    pickables::{PlacedPickable, PlacedPickableCollected},
+    GameWorld, BLOCK_SIZE, CHARACTER_JUMP_SPEED, CHARACTER_MOVEMENT_SPEED, CHARACTER_SIZE, GRAVITY,
+    PIXEL_PERFECT_LAYERS, WORLD_BOTTOM_OFFSET_IN_PIXELS, WORLD_CENTER_COL,
 };
 
 const GROUND_TIMER: f32 = 0.5;
@@ -278,7 +279,9 @@ fn handle_collision(
         for collision in &controller_output.collisions {
             if let Ok(placed_pickable) = placed_pickables.get(collision.entity) {
                 coin_pouch.0 += placed_pickable.item_type.get_coins();
-                commands.entity(placed_pickable.entity).despawn();
+                commands.trigger(PlacedPickableCollected {
+                    entity: placed_pickable.entity,
+                });
                 commands.entity(collision.entity).despawn();
             }
         }

src/pickables.rs

@@ -15,7 +15,8 @@ impl Plugin for PickablesPlugin {
     fn build(&self, app: &mut bevy::prelude::App) {
         app.observe(on_new_day)
             .observe(on_new_chunk)
-            .add_systems(Startup, (initialize, load_textures, startup).chain());
+            .observe(on_pickable_collected)
+            .add_systems(Startup, (load_textures, startup).chain());
     }
 }
 
@@ -64,25 +65,21 @@ impl Distribution<PickableItemType> for Standard {
     }
 }
 
-#[derive(Resource)]
-pub struct PickableCount(u8);
-
 #[derive(Component)]
 pub struct Pickable {
-    pub id: u8,
     pub item_type: PickableItemType,
     pub x: usize,
 }
 
 #[derive(Component)]
 pub struct PlacedPickable {
-    pub id: u8,
     pub entity: Entity,
     pub item_type: PickableItemType,
 }
 
-fn initialize(mut commands: Commands) {
-    commands.insert_resource(PickableCount(0));
+#[derive(Event)]
+pub struct PlacedPickableCollected {
+    pub entity: Entity,
 }
 
 fn load_textures(
@@ -95,13 +92,11 @@ fn load_textures(
     commands.insert_resource(Tiles(asset_server.load("purple-valley-icon-set/icons.png")));
 }
 
-fn startup(mut commands: Commands, game_world: Res<GameWorld>, mut counter: ResMut<PickableCount>) {
+fn startup(mut commands: Commands, game_world: Res<GameWorld>) {
     let mut rng = thread_rng();
     let pickables_count = rng.gen_range(8..64) as usize;
     for _ in 0..pickables_count {
-        counter.0 += 1;
         commands.spawn(Pickable {
-            id: counter.0,
             item_type: random(),
             x: game_world.get_random_x_block(),
         });
@@ -130,7 +125,6 @@ fn on_new_chunk(
         for (entity, item) in items {
             parent.spawn((
                 PlacedPickable {
-                    id: item.id,
                     entity,
                     item_type: item.item_type.clone(),
                 },
@@ -157,3 +151,15 @@ fn on_new_chunk(
         }
     });
 }
+
+fn on_pickable_collected(
+    trigger: Trigger<PlacedPickableCollected>,
+    query: Query<(Entity, &Pickable)>,
+    mut commands: Commands,
+) {
+    if let Ok((entity, _)) = query.get(trigger.event().entity) {
+        commands.entity(entity).despawn();
+    } else {
+        println!("Pickable not found: {:?}", trigger.event().entity);
+    }
+}