1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
|
### What it does
Checks for instances of `map_err(|_| Some::Enum)`
### Why is this bad?
This `map_err` throws away the original error rather than allowing the enum to contain and report the cause of the error
### Example
Before:
```
use std::fmt;
#[derive(Debug)]
enum Error {
Indivisible,
Remainder(u8),
}
impl fmt::Display for Error {
fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
match self {
Error::Indivisible => write!(f, "could not divide input by three"),
Error::Remainder(remainder) => write!(
f,
"input is not divisible by three, remainder = {}",
remainder
),
}
}
}
impl std::error::Error for Error {}
fn divisible_by_3(input: &str) -> Result<(), Error> {
input
.parse::<i32>()
.map_err(|_| Error::Indivisible)
.map(|v| v % 3)
.and_then(|remainder| {
if remainder == 0 {
Ok(())
} else {
Err(Error::Remainder(remainder as u8))
}
})
}
```
After:
```rust
use std::{fmt, num::ParseIntError};
#[derive(Debug)]
enum Error {
Indivisible(ParseIntError),
Remainder(u8),
}
impl fmt::Display for Error {
fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
match self {
Error::Indivisible(_) => write!(f, "could not divide input by three"),
Error::Remainder(remainder) => write!(
f,
"input is not divisible by three, remainder = {}",
remainder
),
}
}
}
impl std::error::Error for Error {
fn source(&self) -> Option<&(dyn std::error::Error + 'static)> {
match self {
Error::Indivisible(source) => Some(source),
_ => None,
}
}
}
fn divisible_by_3(input: &str) -> Result<(), Error> {
input
.parse::<i32>()
.map_err(Error::Indivisible)
.map(|v| v % 3)
.and_then(|remainder| {
if remainder == 0 {
Ok(())
} else {
Err(Error::Remainder(remainder as u8))
}
})
}
```
|