add non-access methods for resources existence

Author: MiniaczQ

crates/bevy_ecs/src/schedule/executor/mod.rs

@@ -202,7 +202,7 @@ mod tests {
     #[test]
     fn invalid_system_param_skips() {
         for executor in EXECUTORS {
-            invalid_system_param_skips_core(executor)
+            invalid_system_param_skips_core(executor);
         }
     }
 
@@ -235,7 +235,7 @@ mod tests {
     #[test]
     fn invalid_condition_param_skips_system() {
         for executor in EXECUTORS {
-            invalid_condition_param_skips_system_core(executor)
+            invalid_condition_param_skips_system_core(executor);
         }
     }
 

crates/bevy_ecs/src/system/system_param.rs

@@ -600,7 +600,7 @@ unsafe impl<'a, T: Resource> SystemParam for Res<'a, T> {
         _system_meta: &SystemMeta,
         world: &World,
     ) -> bool {
-        world.get_resource_by_id(component_id).is_some()
+        world.contains_resource_by_id(component_id)
     }
 
     #[inline]
@@ -711,7 +711,7 @@ unsafe impl<'a, T: Resource> SystemParam for ResMut<'a, T> {
         _system_meta: &SystemMeta,
         world: &World,
     ) -> bool {
-        world.get_resource_by_id(component_id).is_some()
+        world.contains_resource_by_id(component_id)
     }
 
     #[inline]
@@ -1279,7 +1279,7 @@ unsafe impl<'a, T: 'static> SystemParam for NonSend<'a, T> {
         _system_meta: &SystemMeta,
         world: &World,
     ) -> bool {
-        world.get_non_send_by_id(component_id).is_some()
+        world.contains_non_send_by_id(component_id)
     }
 
     #[inline]
@@ -1387,7 +1387,7 @@ unsafe impl<'a, T: 'static> SystemParam for NonSendMut<'a, T> {
         _system_meta: &SystemMeta,
         world: &World,
     ) -> bool {
-        world.get_non_send_by_id(component_id).is_some()
+        world.contains_non_send_by_id(component_id)
     }
 
     #[inline]

crates/bevy_ecs/src/world/mod.rs

@@ -1479,6 +1479,16 @@ impl World {
             .unwrap_or(false)
     }
 
+    /// Returns `true` if a resource with provided `component_id` exists. Otherwise returns `false`.
+    #[inline]
+    pub fn contains_resource_by_id(&self, component_id: ComponentId) -> bool {
+        self.storages
+            .resources
+            .get(component_id)
+            .map(ResourceData::is_present)
+            .unwrap_or(false)
+    }
+
     /// Returns `true` if a resource of type `R` exists. Otherwise returns `false`.
     #[inline]
     pub fn contains_non_send<R: 'static>(&self) -> bool {
@@ -1489,6 +1499,16 @@ impl World {
             .unwrap_or(false)
     }
 
+    /// Returns `true` if a resource with provided `component_id` exists. Otherwise returns `false`.
+    #[inline]
+    pub fn contains_non_send_by_id(&self, component_id: ComponentId) -> bool {
+        self.storages
+            .non_send_resources
+            .get(component_id)
+            .map(ResourceData::is_present)
+            .unwrap_or(false)
+    }
+
     /// Returns `true` if a resource of type `R` exists and was added since the world's
     /// [`last_change_tick`](World::last_change_tick()). Otherwise, this returns `false`.
     ///