diff options
| author | k12ish <45272873+k12ish@users.noreply.github.com> | 2021-04-20 10:18:05 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-04-20 11:18:05 +0200 |
| commit | b4de6594380636817d13c2677ec6f472a964cf43 (patch) | |
| tree | ed08f5807318e60942e81dc917b749c5e452f8d3 /exercises/option | |
| parent | a37a8818c85f6b99d9891e39f900292c45e6c7e7 (diff) | |
fix(option2): Rename uninformative variables (#675)
Renaming uninformative names like `optional_value`, `value`, `optional_values_vec` and `value` helps users distinguish between the two parts of the task.
Diffstat (limited to 'exercises/option')
| -rw-r--r-- | exercises/option/option2.rs | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/exercises/option/option2.rs b/exercises/option/option2.rs index a1517d7..c6b83ec 100644 --- a/exercises/option/option2.rs +++ b/exercises/option/option2.rs @@ -4,22 +4,22 @@ // I AM NOT DONE fn main() { - let optional_value = Some(String::from("rustlings")); + let optional_word = Some(String::from("rustlings")); // TODO: Make this an if let statement whose value is "Some" type - value = optional_value { - println!("the value of optional value is: {}", value); + word = optional_word { + println!("The word is: {}", word); } else { - println!("The optional value doesn't contain anything!"); + println!("The optional word doesn't contain anything"); } - let mut optional_values_vec: Vec<Option<i8>> = Vec::new(); + let mut optional_integers_vec: Vec<Option<i8>> = Vec::new(); for x in 1..10 { - optional_values_vec.push(Some(x)); + optional_integers_vec.push(Some(x)); } // TODO: make this a while let statement - remember that vector.pop also adds another layer of Option<T> // You can stack `Option<T>`'s into while let and if let - value = optional_values_vec.pop() { - println!("current value: {}", value); + integer = optional_integers_vec.pop() { + println!("current value: {}", integer); } } |
