Consider using .clamp(min, max), instead of .max(min).min(max) or .min(max).max(min) RS-W1069
Anti-pattern
Minor
Chained Ord::min and Ord::max functions can be replaced with Ord::clamp, which is also a more readable solution.
Bad practice
fn get_in_range(value: u32, min: u32, max: u32) -> u32 {
value.max(min).min(max)
}
Recommended
fn get_in_range(value: u32, min: u32, max: u32) -> u32 {
value.clamp(min, max)
}