From 11065b827ba70a005b70430e432c61a8cee28a46 Mon Sep 17 00:00:00 2001 From: nutcas3 Date: Fri, 1 May 2026 00:43:41 +0300 Subject: [PATCH] refactor: Simplify Redis RPS extraction using let-chains in monitoring_impl.rs - Replace nested if-let statements with let-chain pattern for Redis RPS parsing - Update module reference from monitoring to monitoring_impl in monitoring.rs --- apps/charging-engine/src/monitoring.rs | 2 +- apps/charging-engine/src/monitoring/monitoring_impl.rs | 10 ++++------ 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/apps/charging-engine/src/monitoring.rs b/apps/charging-engine/src/monitoring.rs index eb3a516..336bc6b 100644 --- a/apps/charging-engine/src/monitoring.rs +++ b/apps/charging-engine/src/monitoring.rs @@ -1,3 +1,3 @@ pub mod sync; pub mod types; -pub mod monitoring; +pub mod monitoring_impl; diff --git a/apps/charging-engine/src/monitoring/monitoring_impl.rs b/apps/charging-engine/src/monitoring/monitoring_impl.rs index 7308391..f889fcc 100644 --- a/apps/charging-engine/src/monitoring/monitoring_impl.rs +++ b/apps/charging-engine/src/monitoring/monitoring_impl.rs @@ -37,13 +37,11 @@ impl crate::charging::ChargingEngine { let info: String = redis::cmd("INFO").query_async(&mut conn).await.unwrap_or_default(); for line in info.lines() { - if line.starts_with("instantaneous_ops_per_sec:") { - if let Some(rps_str) = line.split(':').nth(1) { - if let Ok(rps) = rps_str.parse::() { - return Ok(rps); - } + if line.starts_with("instantaneous_ops_per_sec:") + && let Some(rps_str) = line.split(':').nth(1) + && let Ok(rps) = rps_str.parse::() { + return Ok(rps); } - } } let total_commands = self.extract_metric(&info, "total_commands_processed");