Решение на FMI Buzz от Микаела Стоичкова

Обратно към всички решения

Към профила на Микаела Стоичкова

Резултати

  • 15 точки от тестове
  • 0 бонус точки
  • 15 точки общо
  • 10 успешни тест(а)
  • 0 неуспешни тест(а)

Код

pub fn fizzbuzz(n: usize) -> Vec<String> {
custom_buzz(n, 3, 5)
}
pub fn custom_buzz(n: usize, k1: u8, k2: u8) -> Vec<String> {
let fizzbuzzer = FizzBuzzer {
k1: k1,
k2: k2,
labels: [
String::from("Fizz"),
String::from("Buzz"),
String::from("Fizzbuzz")
],
};
fizzbuzzer.take(n)
}
pub struct FizzBuzzer {
pub k1: u8,
pub k2: u8,
pub labels: [String; 3],
}
impl FizzBuzzer {
fn is_divisible(n : usize, div : usize) -> bool {
n % div == 0
}
/// За всяко число от 1 до `n` включително, искаме съответстващия елемент в резултата да е:
///
/// - Първия String от полето `labels` ако числото се дели на k1, но не на k2
/// - Втория String от полето `labels` ако числото се дели на k2, но не на k1
/// - Третия String от полето `labels` ако числото се дели и на k1, и на k2
/// - Числото конвертирано до низ, във всички други случаи
///
/// Ако `n` е 0, очакваме празен вектор за резултат.
/// Ако `k1` или `k2` са 0 или 1, очакваме функцията да panic-не с каквото съобщение изберете.
///
pub fn take(&self, n: usize) -> Vec<String> {
if self.k1 < 2 || self.k2 < 2 {
panic!("Invalid divisors");
}
let mut vec: Vec<String> = Vec::new();
let k1 = self.k1 as usize;
let k2 = self.k2 as usize;
for number in 1..n+1 {
if !FizzBuzzer::is_divisible(number, k1) && !FizzBuzzer::is_divisible(number, k2){

Би могла да използваш Self::, вместо FizzBuzzer::, за да избегнеш повторението на потенциално дълго (и потенциално променящо се в бъдеще) име на тип. Бих казал, че е въпрос на вкус -- просто да го знаеш като опция.

vec.push(String::from(number.to_string()));
} else if FizzBuzzer::is_divisible(number, k1) && !FizzBuzzer::is_divisible(number, k2) {
vec.push(self.labels[0].clone());
} else if FizzBuzzer::is_divisible(number, k2) && !FizzBuzzer::is_divisible(number,k1) {
vec.push(self.labels[1].clone());
} else {
vec.push(self.labels[2].clone());
}
}
vec
}
pub fn change_label(&mut self, index: usize, value: &String) {
if index > 2 {
panic!("Invalid index");
}
self.labels[index] = value.clone()
}
}

Лог от изпълнението

Compiling solution v0.1.0 (/tmp/d20201028-2816268-1sx2pck/solution)
    Finished test [unoptimized + debuginfo] target(s) in 2.16s
     Running target/debug/deps/solution-ebb42508826ef2b4

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

     Running target/debug/deps/solution_test-9e954a53ed808c89

running 10 tests
test solution_test::test_change_label_basic ... ok
test solution_test::test_change_label_invalid ... ok
test solution_test::test_classic1 ... ok
test solution_test::test_classic2 ... ok
test solution_test::test_coefficients1 ... ok
test solution_test::test_coefficients2 ... ok
test solution_test::test_coefficients_invalid ... ok
test solution_test::test_struct_basic ... ok
test solution_test::test_struct_invalid ... ok
test solution_test::test_zeroes ... ok

test result: ok. 10 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

   Doc-tests solution

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

История (2 версии и 1 коментар)

Микаела качи първо решение на 25.10.2020 15:21 (преди почти 5 години)

Микаела качи решение на 26.10.2020 08:38 (преди почти 5 години)

-pub fn is_divisible(n : usize, div : usize) -> bool {
- n % div == 0
-}
pub fn fizzbuzz(n: usize) -> Vec<String> {
custom_buzz(n, 3, 5)
}
-
+
pub fn custom_buzz(n: usize, k1: u8, k2: u8) -> Vec<String> {
let fizzbuzzer = FizzBuzzer {
k1: k1,
k2: k2,
labels: [
String::from("Fizz"),
String::from("Buzz"),
String::from("Fizzbuzz")
],
};
fizzbuzzer.take(n)
}
-
-
+
pub struct FizzBuzzer {
pub k1: u8,
pub k2: u8,
pub labels: [String; 3],
}
-
+
impl FizzBuzzer {
+ fn is_divisible(n : usize, div : usize) -> bool {
+ n % div == 0
+ }
/// За всяко число от 1 до `n` включително, искаме съответстващия елемент в резултата да е:
///
/// - Първия String от полето `labels` ако числото се дели на k1, но не на k2
/// - Втория String от полето `labels` ако числото се дели на k2, но не на k1
/// - Третия String от полето `labels` ако числото се дели и на k1, и на k2
/// - Числото конвертирано до низ, във всички други случаи
///
/// Ако `n` е 0, очакваме празен вектор за резултат.
/// Ако `k1` или `k2` са 0 или 1, очакваме функцията да panic-не с каквото съобщение изберете.
///
pub fn take(&self, n: usize) -> Vec<String> {
if self.k1 < 2 || self.k2 < 2 {
panic!("Invalid divisors");
}
let mut vec: Vec<String> = Vec::new();
let k1 = self.k1 as usize;
let k2 = self.k2 as usize;
-
+
for number in 1..n+1 {
- if !is_divisible(number, k1) && !is_divisible(number, k2){
+ if !FizzBuzzer::is_divisible(number, k1) && !FizzBuzzer::is_divisible(number, k2){

Би могла да използваш Self::, вместо FizzBuzzer::, за да избегнеш повторението на потенциално дълго (и потенциално променящо се в бъдеще) име на тип. Бих казал, че е въпрос на вкус -- просто да го знаеш като опция.

vec.push(String::from(number.to_string()));
- } else if is_divisible(number, k1) && !is_divisible(number, k2) {
+ } else if FizzBuzzer::is_divisible(number, k1) && !FizzBuzzer::is_divisible(number, k2) {
vec.push(self.labels[0].clone());
- } else if is_divisible(number, k2) && !is_divisible(number,k1) {
+ } else if FizzBuzzer::is_divisible(number, k2) && !FizzBuzzer::is_divisible(number,k1) {
vec.push(self.labels[1].clone());
} else {
vec.push(self.labels[2].clone());
}
}
vec
}
-
+
pub fn change_label(&mut self, index: usize, value: &String) {
if index > 2 {
panic!("Invalid index");
}
self.labels[index] = value.clone()
}
-}
-
+}