much improved item buy button rendering

src/logic/shop/item.rs

@@ -14,17 +14,15 @@ pub struct ShopItemWrapper<T: ItemBase + Clone> {
 
 impl<T: ItemBase + Clone> ShopItemWrapper<T> {
     pub fn new(
-        name: String,
         item: T,
         from_inventory: &Option<T>,
         first_item_bounds: Rectangle,
-        index: u8,
+        index: u8
-        player: &Player,
     ) -> Self {
         // Build new bounds for the UI row
         let new_bounds = Rectangle {
             x: first_item_bounds.x,
-            y: first_item_bounds.y + (first_item_bounds.height + 5.0 * index as f32),
+            y: first_item_bounds.y + ((first_item_bounds.height + 5.0) * index as f32),
             width: first_item_bounds.width,
             height: first_item_bounds.height,
         };
@@ -32,7 +30,7 @@ impl<T: ItemBase + Clone> ShopItemWrapper<T> {
         Self {
             bounds: new_bounds,
             ui: ShopItemUi::new(
-                name,
+                item.get_name(),
                 match from_inventory {
                     Some(x) => x.get_level(),
                     None => 0,

src/logic/shop/mainui.rs

@@ -1,11 +1,8 @@
 use raylib::prelude::*;
 
-use crate::{
+use crate::{gamecore::{GameCore, GameState}, items::{AirBag, Flashlight, Flippers, ItemBase, StunGun}, lib::wrappers::audio::player::AudioPlayer};
-    gamecore::{GameCore, GameState},
-    lib::wrappers::audio::player::AudioPlayer,
-};
 
-use super::itemui::ShopItemUi;
+use super::{item::ShopItemWrapper, itemui::ShopItemUi};
 
 pub fn render_shop(
     draw_handle: &mut RaylibDrawHandle,
@@ -27,90 +24,71 @@ pub fn render_shop(
         Color::BLACK,
     );
 
-    // Stun Gun
+    // Bounds for use in item row sizing
-    let stun_gun_buy_ui = ShopItemUi::new(
+    let first_bounds = Rectangle {
-        "Stun Gun".to_string(),
-        match &game_core.player.inventory.stun_gun {
-            Some(x) => x.level,
-            None => 0,
-        },
-        3,
-        10,
-    );
-    stun_gun_buy_ui.render(
-        draw_handle,
-        Rectangle {
         x: bounds.x + 5.0,
         y: bounds.y + 100.0,
         width: bounds.width - 10.0,
         height: 50.0,
-        },
+    };
-        game_core.player.coins >= stun_gun_buy_ui.cost,
-    );
 
-    // Flippers
+    // Create items
-    let flippers_buy_ui = ShopItemUi::new(
+    let stun_gun_buy_ui = ShopItemWrapper::new(
-        "Flippers".to_string(),
+        match &game_core.player.inventory.stun_gun {
-        match &game_core.player.inventory.flippers {
+            Some(x) => match x.get_level() {
-            Some(x) => x.level,
+                1 => StunGun::lvl2(),
-            None => 0,
+                _ => StunGun::lvl3(),
             },
-        3,
+            None => StunGun::lvl1(),
-        10,
-    );
-    flippers_buy_ui.render(
-        draw_handle,
-        Rectangle {
-            x: bounds.x + 5.0,
-            y: bounds.y + 160.0,
-            width: bounds.width - 10.0,
-            height: 50.0,
         },
-        game_core.player.coins >= flippers_buy_ui.cost,
+        &game_core.player.inventory.stun_gun,
+        first_bounds,
+        0,
     );
-
+    let air_bag_buy_ui = ShopItemWrapper::new(
-    // Flashlight
-    let flashlight_buy_ui = ShopItemUi::new(
-        "Flashlight".to_string(),
-        match &game_core.player.inventory.flashlight {
-            Some(x) => x.level,
-            None => 0,
-        },
-        3,
-        10,
-    );
-    flashlight_buy_ui.render(
-        draw_handle,
-        Rectangle {
-            x: bounds.x + 5.0,
-            y: bounds.y + 220.0,
-            width: bounds.width - 10.0,
-            height: 50.0,
-        },
-        game_core.player.coins >= flashlight_buy_ui.cost,
-    );
-
-    // Air Bag
-    let air_bag_buy_ui = ShopItemUi::new(
-        "Bag of Air".to_string(),
         match &game_core.player.inventory.air_bag {
-            Some(x) => x.level,
+            Some(x) => match x.get_level() {
-            None => 0,
+                1 => AirBag::lvl2(),
+                _ => AirBag::lvl3(),
             },
+            None => AirBag::lvl1(),
+        },
+        &game_core.player.inventory.air_bag,
+        first_bounds,
+        1,
+    );
+    let flashlight_buy_ui = ShopItemWrapper::new(
+        match &game_core.player.inventory.flashlight {
+            Some(x) => match x.get_level() {
+                1 => Flashlight::lvl2(),
+                _ => Flashlight::lvl3(),
+            },
+            None => Flashlight::lvl1(),
+        },
+        &game_core.player.inventory.flashlight,
+        first_bounds,
+        2,
+    );
+    let flippers_buy_ui = ShopItemWrapper::new(
+        match &game_core.player.inventory.flippers {
+            Some(x) => match x.get_level() {
+                1 => Flippers::lvl2(),
+                _ => Flippers::lvl3(),
+            },
+            None => Flippers::lvl1(),
+        },
+        &game_core.player.inventory.flippers,
+        first_bounds,
         3,
-        10,
-    );
-    air_bag_buy_ui.render(
-        draw_handle,
-        Rectangle {
-            x: bounds.x + 5.0,
-            y: bounds.y + 280.0,
-            width: bounds.width - 10.0,
-            height: 50.0,
-        },
-        game_core.player.coins >= air_bag_buy_ui.cost,
     );
 
+    // Render items
+    stun_gun_buy_ui.render(draw_handle, &game_core.player);
+    air_bag_buy_ui.render(draw_handle, &game_core.player);
+    flashlight_buy_ui.render(draw_handle, &game_core.player);
+    flippers_buy_ui.render(draw_handle, &game_core.player);
+
+
     return None;
 }