torsten/yourpart-daemon
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Refactor variable naming in DirectorWorker: Updated variable names to improve code clarity by prefixing with underscores for unused variables. This change enhances readability and maintains functionality without altering logic.

Browse Source
This commit is contained in:
Torsten Schulz (local)
2026-02-04 14:50:39 +01:00
parent d94fa791b3
commit deaf5ac6ca
1 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/worker/director.rs
View File

@@ -777,7 +777,7 @@ impl DirectorWorker {
let mut best_target_region: Option<i32> = None; let mut best_target_region: Option<i32> = None;
let mut best_quantity: i32 = 0; let mut best_quantity: i32 = 0;
let mut best_remote_piece_price: f64 = 0.0; let mut _best_remote_piece_price: f64 = 0.0;
let mut best_gain: f64 = 0.0; let mut best_gain: f64 = 0.0;
// Für jede andere Region prüfen, ob sich ein Transport lohnt. // Für jede andere Region prüfen, ob sich ein Transport lohnt.
@@ -827,7 +827,7 @@ impl DirectorWorker {
best_gain = net_gain; best_gain = net_gain;
best_target_region = Some(region_id); best_target_region = Some(region_id);
best_quantity = qty; best_quantity = qty;
best_remote_piece_price = remote_piece_price; _best_remote_piece_price = remote_piece_price;
} }
} }
@@ -877,7 +877,7 @@ impl DirectorWorker {
&[&falukant_user_id, &source_region], &[&falukant_user_id, &source_region],
)?; )?;
let vehicle_count = vehicle_count_rows let _vehicle_count = vehicle_count_rows
.into_iter() .into_iter()
.next() .next()
.and_then(|row| row.get("count").and_then(|v| v.parse::<i32>().ok())) .and_then(|row| row.get("count").and_then(|v| v.parse::<i32>().ok()))
@@ -891,7 +891,7 @@ impl DirectorWorker {
&[&source_region, &target_region], &[&source_region, &target_region],
)?; )?;
let route_exists = route_rows let _route_exists = route_rows
.into_iter() .into_iter()
.next() .next()
.and_then(|row| row.get("count").and_then(|v| v.parse::<i32>().ok())) .and_then(|row| row.get("count").and_then(|v| v.parse::<i32>().ok()))
@@ -1090,7 +1090,7 @@ impl DirectorWorker {
// Wähle den Branch mit dem besten Preisvorteil (oder einfach den ersten, wenn alle gleich sind) // Wähle den Branch mit dem besten Preisvorteil (oder einfach den ersten, wenn alle gleich sind)
branches_with_vehicles.sort_by(|a, b| b.3.partial_cmp(&a.3).unwrap_or(std::cmp::Ordering::Equal)); branches_with_vehicles.sort_by(|a, b| b.3.partial_cmp(&a.3).unwrap_or(std::cmp::Ordering::Equal));
let (target_branch_id, target_region_id, vehicle_count, price_delta) = branches_with_vehicles[0]; let (_target_branch_id, target_region_id, _vehicle_count, _price_delta) = branches_with_vehicles[0];
// Hole die Fahrzeuge nochmal für diesen Branch // Hole die Fahrzeuge nochmal für diesen Branch
@@ -1102,13 +1102,13 @@ impl DirectorWorker {
)?; )?;
// Leere Transporte für alle verfügbaren Fahrzeuge anlegen // Leere Transporte für alle verfügbaren Fahrzeuge anlegen
let mut transport_count = 0; let mut _transport_count = 0;
for vehicle in &vehicles { for vehicle in &vehicles {
conn.execute( conn.execute(
"insert_empty_transport", "insert_empty_transport",
&[&target_region_id, &current_region_id, &vehicle.id], &[&target_region_id, &current_region_id, &vehicle.id],
)?; )?;
transport_count += 1; _transport_count += 1;
} }
@@ -1144,7 +1144,7 @@ impl DirectorWorker {
rows.into_iter().filter_map(Self::map_row_to_salary_item).collect(); rows.into_iter().filter_map(Self::map_row_to_salary_item).collect();
for item in salaries { for item in salaries {
if let Err(err) = self.base.change_falukant_user_money( if let Err(_err) = self.base.change_falukant_user_money(
item.employer_user_id, item.employer_user_id,
-(item.income as f64), -(item.income as f64),
"director payed out", "director payed out",